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

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

Issue 2444253002: Change accept state of PermissionRequestManager from grouped infobar (Closed)
Patch Set: remove unused stuff Created 4 years, 1 month 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.cc
diff --git a/chrome/browser/permissions/grouped_permission_infobar_delegate_android.cc b/chrome/browser/permissions/grouped_permission_infobar_delegate_android.cc
index c0fd8aa326460acd3dc5156f5fbb78d0ad6b611c..74d5a3c9da2e9307bbabdad06b28aadf9422e2d3 100644
--- a/chrome/browser/permissions/grouped_permission_infobar_delegate_android.cc
+++ b/chrome/browser/permissions/grouped_permission_infobar_delegate_android.cc
@@ -56,7 +56,8 @@ base::string16 GroupedPermissionInfoBarDelegate::GetMessageTextFragment(
void GroupedPermissionInfoBarDelegate::ToggleAccept(size_t position,
bool new_value) {
DCHECK_LT(position, requests_.size());
- accept_states_[position] = new_value;
+ if (permission_prompt_)
+ permission_prompt_->ToggleAccept(position, new_value);
}
base::string16 GroupedPermissionInfoBarDelegate::GetMessageText() const {
@@ -81,18 +82,12 @@ void GroupedPermissionInfoBarDelegate::PermissionPromptDestroyed() {
permission_prompt_ = nullptr;
}
-bool GroupedPermissionInfoBarDelegate::GetAcceptState(size_t position) {
- DCHECK_LT(position, requests_.size());
- return accept_states_[position];
-}
-
GroupedPermissionInfoBarDelegate::GroupedPermissionInfoBarDelegate(
PermissionPromptAndroid* permission_prompt,
const GURL& requesting_origin,
const std::vector<PermissionRequest*>& requests)
: requesting_origin_(requesting_origin),
requests_(requests),
- accept_states_(requests_.size(), true),
persist_(true),
permission_prompt_(permission_prompt) {
DCHECK(permission_prompt);

Powered by Google App Engine
This is Rietveld 408576698