Bommels05/Mozilla
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Bug 135679 Form submit doesn't work if target is same page and current url has anchor

Browse Source 
patch by ere@atp.fi r=jkeiser sr=attinasi a=rjesup


git-svn-id: svn://10.0.0.236/trunk@118617 18797224-902f-48f8-a5cc-f745e15eee43
This commit is contained in:
timeless%mac.com
2002-04-10 13:56:30 +00:00
parent cbbfc8af11
commit b4fe5cdb58
2 changed files with 12 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
mozilla/docshell/base/nsDocShell.cpp
View File

@@ -5132,12 +5132,17 @@ nsDocShell::ScrollIfAnchor(nsIURI * aURI, PRBool * aWasAnchor, PRUint32 aLoadTyp
newSpec.BeginReading(urlStart);
newSpec.EndReading(refEnd);
// hasAnchor is used to check if the new URI really has anchor
// We can't always scroll, it would break submitting a form that has
// same target page.
PRBool hasAnchor = PR_FALSE;
PRInt32 hashNew = newSpec.FindChar(kHash);
if (hashNew == 0) {
return NS_OK; // Strange URI
}
else if (hashNew > 0) {
// found it
hasAnchor = PR_TRUE;
urlEnd = urlStart;
urlEnd.advance(hashNew);
@@ -5254,7 +5259,8 @@ nsDocShell::ScrollIfAnchor(nsIURI * aURI, PRBool * aWasAnchor, PRUint32 aLoadTyp
}
}
else {
*aWasAnchor = PR_TRUE;
// Tell there was an anchor only if there really was one (bug 135679)
*aWasAnchor = hasAnchor;
}
return NS_OK;