Introduces a search term extraction mechanism working for arbitrary search providers.

chrome/browser/google/google_util_unittest.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/command_line.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/google/google_url_tracker.h"
 #include "chrome/browser/google/google_util.h"
 #include "chrome/common/chrome_switches.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 using google_util::IsGoogleDomainUrl;
 using google_util::IsGoogleHomePageUrl;
 using google_util::IsGoogleSearchUrl;
-using google_util::IsInstantExtendedAPIGoogleSearchUrl;
 
 TEST(GoogleUtilTest, GoodHomePagesNonSecure) {
   // Valid home page hosts.
   EXPECT_TRUE(IsGoogleHomePageUrl(GoogleURLTracker::kDefaultGoogleHomepage));
   EXPECT_TRUE(IsGoogleHomePageUrl("http://google.com"));
   EXPECT_TRUE(IsGoogleHomePageUrl("http://www.google.com"));
   EXPECT_TRUE(IsGoogleHomePageUrl("http://www.google.ca"));
   EXPECT_TRUE(IsGoogleHomePageUrl("http://www.google.co.uk"));
   EXPECT_TRUE(IsGoogleHomePageUrl("http://www.google.com:80/"));
 
@@ skipping 214 matching lines @@
       "http://www.google.com/webhp/nogood#q=something"));
   EXPECT_FALSE(IsGoogleSearchUrl(""));
 
   // Case sensitive paths.
   EXPECT_FALSE(IsGoogleSearchUrl(
       "http://www.google.com/SEARCH?q=something"));
   EXPECT_FALSE(IsGoogleSearchUrl(
       "http://www.google.com/WEBHP#q=something"));
 }
 
