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

Unified Diff: chrome/browser/permissions/grouped_permission_infobar_delegate_android.h

Issue 2757483002: Move requests from GroupedPermissionInfoBarDelegate to PermissionPromptAndroid (Closed)
Patch Set: Created 3 years, 9 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/permissions/grouped_permission_infobar_delegate_android.h
diff --git a/chrome/browser/permissions/grouped_permission_infobar_delegate_android.h b/chrome/browser/permissions/grouped_permission_infobar_delegate_android.h
index b76ab69582b782be465e7cc55791b3b5fb60d3fe..15839fb9bad178a0d8599121e7aeedbc7477b2da 100644
--- a/chrome/browser/permissions/grouped_permission_infobar_delegate_android.h
+++ b/chrome/browser/permissions/grouped_permission_infobar_delegate_android.h
@@ -14,7 +14,6 @@
class GURL;
class InfoBarService;
class PermissionPromptAndroid;
-class PermissionRequest;
// An InfoBar that displays a group of permission requests, each of which can be
// allowed or blocked independently.
@@ -24,15 +23,13 @@ class GroupedPermissionInfoBarDelegate : public ConfirmInfoBarDelegate {
// Public so we can have std::unique_ptr<GroupedPermissionInfoBarDelegate>.
~GroupedPermissionInfoBarDelegate() override;
- static infobars::InfoBar* Create(
- PermissionPromptAndroid* permission_prompt,
- InfoBarService* infobar_service,
- const GURL& requesting_origin,
- const std::vector<PermissionRequest*>& requests);
+ static infobars::InfoBar* Create(PermissionPromptAndroid* permission_prompt,
+ InfoBarService* infobar_service,
+ const GURL& requesting_origin);
bool persist() const { return persist_; }
void set_persist(bool persist) { persist_ = persist; }
- size_t permission_count() const { return requests_.size(); }
+ size_t PermissionCount() const;
// Returns true if the infobar should display a toggle to allow users to
// opt-out of persisting their accept/deny decision.
@@ -59,10 +56,8 @@ class GroupedPermissionInfoBarDelegate : public ConfirmInfoBarDelegate {
bool GetAcceptState(size_t position);
private:
- GroupedPermissionInfoBarDelegate(
- PermissionPromptAndroid* permission_prompt,
- const GURL& requesting_origin,
- const std::vector<PermissionRequest*>& requests);
+ GroupedPermissionInfoBarDelegate(PermissionPromptAndroid* permission_prompt,
+ const GURL& requesting_origin);
// ConfirmInfoBarDelegate:
InfoBarIdentifier GetIdentifier() const override;
@@ -71,7 +66,6 @@ class GroupedPermissionInfoBarDelegate : public ConfirmInfoBarDelegate {
base::string16 GetButtonLabel(InfoBarButton button) const override;
const GURL requesting_origin_;
- const std::vector<PermissionRequest*> requests_;
// Whether the accept/deny decision is persisted.
bool persist_;
PermissionPromptAndroid* permission_prompt_;

Powered by Google App Engine
This is Rietveld 408576698