diff options
| author | jam@chromium.org <jam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-08-15 00:04:01 +0000 |
|---|---|---|
| committer | jam@chromium.org <jam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-08-15 00:04:01 +0000 |
| commit | 35fa6a20ef621d22e6ba79e7306abad870cab4e4 (patch) | |
| tree | d976dc15d9ac13976cb444cce875d5cbb790e102 /chrome/renderer/renderer_glue.cc | |
| parent | b7a20d363af744159a2bb85bfe3db027d2d4819e (diff) | |
| download | chromium_src-35fa6a20ef621d22e6ba79e7306abad870cab4e4.zip chromium_src-35fa6a20ef621d22e6ba79e7306abad870cab4e4.tar.gz chromium_src-35fa6a20ef621d22e6ba79e7306abad870cab4e4.tar.bz2 | |
Ensure we don't load plugins on the IO thread.
I had to move the locks from PluginService to PluginList, so that a lock (which can block other threads) isn't held while loading the plugins.
BUG=17938
TEST=added asserts which crash if plugins loaded on IO thread, current UI tests exercise them
Review URL: http://codereview.chromium.org/164305
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23501 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/renderer/renderer_glue.cc')
| -rw-r--r-- | chrome/renderer/renderer_glue.cc | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/chrome/renderer/renderer_glue.cc b/chrome/renderer/renderer_glue.cc index 12ec6fa..affb348 100644 --- a/chrome/renderer/renderer_glue.cc +++ b/chrome/renderer/renderer_glue.cc @@ -202,11 +202,10 @@ std::string GetUIResourceProtocol() { return "chrome"; } -bool GetPlugins(bool refresh, std::vector<WebPluginInfo>* plugins) { +void GetPlugins(bool refresh, std::vector<WebPluginInfo>* plugins) { if (!RenderThread::current()->plugin_refresh_allowed()) refresh = false; - return RenderThread::current()->Send(new ViewHostMsg_GetPlugins( - refresh, plugins)); + RenderThread::current()->Send(new ViewHostMsg_GetPlugins(refresh, plugins)); } // static factory function |