Add RAPPOR metrics of ServiceWorker.ControlledPage

content/public/browser/content_browser_client.h

 // 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.
 
 #ifndef CONTENT_PUBLIC_BROWSER_CONTENT_BROWSER_CLIENT_H_
 #define CONTENT_PUBLIC_BROWSER_CONTENT_BROWSER_CLIENT_H_
 
 #include <map>
 #include <string>
 #include <utility>
@@ skipping 608 matching lines @@
   virtual PresentationServiceDelegate* GetPresentationServiceDelegate(
       WebContents* web_contents);
 
   // Allows programmatic opening of a new tab/window without going through
   // another WebContents. For example, from a Worker. |callback| will be
   // invoked with the appropriate WebContents* when available.
   virtual void OpenURL(BrowserContext* browser_context,
                        const OpenURLParams& params,
                        const base::Callback<void(WebContents*)>& callback);
 
+  // Records the domain and registry of a url to a Rappor metric.
+  virtual void SampleDomainAndRegistryFromGURL(const std::string& metric,

[nasko, 2015/02/26 01:03:20]

This client interface should be towards a generic embedder, not specifically
chrome. As such, I find the name to be overly specific to the implementation
that chrome/ provides. Let's rename this to a bit more generic method and
comment it properly.

[horo, 2015/02/26 07:07:45]

Done.

+                                               const GURL& gurl) {}
+
 #if defined(OS_POSIX) && !defined(OS_MACOSX)
   // Populates |mappings| with all files that need to be mapped before launching
   // a child process.
   virtual void GetAdditionalMappedFilesForChildProcess(
       const base::CommandLine& command_line,
       int child_process_id,
       FileDescriptorInfo* mappings) {}
 #endif
 
 #if defined(OS_WIN)
@@ skipping 11 matching lines @@
   // Allows an embedder to provide its own ExternalVideoSurfaceContainer
   // implementation.  Return nullptr to disable external surface video.
   virtual ExternalVideoSurfaceContainer*
   OverrideCreateExternalVideoSurfaceContainer(WebContents* web_contents);
 #endif
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_CONTENT_BROWSER_CLIENT_H_