Add RAPPOR metrics of ServiceWorker.ControlledPage

content/public/browser/content_browser_client.h

Index: content/public/browser/content_browser_client.h
diff --git a/content/public/browser/content_browser_client.h b/content/public/browser/content_browser_client.h
index fbe8214fd2462901e9ebf31f5630974233bfb58a..f0c7eaf32924a70429050e77a2a2ad35af3f1aff 100644
--- a/content/public/browser/content_browser_client.h
+++ b/content/public/browser/content_browser_client.h
@@ -626,6 +626,10 @@ class CONTENT_EXPORT ContentBrowserClient {
                        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.