diff options
| author | jam@chromium.org <jam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-04-21 21:48:36 +0000 |
|---|---|---|
| committer | jam@chromium.org <jam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-04-21 21:48:36 +0000 |
| commit | 875d489e8aadd251c638c1bad6bb7ca967605f98 (patch) | |
| tree | e1be54c561d33b240f6c88b042c9b2df732ac5c6 /chrome/browser/plugin_process_host.h | |
| parent | e3db4fabfc7aabca3cd1998e05d2a769fbd0c5b6 (diff) | |
| download | chromium_src-875d489e8aadd251c638c1bad6bb7ca967605f98.zip chromium_src-875d489e8aadd251c638c1bad6bb7ca967605f98.tar.gz chromium_src-875d489e8aadd251c638c1bad6bb7ca967605f98.tar.bz2 | |
Fix hang seen in plugin process because plugin creation ended up having to wait on UI thread. Instead of using sync messages, the plugin hwnd is initially parented to the RenderWidgetHost's HWND. It's then lazily reparented to an intermediate HWND on the UI thread when it comes time to move it.
BUG=10711
TEST=added regression tests, but testers please confirm plugins on top video sites are placed correctly.
Review URL: http://codereview.chromium.org/67285
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@14137 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/plugin_process_host.h')
| -rw-r--r-- | chrome/browser/plugin_process_host.h | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/chrome/browser/plugin_process_host.h b/chrome/browser/plugin_process_host.h index c43fef2..a326bee 100644 --- a/chrome/browser/plugin_process_host.h +++ b/chrome/browser/plugin_process_host.h @@ -104,8 +104,7 @@ class PluginProcessHost : public ChildProcessHost, void OnPluginMessage(const std::vector<uint8>& data); #if defined(OS_WIN) - void OnCreateWindow(HWND parent, IPC::Message* reply_msg); - void OnDestroyWindow(HWND window); + void OnPluginWindowDestroyed(HWND window, HWND parent); void OnDownloadUrl(const std::string& url, int source_pid, gfx::NativeWindow caller_window); #endif |