Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1819)

Unified Diff: chrome/browser/safe_browsing/permission_reporter.h

Issue 2153763002: Add user gesture field to permission report (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@add-source-ui-to-permission-report
Patch Set: Created 4 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/safe_browsing/permission_reporter.h
diff --git a/chrome/browser/safe_browsing/permission_reporter.h b/chrome/browser/safe_browsing/permission_reporter.h
index 58bd311565251d5cc55820f1886fd4ca01ce3ed2..55695e822d6d372b1a1d7396d7530a236ed82a11 100644
--- a/chrome/browser/safe_browsing/permission_reporter.h
+++ b/chrome/browser/safe_browsing/permission_reporter.h
@@ -31,13 +31,15 @@ class PermissionReporter {
// Sends a serialized permission report to the report collection server.
// The permission report includes |origin| as the origin of
// the site requesting permission, |permission| as the type of permission
- // requested, and |action| as the action taken. The report will be serialized
+ // requested, |action| as the action taken, and request trigger when
+ // |user_gesture| is enabled. The report will be serialized
// using protobuf defined in
// //src/chrome/common/safe_browsing/permission_report.proto
void SendReport(const GURL& origin,
content::PermissionType permission,
PermissionAction action,
- PermissionSourceUI source_ui);
+ PermissionSourceUI source_ui,
+ bool user_gesture);
private:
friend class PermissionReporterTest;
@@ -55,6 +57,7 @@ class PermissionReporter {
content::PermissionType permission,
PermissionAction action,
PermissionSourceUI source_ui,
+ bool user_gesture,
std::string* output);
std::unique_ptr<net::ReportSender> permission_report_sender_;

Powered by Google App Engine
This is Rietveld 408576698