diff options
| author | stuartmorgan@chromium.org <stuartmorgan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-11-12 22:50:47 +0000 |
|---|---|---|
| committer | stuartmorgan@chromium.org <stuartmorgan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-11-12 22:50:47 +0000 |
| commit | 8f4752537eeff04982fddbbf4bd6bf92fd1ab9e1 (patch) | |
| tree | ab9aaabc1e200a9c073d8fda89d55a1e7ca36c1f /webkit/glue | |
| parent | 4ed2e36260b3ef4cd004b4787f62d54aa3d47d28 (diff) | |
| download | chromium_src-8f4752537eeff04982fddbbf4bd6bf92fd1ab9e1.zip chromium_src-8f4752537eeff04982fddbbf4bd6bf92fd1ab9e1.tar.gz chromium_src-8f4752537eeff04982fddbbf4bd6bf92fd1ab9e1.tar.bz2 | |
Always call NPP_SetWindow, even for 0x0 plugins, on the Mac
QuickTime can be embedded for audio, with a 0x0 plugin, but it still expects to get information about the window via SetWindow. Windows in fact made this same fix a while ago, for the same reason, but in the Windowed version of this method.
BUG=25883
TEST=Open a site like the one in the bug with a 0x0 QuickTime plugin; the plugin should not crash.
Review URL: http://codereview.chromium.org/389026
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@31845 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'webkit/glue')
| -rw-r--r-- | webkit/glue/plugins/webplugin_delegate_impl_mac.mm | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/webkit/glue/plugins/webplugin_delegate_impl_mac.mm b/webkit/glue/plugins/webplugin_delegate_impl_mac.mm index 1042cae..737954a 100644 --- a/webkit/glue/plugins/webplugin_delegate_impl_mac.mm +++ b/webkit/glue/plugins/webplugin_delegate_impl_mac.mm @@ -376,9 +376,6 @@ void WebPluginDelegateImpl::WindowlessSetWindow(bool force_set_window) { if (!instance()) return; - if (window_rect_.IsEmpty()) // wait for geometry to be set. - return; - window_.clipRect.top = 0; window_.clipRect.left = 0; window_.clipRect.bottom = window_rect_.height(); |