Make GroupedPermissionInfoBarDelegate's methods call through PermissionPromptAndroid

chrome/browser/permissions/permission_prompt_android.cc

Index: chrome/browser/permissions/permission_prompt_android.cc
diff --git a/chrome/browser/permissions/permission_prompt_android.cc b/chrome/browser/permissions/permission_prompt_android.cc
index 19631a2c5131df989b5444694d33c0aef4714ee3..00736bfae36bbac75d2ec722fc56b8c174eb40dc 100644
--- a/chrome/browser/permissions/permission_prompt_android.cc
+++ b/chrome/browser/permissions/permission_prompt_android.cc
@@ -19,7 +19,11 @@ PermissionPromptAndroid::PermissionPromptAndroid(
   DCHECK(web_contents);
 }
 
-PermissionPromptAndroid::~PermissionPromptAndroid() {}
+PermissionPromptAndroid::~PermissionPromptAndroid() {
+  GroupedPermissionInfoBarDelegate* infobar_delegate =
+      static_cast<GroupedPermissionInfoBarDelegate*>(infobar_->delegate());

[raymes, 2016/11/01 05:29:15]

Could infobar_ be an GroupedPermissionInfoBar? Then you could just call
infobar_->GetDelegate() here? 

[lshang, 2016/11/01 22:58:55]

GetDelegate() is a private method of infobar, so I think static cast
InfobarDelegate is better here.

+  infobar_delegate->PermissionPromptDestroyed();
+}
 
 void PermissionPromptAndroid::SetDelegate(Delegate* delegate) {
   delegate_ = delegate;
@@ -34,7 +38,7 @@ void PermissionPromptAndroid::Show(
     return;
 
   infobar_ = GroupedPermissionInfoBarDelegate::Create(
-      infobar_service, requests[0]->GetOrigin(), requests);
+      this, infobar_service, requests[0]->GetOrigin(), requests);
 }
 
 bool PermissionPromptAndroid::CanAcceptRequestUpdate() {
@@ -69,6 +73,16 @@ void PermissionPromptAndroid::Closing() {
     delegate_->Closing();
 }
 
+void PermissionPromptAndroid::Accept() {
+  if (delegate_)
+    delegate_->Accept();
+}
+
+void PermissionPromptAndroid::Deny() {
+  if (delegate_)
+    delegate_->Deny();
+}
+
 // static
 std::unique_ptr<PermissionPrompt> PermissionPrompt::Create(
     content::WebContents* web_contents) {