Move Google search related util methods to page_load_metrics_util

chrome/browser/page_load_metrics/page_load_metrics_util.h

Index: chrome/browser/page_load_metrics/page_load_metrics_util.h
diff --git a/chrome/browser/page_load_metrics/page_load_metrics_util.h b/chrome/browser/page_load_metrics/page_load_metrics_util.h
index d181d62c492900a2c014f3fa377f07c57d3126d8..09bf42e0161682a19f9023e832719a002bccc934 100644
--- a/chrome/browser/page_load_metrics/page_load_metrics_util.h
+++ b/chrome/browser/page_load_metrics/page_load_metrics_util.h
@@ -134,6 +134,30 @@ 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_