Make dismiss works for 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 c4be92e2b0a114322d92db0e5625a4a712e1392a..ba0b531d7b7272eaacc222f16b19b0a0a0dd898a 100644
--- a/chrome/browser/permissions/permission_prompt_android.cc
+++ b/chrome/browser/permissions/permission_prompt_android.cc
@@ -70,7 +70,6 @@ gfx::NativeWindow PermissionPromptAndroid::GetNativeWindow() {
 }
 
 void PermissionPromptAndroid::Closing() {
-  infobar_ = nullptr;

[raymes, 2016/11/30 02:21:48]

Hmm, how come this was removed?

[lshang, 2016/11/30 04:59:00]

So the infobar_ should only be set to null after
infobar_service->RemoveInfoBar(infobar_) is called, like in the Hide() method
above. Otherwise it won't remove the infobar for infobar service and pending
requests won't get displayed afterwards(PermissionRequestManager::Closing() will
call PermissionPromptAndroid::Hide() and line 54 will be skipped).

[raymes, 2016/11/30 05:07:12]

Ah that makes sense, thanks for explaining!

   if (delegate_)
     delegate_->Closing();
 }