Commit 42bbd7c9 authored by jyasskin@chromium.org's avatar jyasskin@chromium.org

In ~ExtensionHost, close swapped-out RenderViews referring to this extension.

I don't think this is the ideal fix, but it works, and I don't think I
can get the real fix debugged by the M27 branch.

BUG=178542
TEST=Use the instructions at http://crbug.com/178542#c16

Review URL: https://codereview.chromium.org/13067002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@190543 0039d316-1c4b-4281-b951-d872f2087c98
parent 68e57082
...@@ -163,6 +163,15 @@ ExtensionHost::~ExtensionHost() { ...@@ -163,6 +163,15 @@ ExtensionHost::~ExtensionHost() {
UMA_HISTOGRAM_LONG_TIMES("Extensions.EventPageActiveTime", UMA_HISTOGRAM_LONG_TIMES("Extensions.EventPageActiveTime",
since_created_.Elapsed()); since_created_.Elapsed());
} }
// Clear the extension's renderer process out from any WebContents
// that might have it in their back history.
// TODO(jyasskin): Remove this hack by making extension reloading
// wait for renderer shutdown.
content::NotificationService::current()->Notify(
content::NOTIFICATION_RENDERER_PROCESS_CLOSING,
content::Source<content::RenderProcessHost>(render_process_host()),
content::NotificationService::NoDetails());
content::NotificationService::current()->Notify( content::NotificationService::current()->Notify(
chrome::NOTIFICATION_EXTENSION_HOST_DESTROYED, chrome::NOTIFICATION_EXTENSION_HOST_DESTROYED,
content::Source<Profile>(profile_), content::Source<Profile>(profile_),
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment