Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(979)

Unified Diff: chrome/browser/google/google_util.h

Issue 333723002: Componentize google_util (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase + build fix Created 6 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/google/google_url_tracker_unittest.cc ('k') | chrome/browser/google/google_util.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/google/google_util.h
diff --git a/chrome/browser/google/google_util.h b/chrome/browser/google/google_util.h
deleted file mode 100644
index 8037ebde07a8f4e5294a0e7667b84465ad686b7a..0000000000000000000000000000000000000000
--- a/chrome/browser/google/google_util.h
+++ /dev/null
@@ -1,114 +0,0 @@
-// 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.
-//
-// Some Google related utility functions.
-
-#ifndef CHROME_BROWSER_GOOGLE_GOOGLE_UTIL_H__
-#define CHROME_BROWSER_GOOGLE_GOOGLE_UTIL_H__
-
-#include <string>
-
-#include "base/basictypes.h"
-
-class GURL;
-
-// This namespace provides various helpers around handling Google-related URLs.
-namespace google_util {
-
-// True iff |str| contains a "q=" query parameter with a non-empty value.
-// |str| should be a query or a hash fragment, without the ? or # (as
-// returned by GURL::query() or GURL::ref().
-bool HasGoogleSearchQueryParam(const std::string& str);
-
-// The query key that identifies a Google Extended API request for Instant.
-const char kInstantExtendedAPIParam[] = "espv";
-
-GURL LinkDoctorBaseURL();
-void SetMockLinkDoctorBaseURLForTesting();
-
-// Returns the Google locale corresponding to |application_locale|. This is
-// the same string as AppendGoogleLocaleParam adds to the URL, only without the
-// leading "hl".
-std::string GetGoogleLocale(const std::string& application_locale);
-
-// Adds the Google locale string to the URL (e.g., hl=en-US). This does not
-// check to see if the param already exists.
-GURL AppendGoogleLocaleParam(const GURL& url,
- const std::string& application_locale);
-
-// Returns the Google country code string for the given Google homepage URL.
-std::string GetGoogleCountryCode(GURL google_homepage_url);
-
-// Returns the Google search URL for the given Google homepage URL.
-GURL GetGoogleSearchURL(GURL google_homepage_url);
-
-// Returns the Google base URL specified on the command line, if it exists.
-// This performs some fixup and sanity-checking to ensure that the resulting URL
-// is valid and has no query or ref.
-GURL CommandLineGoogleBaseURL();
-
-// Returns true if a Google base URL was specified on the command line and |url|
-// begins with that base URL. This uses a simple string equality check.
-bool StartsWithCommandLineGoogleBaseURL(const GURL& url);
-
-// WARNING: The following IsGoogleXXX() functions use heuristics to rule out
-// "obviously false" answers. They do NOT guarantee that the string in question
-// is actually on a Google-owned domain, just that it looks plausible. If you
-// need to restrict some behavior to only happen on Google's officially-owned
-// domains, use TransportSecurityState::IsGooglePinnedProperty() instead.
-
-// Designate whether or not a URL checking function also checks for specific
-// subdomains, or only "www" and empty subdomains.
-enum SubdomainPermission {
- ALLOW_SUBDOMAIN,
- DISALLOW_SUBDOMAIN,
-};
-
-// Designate whether or not a URL checking function also checks for standard
-// ports (80 for http, 443 for https), or if it allows all port numbers.
-enum PortPermission {
- ALLOW_NON_STANDARD_PORTS,
- DISALLOW_NON_STANDARD_PORTS,
-};
-
-// True if |host| is "[www.]google.<TLD>" with a valid TLD. If
-// |subdomain_permission| is ALLOW_SUBDOMAIN, we check against host
-// "*.google.<TLD>" instead.
-//
-// If the Google base URL has been overridden on the command line, this function
-// will also return true for any URL whose hostname exactly matches the hostname
-// of the URL specified on the command line. In this case,
-// |subdomain_permission| is ignored.
-bool IsGoogleHostname(const std::string& host,
- SubdomainPermission subdomain_permission);
-
-// True if |url| is a valid URL with a host that returns true for
-// IsGoogleHostname(), and an HTTP or HTTPS scheme. If |port_permission| is
-// DISALLOW_NON_STANDARD_PORTS, this also requires |url| to use the standard
-// port for its scheme (80 for HTTP, 443 for HTTPS).
-//
-// Note that this only checks for google.<TLD> domains, but not other Google
-// properties. There is code in variations_http_header_provider.cc that checks
-// for additional Google properties, which can be moved here if more callers
-// are interested in this in the future.
-bool IsGoogleDomainUrl(const GURL& url,
- SubdomainPermission subdomain_permission,
- PortPermission port_permission);
-
-// True if |url| represents a valid Google home page URL.
-bool IsGoogleHomePageUrl(const GURL& url);
-
-// True if |url| represents a valid Google search URL.
-bool IsGoogleSearchUrl(const GURL& url);
-
-// True if |url| is a valid youtube.<TLD> URL. If |port_permission| is
-// DISALLOW_NON_STANDARD_PORTS, this also requires |url| to use the standard
-// port for its scheme (80 for HTTP, 443 for HTTPS).
-bool IsYoutubeDomainUrl(const GURL& url,
- SubdomainPermission subdomain_permission,
- PortPermission port_permission);
-
-} // namespace google_util
-
-#endif // CHROME_BROWSER_GOOGLE_GOOGLE_UTIL_H__
« no previous file with comments | « chrome/browser/google/google_url_tracker_unittest.cc ('k') | chrome/browser/google/google_util.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698