2009-03-13 Mike Belshe <mike@belse.com>
authordglazkov@chromium.org <dglazkov@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 13 Mar 2009 22:42:29 +0000 (22:42 +0000)
committerdglazkov@chromium.org <dglazkov@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 13 Mar 2009 22:42:29 +0000 (22:42 +0000)
        Reviewed by Dimitri Glazkov.

        https://bugs.webkit.org/show_bug.cgi?id=24580
        Fix query() to match KURL behavior.

        * platform/KURLGoogle.cpp:
        (WebCore::KURL::query): remove extra logic around question mark.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41689 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebCore/ChangeLog
WebCore/platform/KURLGoogle.cpp

index 0ad7e3d..0b5c7fe 100644 (file)
@@ -1,3 +1,13 @@
+2009-03-13  Mike Belshe  <mike@belse.com>
+
+        Reviewed by Dimitri Glazkov.
+
+        https://bugs.webkit.org/show_bug.cgi?id=24580
+        Fix query() to match KURL behavior.
+
+        * platform/KURLGoogle.cpp:
+        (WebCore::KURL::query): remove extra logic around question mark.
+
 2009-03-13  Chris Fleizach  <cfleizach@apple.com>
 
         Reviewed by Beth Dakin.
index 382d7a1..60dc4c8 100644 (file)
@@ -519,15 +519,8 @@ bool KURL::hasRef() const
 
 String KURL::query() const
 {
-    if (m_url.m_parsed.query.len >= 0) {
-        // KURL's query() includes the question mark, even though the reference
-        // doesn't. Move the query component backwards one to account for it
-        // (our library doesn't count the question mark).
-        url_parse::Component queryComp = m_url.m_parsed.query;
-        queryComp.begin--;
-        queryComp.len++;
-        return m_url.componentString(queryComp);
-    }
+    if (m_url.m_parsed.query.len >= 0)
+        return m_url.componentString(query);
 
     // Bug: https://bugs.webkit.org/show_bug.cgi?id=21015 this function returns
     // an empty string when the query is empty rather than a null (not sure