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

Side by Side Diff: chrome/browser/permissions/permission_queue_controller.h

Issue 2184823007: Add a feature which, when enabled, blocks permissions after X prompt dismissals. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Unify implementation in permission_context_base, make log static Created 4 years, 4 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 unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_BROWSER_PERMISSIONS_PERMISSION_QUEUE_CONTROLLER_H_ 5 #ifndef CHROME_BROWSER_PERMISSIONS_PERMISSION_QUEUE_CONTROLLER_H_
6 #define CHROME_BROWSER_PERMISSIONS_PERMISSION_QUEUE_CONTROLLER_H_ 6 #define CHROME_BROWSER_PERMISSIONS_PERMISSION_QUEUE_CONTROLLER_H_
7 7
8 #include "base/bind.h" 8 #include "base/bind.h"
9 #include "base/macros.h" 9 #include "base/macros.h"
10 #include "components/content_settings/core/common/content_settings.h" 10 #include "components/content_settings/core/common/content_settings.h"
(...skipping 50 matching lines...) Expand 10 before | Expand all | Expand 10 after
61 void UpdateContentSetting( 61 void UpdateContentSetting(
62 const GURL& requesting_frame, const GURL& embedder, bool allowed); 62 const GURL& requesting_frame, const GURL& embedder, bool allowed);
63 63
64 protected: 64 protected:
65 // content::NotificationObserver: 65 // content::NotificationObserver:
66 void Observe(int type, 66 void Observe(int type,
67 const content::NotificationSource& source, 67 const content::NotificationSource& source,
68 const content::NotificationDetails& details) override; 68 const content::NotificationDetails& details) override;
69 69
70 private: 70 private:
71 friend class PermissionContextBaseTests;
raymes 2016/08/05 03:27:48 nit: Is this needed? I didn't see anything added t
dominickn 2016/08/05 04:29:30 Done.
71 class PendingInfobarRequest; 72 class PendingInfobarRequest;
72 class RequestEquals; 73 class RequestEquals;
73 74
74 typedef std::vector<PendingInfobarRequest> PendingInfobarRequests; 75 typedef std::vector<PendingInfobarRequest> PendingInfobarRequests;
75 76
76 // Returns true if a geolocation infobar is already visible for the tab 77 // Returns true if a geolocation infobar is already visible for the tab
77 // corresponding to |id|. 78 // corresponding to |id|.
78 bool AlreadyShowingInfoBarForTab(const PermissionRequestID& id) const; 79 bool AlreadyShowingInfoBarForTab(const PermissionRequestID& id) const;
79 80
80 // Shows the next pending infobar for the tab corresponding to |id|, if any. 81 // Shows the next pending infobar for the tab corresponding to |id|, if any.
(...skipping 13 matching lines...) Expand all
94 Profile* const profile_; 95 Profile* const profile_;
95 content::PermissionType permission_type_; 96 content::PermissionType permission_type_;
96 ContentSettingsType content_settings_type_; 97 ContentSettingsType content_settings_type_;
97 PendingInfobarRequests pending_infobar_requests_; 98 PendingInfobarRequests pending_infobar_requests_;
98 bool in_shutdown_; 99 bool in_shutdown_;
99 100
100 DISALLOW_COPY_AND_ASSIGN(PermissionQueueController); 101 DISALLOW_COPY_AND_ASSIGN(PermissionQueueController);
101 }; 102 };
102 103
103 #endif // CHROME_BROWSER_PERMISSIONS_PERMISSION_QUEUE_CONTROLLER_H_ 104 #endif // CHROME_BROWSER_PERMISSIONS_PERMISSION_QUEUE_CONTROLLER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698