Fix for 165634

chrome/browser/extensions/api/webstore_private/webstore_private_api.cc

Index: chrome/browser/extensions/api/webstore_private/webstore_private_api.cc
diff --git a/chrome/browser/extensions/api/webstore_private/webstore_private_api.cc b/chrome/browser/extensions/api/webstore_private/webstore_private_api.cc
index 1a20ed9c89f93c5f018a9f30041cb152afe7487f..a5f3225d972810d64d2a41c8b90c6cc37165f740 100644
--- a/chrome/browser/extensions/api/webstore_private/webstore_private_api.cc
+++ b/chrome/browser/extensions/api/webstore_private/webstore_private_api.cc
@@ -569,13 +569,13 @@ bool WebstorePrivateCompleteInstallFunction::RunImpl() {
 
   // The extension will install through the normal extension install flow, but
   // the whitelist entry will bypass the normal permissions install dialog.
-  scoped_refptr<WebstoreInstaller> installer = new WebstoreInstaller(
-      GetProfile(),
-      this,
-      &(dispatcher()->delegate()->GetAssociatedWebContents()->GetController()),
-      params->expected_id,
-      approval_.Pass(),
-      WebstoreInstaller::INSTALL_SOURCE_OTHER);
+  scoped_refptr<WebstoreInstaller> installer =
+      new WebstoreInstaller(GetProfile(),
+                            this,
+                            GetAssociatedWebContents(),

[tmdiep, 2014/02/11 23:27:34]

Thanks for investigating this issue and submitting the patch! I submitted
essentially the same patch yesterday and it has just been lgtm'ed by an owner:
https://codereview.chromium.org/138803012

Before I commit, just wanted to check with you - where our patches differ is
here. Is calling GetAssociatedWebContents() essential to the fix? Or can
dispatcher()->delegate()->GetAssociatedWebContents() be used?

+                            params->expected_id,
+                            approval_.Pass(),
+                            WebstoreInstaller::INSTALL_SOURCE_OTHER);
   installer->Start();
 
   return true;