Fix unittest to match new querystring parsing rules....

webkit/tools/webcore_unit_tests/GKURL_unittest.cpp

 // Copyright (c) 2008, Google Inc.
 // All rights reserved.
 // 
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 // 
 //     * Redistributions of source code must retain the above copyright
 // notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
@@ skipping 65 matching lines @@
     const char* protocol;
     const char* host;
     int port;
     const char* user;
     const char* pass;
     const char* last_path_component;
     const char* query;
     const char* ref;
     bool has_ref;
   } cases[] = {
-    {"http://www.google.com/foo/blah?bar=baz#ref", "http", "www.google.com", 0, "", NULL, "blah", "?bar=baz", "ref", true},
+    {"http://www.google.com/foo/blah?bar=baz#ref", "http", "www.google.com", 0, "", NULL, "blah", "bar=baz", "ref", true},
     {"http://foo.com:1234/foo/bar/", "http", "foo.com", 1234, "", NULL, "bar", "", NULL, false},
-    {"http://www.google.com?#", "http", "www.google.com", 0, "", NULL, NULL, "?", "", true},
+    {"http://www.google.com?#", "http", "www.google.com", 0, "", NULL, NULL, "", "", true},
     {"https://me:pass@google.com:23#foo", "https", "google.com", 23, "me", "pass", NULL, "", "foo", true},
     {"javascript:hello!//world", "javascript", "", 0, "", NULL, "world", "", NULL, false},
   };
 
   for (size_t i = 0; i < ARRAYSIZE_UNSAFE(cases); i++) {
     // UTF-8
     WebCore::KURL gurl(cases[i].url);
 
     EXPECT_EQ(cases[i].protocol, gurl.protocol());
     EXPECT_EQ(cases[i].host, gurl.host());
@@ skipping 35 matching lines @@
     {"javascript:hello!?#/\\world",           "javascript", "",         0,    "",   NULL,   "hello!?#/\\world", "world",  "",      NULL},
 
     // Old WebKit doesn't handle "parameters" in paths, so will
     // disagree with us about where the path is for this URL.
     {"http://a.com/hello;world",              "http",       "a.com",    0,    "",   NULL,     "/hello;world",     "hello",  "",    NULL},
 
     // WebKit doesn't like UTF-8 or UTF-16 input.
     {"http://\xe4\xbd\xa0\xe5\xa5\xbd\xe4\xbd\xa0\xe5\xa5\xbd/", "http", "xn--6qqa088eba", 0, "", NULL, "/",   NULL,     "",       NULL},
 
     // WebKit %-escapes non-ASCII characters in reference, but we don't.
-    {"http://www.google.com/foo/blah?bar=baz#\xce\xb1\xce\xb2", "http", "www.google.com", 0, "", NULL, "/foo/blah/", "blah", "?bar=baz", "\xce\xb1\xce\xb2"}
+    {"http://www.google.com/foo/blah?bar=baz#\xce\xb1\xce\xb2", "http", "www.google.com", 0, "", NULL, "/foo/blah/", "blah", "bar=baz", "\xce\xb1\xce\xb2"}
   };
 
   for (size_t i = 0; i < arraysize(cases); i++) {
     WebCore::KURL gurl(cases[i].url);
 
     EXPECT_EQ(cases[i].protocol, gurl.protocol());
     EXPECT_EQ(cases[i].host, gurl.host());
     EXPECT_EQ(cases[i].port, gurl.port());
     EXPECT_EQ(cases[i].user, gurl.user());
     EXPECT_EQ(cases[i].pass, gurl.pass());
@@ skipping 436 matching lines @@
   const char url[] = "http://www.google.com/";
   WebCore::KURL src(url);
   EXPECT_TRUE(src.string() == url);  // This really just initializes the cache.
   WebCore::KURL dest = src.copy();
   EXPECT_TRUE(dest.string() == url);  // This really just initializes the cache.
 
   // The pointers should be different for both UTF-8 and UTF-16.
   EXPECT_NE(dest.string().characters(), src.string().characters());
   EXPECT_NE(dest.utf8String().data(), src.utf8String().data());
 }