diff options
| author | jrg@chromium.org <jrg@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-01-27 01:26:16 +0000 |
|---|---|---|
| committer | jrg@chromium.org <jrg@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-01-27 01:26:16 +0000 |
| commit | cd636d8fb36d118a995f913920031b1f6c33fdec (patch) | |
| tree | ea455370d2fb0a318ae88f6e8090cc36ad53fe67 /webkit/default_plugin/plugin_impl.h | |
| parent | dac395fecf2ce32c37737143f3b8eb71d3ce38b3 (diff) | |
| download | chromium_src-cd636d8fb36d118a995f913920031b1f6c33fdec.zip chromium_src-cd636d8fb36d118a995f913920031b1f6c33fdec.tar.gz chromium_src-cd636d8fb36d118a995f913920031b1f6c33fdec.tar.bz2 | |
Moved Init() before startup_data_->event.Signal() because derived
classes may not be safe to use until Init() has been called. As an
example, RenderThread() creates it's IPC::SyncChannel in Init(), so it
isn't safe to call Send() method until after.
Update use of a Thread (hit with ui_tests) to no longer require Init()
be called after the startup event is signaled (required old thread
behavior).
Ran a bunch more tests on Windows to try harder to make sure I didn't break anything.
Review URL: http://codereview.chromium.org/18824
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@8695 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'webkit/default_plugin/plugin_impl.h')
| -rw-r--r-- | webkit/default_plugin/plugin_impl.h | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/webkit/default_plugin/plugin_impl.h b/webkit/default_plugin/plugin_impl.h index 8da77cb..1e94a3a 100644 --- a/webkit/default_plugin/plugin_impl.h +++ b/webkit/default_plugin/plugin_impl.h @@ -350,7 +350,8 @@ class PluginInstallerImpl : public CWindowImpl<PluginInstallerImpl> { // which enables the downloaded plugin to be instantiated. // The completion events from the job are monitored in an independent // thread. - PluginInstallationJobMonitorThread installation_job_monitor_thread_; + scoped_refptr<PluginInstallationJobMonitorThread> + installation_job_monitor_thread_; // This object handles download and parsing of the plugins database. PluginDatabaseHandler plugin_database_handler_; // Indicates if the left click to download/refresh should be enabled or not. |