Minor clean-ups to GroupedPermissionInfoBarDelegate

- Remove unused requesting_origin_
- Make destructor private
- Remove unused GetAcceptState() declaration

Change-Id: Ie8a8984d2a6975568cd8b26e57c0844c88acb594
Reviewed-on: https://chromium-review.googlesource.com/575737Reviewed-by: Raymes Khoury <raymes@chromium.org>
Commit-Queue: Timothy Loh <timloh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488120}

chrome/browser/permissions/grouped_permission_infobar_delegate_android.cc

@@ -22,11 +22,10 @@ GroupedPermissionInfoBarDelegate::~GroupedPermissionInfoBarDelegate() {}
 // static
 infobars::InfoBar* GroupedPermissionInfoBarDelegate::Create(
     const base::WeakPtr<PermissionPromptAndroid>& permission_prompt,
-    InfoBarService* infobar_service,
-    const GURL& requesting_origin) {
-  return infobar_service->AddInfoBar(base::MakeUnique<GroupedPermissionInfoBar>(
-      base::WrapUnique(new GroupedPermissionInfoBarDelegate(
-          permission_prompt, requesting_origin))));
+    InfoBarService* infobar_service) {
+  return infobar_service->AddInfoBar(
+      base::MakeUnique<GroupedPermissionInfoBar>(base::WrapUnique(
+          new GroupedPermissionInfoBarDelegate(permission_prompt))));
 }
 
 size_t GroupedPermissionInfoBarDelegate::PermissionCount() const {
@@ -80,11 +79,8 @@ base::string16 GroupedPermissionInfoBarDelegate::GetLinkText() const {
 }
 
 GroupedPermissionInfoBarDelegate::GroupedPermissionInfoBarDelegate(
-    const base::WeakPtr<PermissionPromptAndroid>& permission_prompt,
-    const GURL& requesting_origin)
-    : requesting_origin_(requesting_origin),
-      persist_(true),
-      permission_prompt_(permission_prompt) {
+    const base::WeakPtr<PermissionPromptAndroid>& permission_prompt)
+    : persist_(true), permission_prompt_(permission_prompt) {
   DCHECK(permission_prompt);
 }
 

chrome/browser/permissions/grouped_permission_infobar_delegate_android.h

@@ -26,8 +26,7 @@ class GroupedPermissionInfoBarDelegate : public ConfirmInfoBarDelegate {
 
   static infobars::InfoBar* Create(
       const base::WeakPtr<PermissionPromptAndroid>& permission_prompt,
-      InfoBarService* infobar_service,
-      const GURL& requesting_origin);
+      InfoBarService* infobar_service);
 
   bool persist() const { return persist_; }
   void set_persist(bool persist) { persist_ = persist; }
@@ -49,13 +48,9 @@ class GroupedPermissionInfoBarDelegate : public ConfirmInfoBarDelegate {
   void InfoBarDismissed() override;
   base::string16 GetLinkText() const override;
 
- protected:
-  bool GetAcceptState(size_t position);
-
  private:
   GroupedPermissionInfoBarDelegate(
-      const base::WeakPtr<PermissionPromptAndroid>& permission_prompt,
-      const GURL& requesting_origin);
+      const base::WeakPtr<PermissionPromptAndroid>& permission_prompt);
 
   // ConfirmInfoBarDelegate:
   InfoBarIdentifier GetIdentifier() const override;
@@ -67,7 +62,6 @@ class GroupedPermissionInfoBarDelegate : public ConfirmInfoBarDelegate {
   // InfoBarDelegate:
   bool EqualsDelegate(infobars::InfoBarDelegate* delegate) const override;
 
-  const GURL requesting_origin_;
   // Whether the accept/deny decision is persisted.
   bool persist_;
   base::WeakPtr<PermissionPromptAndroid> permission_prompt_;

chrome/browser/permissions/permission_prompt_android.cc

@@ -41,9 +41,8 @@ PermissionPromptAndroid::PermissionPromptAndroid(
   if (!infobar_service)
     return;
 
-  GroupedPermissionInfoBarDelegate::Create(
-      weak_factory_.GetWeakPtr(), infobar_service,
-      delegate_->Requests()[0]->GetOrigin());
+  GroupedPermissionInfoBarDelegate::Create(weak_factory_.GetWeakPtr(),
+                                           infobar_service);
 }
 
 PermissionPromptAndroid::~PermissionPromptAndroid() {}