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

Unified Diff: chrome/browser/download/download_permission_request.h

Issue 1884073002: Add a new metric for how many times permissions prompts are shown. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Feedback Created 4 years, 8 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/download/download_permission_request.h
diff --git a/chrome/browser/download/download_permission_request.h b/chrome/browser/download/download_permission_request.h
index 979711af90aaab5dbcac5dba3f4f54d6f628a0ea..fa4e0d99796362e14d701e8a2ff44026abf7cbb3 100644
--- a/chrome/browser/download/download_permission_request.h
+++ b/chrome/browser/download/download_permission_request.h
@@ -20,6 +20,7 @@ class DownloadPermissionRequest : public PermissionBubbleRequest {
base::WeakPtr<DownloadRequestLimiter::TabDownloadState> host);
~DownloadPermissionRequest() override;
+ private:
// PermisisonBubbleDelegate:
int GetIconId() const override;
base::string16 GetMessageText() const override;
@@ -29,8 +30,8 @@ class DownloadPermissionRequest : public PermissionBubbleRequest {
void PermissionDenied() override;
void Cancelled() override;
void RequestFinished() override;
+ PermissionBubbleType GetPermissionBubbleType() const override;
- private:
base::WeakPtr<DownloadRequestLimiter::TabDownloadState> host_;
GURL request_origin_;

Powered by Google App Engine
This is Rietveld 408576698