This fixes the rendering issue in Firefox by ignoring zero height

resize requests from Firefox.

When the fullscreen stuff was redone (r26489), we removed a
WM_SIZE handler that would call the plugin's "Resize" API, and
moved it into NPP_SetWindow, but at the same time a SetWindowPos
call was also added that would take the window size as gospel and
set it, but only if we're not in fullscreen mode.  Turns out that
this is not necessary because the Resize API already checks to
see if we're in full-screen mode, and handles the spurious zero
height messages properly too.  So I just took out the explicit
SetWindowPos.
Review URL: http://codereview.chromium.org/268022

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@28482 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 0 insertions, 5 deletions

diff --git a/o3d/plugin/win/main_win.cc b/o3d/plugin/win/main_win.cc
index f8c6d20..7d6ec75 100644
--- a/o3d/plugin/win/main_win.cc
+++ b/o3d/plugin/win/main_win.cc
@@ -836,11 +836,6 @@ NPError NPP_SetWindow(NPP instance, NPWindow *window) {
     // Avoid spurious resize requests.
     if (window->width != obj->width() ||
         window->height != obj->height()) {
-      if (!obj->fullscreen()) {
-        ::SetWindowPos(obj->GetContentHWnd(), obj->GetPluginHWnd(), 0, 0,
-                       window->width, window->height,
-                       SWP_NOZORDER | SWP_NOREPOSITION);
-      }
       // Even if we are in full-screen mode, store off the new width
       // and height to restore to them later.
       obj->Resize(window->width, window->height);