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,

[falken, 2015/02/25 01:43:59]

What's the reason we're recording domain and registry vs origin?

[horo, 2015/02/25 02:27:00]

Rappor is designed to use "the eTLD+1 of the user's homepage".
http://www.chromium.org/developers/design-documents/rappor

[falken, 2015/02/25 03:36:57]

I think that's just saying "Settings.HomePage2" uses eTLD+1 but it's not a
design restriction of Rappor itself. Anyway, domain/registry seems sufficient. I
was just curious because SW are per-origin. Maybe Steven Holte can chime in.

[Steven Holte, 2015/02/25 18:30:12]

It's possible to record things other that eTLD+1, but I would generally
recommend that approach.  In order to detect a result, we need to see the same
value reported from a significant proportion of sources.  The more you subdivide
the smaller the percentage of what your report will actually be estimable.

+                                               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_