Add hooks to permission layer for permission action reporting

chrome/browser/safe_browsing/ping_manager.h

Index: chrome/browser/safe_browsing/ping_manager.h
diff --git a/chrome/browser/safe_browsing/ping_manager.h b/chrome/browser/safe_browsing/ping_manager.h
index f8730ba19357cd672b8a3a9272280e87f802b7f9..76cbf7e14fc899e348355d2655bdc2244f99aa3c 100644
--- a/chrome/browser/safe_browsing/ping_manager.h
+++ b/chrome/browser/safe_browsing/ping_manager.h
@@ -31,6 +31,8 @@ class URLRequestContextGetter;
 
 namespace safe_browsing {
 
+class PermissionReporter;
+
 class SafeBrowsingPingManager : public net::URLFetcherDelegate {
  public:
   ~SafeBrowsingPingManager() override;
@@ -60,6 +62,8 @@ class SafeBrowsingPingManager : public net::URLFetcherDelegate {
       std::unique_ptr<certificate_reporting::ErrorReporter>
           certificate_error_reporter);
 
+  PermissionReporter* permission_reporter();

[raymes, 2016/06/14 02:47:11]

Do you think we should have a Send function here, rather than exposing the
PermissionReporter? I don't feel strongly except that it would be more aligned
with the other functions in this class. In that case the flag check could live
in this class too.

[stefanocs, 2016/06/14 03:37:37]

Should we remove the ReportPermissionAction function from PermissionUmaUtil and
let the Send function be called by the RecordPermissionAction then?

[raymes, 2016/06/14 03:58:16]

That seems good since the function will be so small

+
  private:
   FRIEND_TEST_ALL_PREFIXES(SafeBrowsingPingManagerTest,
                            TestSafeBrowsingHitUrl);
@@ -100,6 +104,9 @@ class SafeBrowsingPingManager : public net::URLFetcherDelegate {
   std::unique_ptr<certificate_reporting::ErrorReporter>
       certificate_error_reporter_;
 
+  // Sends reports of permission actions.
+  std::unique_ptr<PermissionReporter> permission_reporter_;
+
   DISALLOW_COPY_AND_ASSIGN(SafeBrowsingPingManager);
 };