Comment #3 on issue 15421 by [email protected]: Don't create  
GtkSockets browser-side for windowless plugins
http://code.google.com/p/chromium/issues/detail?id=15421

The following revision refers to this bug:
     http://src.chromium.org/viewvc/chrome?view=rev&revision=19602

------------------------------------------------------------------------
r19602 | [email protected] | 2009-06-30 10:14:59 -0700 (Tue, 30 Jun 2009) |  
19 lines
Changed paths:
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/plugin/webplugin_proxy.h?r1=19602&r2=19601
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/plugins/webplugin_delegate_impl_gtk.cc?r1=19602&r2=19601
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/webplugin.h?r1=19602&r2=19601
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/webplugin_impl.cc?r1=19602&r2=19601
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/webplugin_impl.h?r1=19602&r2=19601
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/webview_delegate.h?r1=19602&r2=19601
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/tools/test_shell/test_webview_delegate.h?r1=19602&r2=19601
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/tools/test_shell/test_webview_delegate_gtk.cc?r1=19602&r2=19601

linux: only create browser-side plugin container after plugin requests it

On Windows, windowed plugins are parented in a two-stage process: initially,
the plugin is parented to the render view, then later it's reparented to
a special per-plugin window.

On Linux, plugin embedding always needs a special per-plugin window before
it can be initialized.  So there's no way to do the two-stage initialization
process, and we only want to construct the browser-side plugin container for
windowed plugins after the plugin requests it.

Plumbing this through the WebPlugin interface will also allow us to  
side-route
this request for window creation out to the browser process in the
multi-process case.

BUG=15421
TEST=plugins still work in test_shell

Review URL: http://codereview.chromium.org/150034
------------------------------------------------------------------------


--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~---------~--~----~------------~-------~--~----~
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
-~----------~----~----~----~------~----~------~--~---

Reply via email to