Move Google search related util methods to page_load_metrics_util

chrome/browser/page_load_metrics/page_load_metrics_util.h

 // 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.
 
 #ifndef CHROME_BROWSER_PAGE_LOAD_METRICS_PAGE_LOAD_METRICS_UTIL_H_
 #define CHROME_BROWSER_PAGE_LOAD_METRICS_PAGE_LOAD_METRICS_UTIL_H_
 
 #include "base/metrics/histogram_macros.h"
 #include "base/optional.h"
 #include "base/time/time.h"
@@ skipping 116 matching lines @@
 base::Optional<base::TimeDelta> GetInitialForegroundDuration(
     const PageLoadExtraInfo& info,
     base::TimeTicks app_background_time);
 
 // Whether the given loading behavior was observed in any frame (either the main
 // frame or a subframe).
 bool DidObserveLoadingBehaviorInAnyFrame(
     const page_load_metrics::PageLoadExtraInfo& info,
     blink::WebLoadingBehaviorFlag behavior);
 
+// Whether the given url has a google serch hostname.

[falken, 2017/06/12 07:19:43]

// Google Search

Consdier adding some examples:
// https://www.google.com -> true
// https://www.google.co.jp -> true
// https://www.google.mysite.com -> false
// https://docs.google.com -> false

[horo, 2017/06/12 15:10:37]

Done.

+bool IsGoogleSearchHostname(const GURL& url);
+
+// Whether the given url has a google serch result url.

[falken, 2017/06/12 07:19:43]

"whether the URL has a XXX URL" doesn't really make sense.

Maybe "is for a Google Search results page."

Examples:
// https://www.google.co.jp/ -> true
// https://www.google.com/search -> true
// https://www.google.com/maps -> false

This is pretty complex so consider adding the Google docs link from the .cc
file.

[horo, 2017/06/12 15:10:37]

Done.

+bool IsGoogleSearchResultUrl(const GURL& url);
+
+// Whether the given url has a google serch's redirector url.

[falken, 2017/06/12 07:19:43]

"is a Google Search redirector URL" ?

[horo, 2017/06/12 15:10:37]

Done.

+bool IsGoogleSearchRedirectorUrl(const GURL& url);
+
+// Whether the given query string contains the given component. The query
+// parameter should contain the query string of a URL (the portion following
+// the question mark, excluding the question mark). The component must fully
+// match a component in the query string. For example, 'foo=bar' would match
+// the query string 'a=b&foo=bar&c=d' but would not match 'a=b&zzzfoo=bar&c=d'
+// since, though foo=bar appears in the query string, the key specified in the
+// component 'foo' does not match the full key in the query string
+// 'zzzfoo'. For QueryContainsComponent, the component should of the form
+// 'key=value'. For QueryContainsComponentPrefix, the component should be of
+// the form 'key=' (where the value is not specified).
+bool QueryContainsComponent(const base::StringPiece query,
+                            const base::StringPiece component);
+bool QueryContainsComponentPrefix(const base::StringPiece query,
+                                  const base::StringPiece component);
+
 }  // namespace page_load_metrics
 
 #endif  // CHROME_BROWSER_PAGE_LOAD_METRICS_PAGE_LOAD_METRICS_UTIL_H_