-TEST(GoogleUtilTest, IsInstantExtendedAPIGoogleSearchUrl) {
-  EXPECT_TRUE(IsInstantExtendedAPIGoogleSearchUrl(
-        "http://www.google.com/search?q=something&espv=1"));
-  EXPECT_TRUE(IsInstantExtendedAPIGoogleSearchUrl(
-        "http://www.google.com/search?q=something&espv=3"));
-  EXPECT_TRUE(IsInstantExtendedAPIGoogleSearchUrl(
-        "http://www.google.com/search?q=something&espv=42"));
-
-  EXPECT_FALSE(IsInstantExtendedAPIGoogleSearchUrl(
-        "http://www.google.com/search?q=something&espv="));
-  EXPECT_FALSE(IsInstantExtendedAPIGoogleSearchUrl(
-        "http://www.google.com/search?q=something&espv=0"));
-  EXPECT_FALSE(IsInstantExtendedAPIGoogleSearchUrl(
-        "http://www.google.com/search?q=something&espv=00"));
-
-  EXPECT_FALSE(IsInstantExtendedAPIGoogleSearchUrl(
-        "http://www.foo.com/search?q=something&espv=0"));
-  EXPECT_FALSE(IsInstantExtendedAPIGoogleSearchUrl(
-        "http://www.foo.com/search?q=something&espv=1"));
-  EXPECT_FALSE(IsInstantExtendedAPIGoogleSearchUrl(
-        "http://www.google.com/bing?q=something&espv=1"));
-  EXPECT_FALSE(IsInstantExtendedAPIGoogleSearchUrl(
-        "http://www.google.com/search?q=something&vespv=1"));
-  EXPECT_FALSE(IsInstantExtendedAPIGoogleSearchUrl(
-        "http://www.google.com/search?q=something&espvx=1"));
-  EXPECT_FALSE(IsInstantExtendedAPIGoogleSearchUrl(
-        "http://www.google.com/search#q=something&espv=1"));
-}
-
 TEST(GoogleUtilTest, GoogleDomains) {
   // Test some good Google domains (valid TLDs).
   EXPECT_TRUE(IsGoogleDomainUrl("http://www.google.com",
                                 google_util::ALLOW_SUBDOMAIN,
                                 google_util::DISALLOW_NON_STANDARD_PORTS));
   EXPECT_TRUE(IsGoogleDomainUrl("http://google.com",
                                 google_util::ALLOW_SUBDOMAIN,
                                 google_util::DISALLOW_NON_STANDARD_PORTS));
   EXPECT_TRUE(IsGoogleDomainUrl("http://www.google.ca",
                                 google_util::ALLOW_SUBDOMAIN,
@@ skipping 80 matching lines @@
   EXPECT_TRUE(IsGoogleDomainUrl("http://test.foo.com:1234",
                                 google_util::DISALLOW_SUBDOMAIN,
                                 google_util::DISALLOW_NON_STANDARD_PORTS));
   EXPECT_FALSE(IsGoogleDomainUrl("file://test.foo.com:1234",
                                  google_util::DISALLOW_SUBDOMAIN,
                                  google_util::DISALLOW_NON_STANDARD_PORTS));
   EXPECT_TRUE(IsGoogleDomainUrl("http://www.google.com",
                                 google_util::DISALLOW_SUBDOMAIN,
                                 google_util::DISALLOW_NON_STANDARD_PORTS));
 }
-
-TEST(GoogleUtilTest, SearchTerms) {
-  // Simple searches.
-  EXPECT_EQ(string16(), google_util::GetSearchTermsFromGoogleSearchURL(
-      "http://google.com/search?q=tractor+supply"));
-  EXPECT_EQ(ASCIIToUTF16("tractor supply"),
-            google_util::GetSearchTermsFromGoogleSearchURL(
-                "http://google.com/search?q=tractor+supply&espv=1"));
-  // espv=1 only applies in query.
-  EXPECT_EQ(string16(), google_util::GetSearchTermsFromGoogleSearchURL(
-      "http://google.com/search?q=potato#espv=1"));
-
-  // Instant searches.
-  EXPECT_EQ(string16(), google_util::GetSearchTermsFromGoogleSearchURL(
-      "http://google.com/webhp#q=tractor+supply"));
-  EXPECT_EQ(ASCIIToUTF16("tractor supply"),
-            google_util::GetSearchTermsFromGoogleSearchURL(
-                "http://google.com/webhp?espv=1#q=tractor+supply"));
-  // espv=1 only applies in query.
-  EXPECT_EQ(string16(), google_util::GetSearchTermsFromGoogleSearchURL(
-      "http://google.com/webhp?#espv=1&q=potato"));
-
-  // Both query and ref components have a search term.
-  EXPECT_EQ(ASCIIToUTF16("tractor supply"),
-            google_util::GetSearchTermsFromGoogleSearchURL(
-                "http://google.com/webhp?q=potato&espv=1#q=tractor+supply"));
-
-  // Blank queries.
-  EXPECT_EQ(string16(), google_util::GetSearchTermsFromGoogleSearchURL(
-      "http://google.com/search?q=&q=potato&espv=1"));
-  EXPECT_EQ(string16(), google_util::GetSearchTermsFromGoogleSearchURL(
-      "http://google.com/webhp?espv=1#q=&q=tractor+supply"));
-
-  // Multiple non-empty queries.
-  EXPECT_EQ(ASCIIToUTF16("tractor supply"),
-            google_util::GetSearchTermsFromGoogleSearchURL(
-                "http://google.com/search?q=tractor+supply&q=potato&espv=1"));
-  EXPECT_EQ(ASCIIToUTF16("tractor supply"),
-            google_util::GetSearchTermsFromGoogleSearchURL(
-                "http://google.com/webhp?espv=1#q=tractor+supply&q=potato"));
-
-  // Blank terms in ref override non-blank terms in query.
-  EXPECT_EQ(string16(), google_util::GetSearchTermsFromGoogleSearchURL(
-      "http://google.com/search?q=potato&espv=1#q="));
-
-  // Blank terms in query do not override non-blank terms in ref.
-  EXPECT_EQ(ASCIIToUTF16("tractor supply"),
-            google_util::GetSearchTermsFromGoogleSearchURL(
-                "http://google.com/search?q=&espv=1#q=tractor+supply"));
-}