From 591cd3bd142e5c9f4e46c65ca68aba2371fb8b1b Mon Sep 17 00:00:00 2001
From: "pollmann%netscape.com"
 <pollmann%netscape.com@18797224-902f-48f8-a5cc-f745e15eee43>
Date: Thu, 3 Aug 2000 23:03:58 +0000
Subject: [PATCH] Bug 42616: form actions with anchor specified (#) do not
 work; r=kmcclusk

git-svn-id: svn://10.0.0.236/trunk@75546 18797224-902f-48f8-a5cc-f745e15eee43
---
 mozilla/layout/html/forms/src/nsFormFrame.cpp | 18 ++++++++++++++++--
 1 file changed, 16 insertions(+), 2 deletions(-)

diff --git a/mozilla/layout/html/forms/src/nsFormFrame.cpp b/mozilla/layout/html/forms/src/nsFormFrame.cpp
index bb0a64627cd..22f85a0baf7 100644
--- a/mozilla/layout/html/forms/src/nsFormFrame.cpp
+++ b/mozilla/layout/html/forms/src/nsFormFrame.cpp
@@ -819,8 +819,17 @@ nsFormFrame::OnSubmit(nsIPresContext* aPresContext, nsIFrame* aFrame)
     }
     nsAutoString theScheme; theScheme.AssignWithConversion( NS_STATIC_CAST(const char*, scheme) );
     // Append the URI encoded variable/value pairs for GET's
-    if (!theScheme.EqualsIgnoreCase("javascript")) { // Not for JS URIs, see bug 26917
-      if (!isPost) {
+    if (!isPost) {
+      if (!theScheme.EqualsIgnoreCase("javascript")) { // Not for JS URIs, see bug 26917
+
+        // Bug 42616: Trim off named anchor before query string
+        PRInt32 namedAnchorPos = href.FindChar('#', PR_FALSE, 0);
+        nsAutoString namedAnchor;
+        if (kNotFound != namedAnchorPos) {
+          href.Right(namedAnchor, namedAnchorPos);
+          href.Truncate(namedAnchorPos);
+        }
+
         if (href.FindChar('?', PR_FALSE, 0) == kNotFound) { // Add a ? if needed
           href.AppendWithConversion('?');
         } else {                              // Adding to existing query string
@@ -829,6 +838,11 @@ nsFormFrame::OnSubmit(nsIPresContext* aPresContext, nsIFrame* aFrame)
           }
         }
         href.Append(data);
+
+        // Bug 42616: Add named anchor to end after query string
+        if (namedAnchor.Length()) {
+          href.Append(namedAnchor);
+        }
       }
     }
     nsAutoString absURLSpec;