Add hooks to permission layer for permission action reporting

chrome/browser/permissions/permission_uma_util.cc

 // Copyright 2014 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.
 
 #include "chrome/browser/permissions/permission_uma_util.h"
 
 #include <utility>
 
+#include "base/command_line.h"
 #include "base/metrics/histogram_macros.h"
 #include "base/strings/stringprintf.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/permissions/permission_manager.h"
 #include "chrome/browser/permissions/permission_util.h"
 #include "chrome/browser/profiles/profile.h"
+#include "chrome/browser/profiles/profile_manager.h"
+#include "chrome/browser/safe_browsing/permission_reporter.h"
+#include "chrome/browser/safe_browsing/ping_manager.h"
+#include "chrome/browser/safe_browsing/safe_browsing_service.h"
+#include "chrome/browser/sync/profile_sync_service_factory.h"
 #include "chrome/browser/ui/website_settings/permission_bubble_request.h"
+#include "chrome/common/chrome_switches.h"
+#include "chrome/common/pref_names.h"
+#include "components/browser_sync/browser/profile_sync_service.h"
+#include "components/prefs/pref_service.h"
 #include "components/rappor/rappor_service.h"
 #include "components/rappor/rappor_utils.h"
 #include "content/public/browser/permission_type.h"
 #include "content/public/common/origin_util.h"
 #include "url/gurl.h"
 
 // UMA keys need to be statically initialized so plain function would not
 // work. Use macros instead.
 #define PERMISSION_ACTION_UMA(secure_origin, permission, permission_secure, \
                               permission_insecure, action)                  \
@@ skipping 41 matching lines @@
   }
 
   std::string permission_str =
       PermissionUtil::GetPermissionString(permission);
   if (permission_str.empty())
     return "";
   return base::StringPrintf("ContentSettings.PermissionActions_%s.%s.Url",
                             permission_str.c_str(), action_str.c_str());
 }
 
+void ReportPermissionAction(const GURL& origin,
+                            PermissionType permission,
+                            PermissionAction action) {
+  // Do not report permission action when the permission action reporting
+  // flag is not enabled.
+  if (!base::CommandLine::ForCurrentProcess()->HasSwitch(
+          switches::kEnablePermissionActionReporting)) {
+    return;
+  }
+
+  Profile* profile = ProfileManager::GetLastUsedProfile();

[stefanocs, 2016/06/15 05:31:15]

Is this the right way to get the profile? Or should the profile object be passed
to this function?

+
+  // Do not report when SafeBrowsing is not enabled.
+  if (!profile->GetPrefs()->GetBoolean(prefs::kSafeBrowsingEnabled)) {
+    return;
+  }
+  // Do not report if profile doesn't have profile sync service.
+  if (!ProfileSyncServiceFactory::HasProfileSyncService(profile)) {
+    return;
+  }
+
+  ProfileSyncService* profile_sync_service =
+      ProfileSyncServiceFactory::GetForProfile(profile);
+  syncer::ModelTypeSet preferred_data_types =
+      profile_sync_service->GetPreferredDataTypes();
+
+  // Do not report if profile can't start sync immediately.
+  if (!profile_sync_service->CanSyncStart()) {
+    return;
+  }
+  // Do not report if Chrome Tab Sync is not enabled.
+  if (!preferred_data_types.Has(syncer::PROXY_TABS)) {
+    return;
+  }
+  // Do not report if Chrome Pref Sync is not enabled.
+  if (!preferred_data_types.Has(syncer::PREFERENCES)) {
+    return;
+  }
+
+  safe_browsing::PermissionReporter* permission_reporter =
+      g_browser_process->safe_browsing_service()
+          ->ping_manager()

[stefanocs, 2016/06/15 05:31:15]

Chrome crashes when ping_manager() is called due to this assertion
(https://cs.chromium.org/chromium/src/chrome/browser/safe_browsing/safe_browsi...).

The crash message is "Check failed:
::content::BrowserThread::CurrentlyOn(BrowserThread::IO). Must be called on
Chrome_IOThread; actually called on CrBrowserMain."
Is there any workaround for this?

[stefanocs, 2016/06/15 07:58:37]

Done.

+          ->permission_reporter();
+  permission_reporter->SendReport(origin, permission, action);
+}
+
 void RecordPermissionAction(PermissionType permission,
                             PermissionAction action,
                             const GURL& requesting_origin) {
+  ReportPermissionAction(requesting_origin, permission, action);
+
   bool secure_origin = content::IsOriginSecure(requesting_origin);
 
   switch (permission) {
     case PermissionType::GEOLOCATION:
         PERMISSION_ACTION_UMA(
             secure_origin,
             "Permissions.Action.Geolocation",
             "Permissions.Action.SecureOrigin.Geolocation",
             "Permissions.Action.InsecureOrigin.Geolocation",
             action);
@@ skipping 277 matching lines @@
 }
 
 void PermissionUmaUtil::PermissionPromptDenied(
     const std::vector<PermissionBubbleRequest*>& requests) {
   DCHECK(!requests.empty());
   DCHECK(requests.size() == 1);
 
   PERMISSION_BUBBLE_TYPE_UMA(kPermissionsPromptDenied,
                              requests[0]->GetPermissionBubbleType());
 }