Move Google search related util methods to page_load_metrics_util

chrome/browser/page_load_metrics/observers/from_gws_page_load_metrics_observer_unittest.cc

 // Copyright 2015 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 "chrome/browser/page_load_metrics/observers/from_gws_page_load_metrics_observer.h"
 
 #include "base/macros.h"
 #include "base/memory/ptr_util.h"
 #include "chrome/browser/page_load_metrics/observers/page_load_metrics_observer_test_harness.h"
 #include "chrome/browser/page_load_metrics/page_load_metrics_util.h"
@@ skipping 714 matching lines @@
 
 TEST_F(FromGWSPageLoadMetricsObserverTest, ProvisionalIntent) {
   NavigateAndCommit(GURL(kGoogleSearchResultsUrl));
   StartNavigation(GURL("intent://en.m.wikipedia.org/wiki/Test"));
   // Simulate closing the tab.
   DeleteContents();
   histogram_tester().ExpectTotalCount(
       internal::kHistogramFromGWSAbortCloseBeforeCommit, 0);
 }
 
-TEST_F(FromGWSPageLoadMetricsLoggerTest, IsGoogleSearchHostname) {
-  struct {
-    bool expected_result;
-    const char* url;
-  } test_cases[] = {
-      {true, "https://www.google.com/"},
-      {true, "https://www.google.co.uk/"},
-      {true, "https://www.google.co.in/"},
-      {false, "https://other.google.com/"},
-      {false, "https://other.www.google.com/"},
-      {false, "https://www.other.google.com/"},
-      {false, "https://www.www.google.com/"},
-      {false, "https://www.google.appspot.com/"},
-      {false, "https://www.google.example.com/"},
-      // Search results are not served from the bare google.com domain.
-      {false, "https://google.com/"},
-  };
-  for (const auto& test : test_cases) {
-    EXPECT_EQ(
-        test.expected_result,
-        FromGWSPageLoadMetricsLogger::IsGoogleSearchHostname(GURL(test.url)))
-        << "for URL: " << test.url;
-  }
-}
-
-TEST_F(FromGWSPageLoadMetricsLoggerTest, IsGoogleSearchResultUrl) {
-  struct {
-    bool expected_result;
-    const char* url;
-  } test_cases[] = {
-      {true, "https://www.google.com/#q=test"},
-      {true, "https://www.google.com/search#q=test"},
-      {true, "https://www.google.com/search?q=test"},
-      {true, "https://www.google.com/webhp#q=test"},
-      {true, "https://www.google.com/webhp?q=test"},
-      {true, "https://www.google.com/webhp?a=b&q=test"},
-      {true, "https://www.google.com/webhp?a=b&q=test&c=d"},
-      {true, "https://www.google.com/webhp#a=b&q=test&c=d"},
-      {true, "https://www.google.com/webhp?#a=b&q=test&c=d"},
-      {false, "https://www.google.com/"},
-      {false, "https://www.google.com/about/"},
-      {false, "https://other.google.com/"},
-      {false, "https://other.google.com/webhp?q=test"},
-      {false, kExampleUrl},
-      {false, "https://www.example.com/webhp?q=test"},
-      {false, "https://google.com/#q=test"},
-  };
-  for (const auto& test : test_cases) {
-    EXPECT_EQ(
-        test.expected_result,
-        FromGWSPageLoadMetricsLogger::IsGoogleSearchResultUrl(GURL(test.url)))
-        << "for URL: " << test.url;
-  }
-}
-
-TEST_F(FromGWSPageLoadMetricsLoggerTest, IsGoogleSearchRedirectorUrl) {
-  struct {
-    bool expected_result;
-    const char* url;
-  } test_cases[] = {
-      {true, "https://www.google.com/url?source=web"},
-      {true, "https://www.google.com/url?source=web#foo"},
-      {true, "https://www.google.com/searchurl/r.html#foo"},
-      {true, "https://www.google.com/url?a=b&source=web&c=d"},
-      {false, "https://www.google.com/?"},
-      {false, "https://www.google.com/?url"},
-      {false, "https://www.example.com/url?source=web"},
-      {false, "https://google.com/url?"},
-      {false, "https://www.google.com/?source=web"},
-      {false, "https://www.google.com/source=web"},
-      {false, "https://www.example.com/url?source=web"},
-      {false, "https://www.google.com/url?"},
-      {false, "https://www.google.com/url?a=b"},
-  };
-  for (const auto& test : test_cases) {
-    EXPECT_EQ(test.expected_result,
-              FromGWSPageLoadMetricsLogger::IsGoogleSearchRedirectorUrl(
-                  GURL(test.url)))
-        << "for URL: " << test.url;
-  }
-}
-
-TEST_F(FromGWSPageLoadMetricsLoggerTest, QueryContainsComponent) {
-  struct {
-    bool expected_result;
-    const char* query;
-    const char* component;
-  } test_cases[] = {
-      {true, "a=b", "a=b"},
-      {true, "a=b&c=d", "a=b"},
-      {true, "a=b&c=d", "c=d"},
-      {true, "a=b&c=d&e=f", "c=d"},
-      {true, "za=b&a=b", "a=b"},
-      {true, "a=bz&a=b", "a=b"},
-      {true, "a=ba=b&a=b", "a=b"},
-      {true, "a=a=a&a=a", "a=a"},
-      {true, "source=web", "source=web"},
-      {true, "a=b&source=web", "source=web"},
-      {true, "a=b&source=web&c=d", "source=web"},
-      {false, "a=a=a", "a=a"},
-      {false, "", ""},
-      {false, "a=b", ""},
-      {false, "", "a=b"},
-      {false, "za=b", "a=b"},
-      {false, "za=bz", "a=b"},
-      {false, "a=bz", "a=b"},
-      {false, "za=b&c=d", "a=b"},
-      {false, "a=b&c=dz", "c=d"},
-      {false, "a=b&zc=d&e=f", "c=d"},
-      {false, "a=b&c=dz&e=f", "c=d"},
-      {false, "a=b&zc=dz&e=f", "c=d"},
-      {false, "a=b&foosource=web&c=d", "source=web"},
-      {false, "a=b&source=webbar&c=d", "source=web"},
-      {false, "a=b&foosource=webbar&c=d", "source=web"},
-  };
-  for (const auto& test : test_cases) {
-    EXPECT_EQ(test.expected_result,
-              FromGWSPageLoadMetricsLogger::QueryContainsComponent(
-                  test.query, test.component))
-        << "For query: " << test.query << " with component: " << test.component;
-  }
-}
-
-TEST_F(FromGWSPageLoadMetricsLoggerTest, QueryContainsComponentPrefix) {
-  struct {
-    bool expected_result;
-    const char* query;
-    const char* component;
-  } test_cases[] = {
-      {true, "a=b", "a="},
-      {true, "a=b&c=d", "a="},
-      {true, "a=b&c=d", "c="},
-      {true, "a=b&c=d&e=f", "c="},
-      {true, "za=b&a=b", "a="},
-      {true, "ba=a=b&a=b", "a="},
-      {true, "q=test", "q="},
-      {true, "a=b&q=test", "q="},
-      {true, "q=test&c=d", "q="},
-      {true, "a=b&q=test&c=d", "q="},
-      {false, "", ""},
-      {false, "za=b", "a="},
-      {false, "za=b&c=d", "a="},
-      {false, "a=b&zc=d", "c="},
-      {false, "a=b&zc=d&e=f", "c="},
-      {false, "a=b&zq=test&c=d", "q="},
-      {false, "ba=a=b", "a="},
-  };
-  for (const auto& test : test_cases) {
-    EXPECT_EQ(test.expected_result,
-              FromGWSPageLoadMetricsLogger::QueryContainsComponentPrefix(
-                  test.query, test.component))
-        << "For query: " << test.query << " with component: " << test.component;
-  }
-}
-
 TEST_F(FromGWSPageLoadMetricsLoggerTest, Basic) {
   FromGWSPageLoadMetricsLogger logger;
   ASSERT_FALSE(logger.ShouldLogPostCommitMetrics(GURL(kExampleUrl)));
 }
 
 TEST_F(FromGWSPageLoadMetricsLoggerTest, NoPreviousPage) {
   FromGWSPageLoadMetricsLogger logger;
   logger.SetPreviouslyCommittedUrl(GURL());
   logger.set_navigation_initiated_via_link(true);
   ASSERT_FALSE(logger.ShouldLogPostCommitMetrics(GURL(kExampleUrl)));
@@ skipping 52 matching lines @@
       logger.ShouldLogPostCommitMetrics(GURL("https://www.google.com/about/")));
 }
 
 TEST_F(FromGWSPageLoadMetricsLoggerTest, NavigationFromSearchRedirector) {
   FromGWSPageLoadMetricsLogger logger;
   logger.SetPreviouslyCommittedUrl(
       GURL("https://www.google.com/url?source=web"));
   logger.set_navigation_initiated_via_link(true);
   ASSERT_TRUE(logger.ShouldLogPostCommitMetrics(GURL(kExampleUrl)));
 }