2010-01-17 Srinidhi Shreedhara <srinidhi.shreedhara@nokia.com>
authorlaszlo.1.gombos@nokia.com <laszlo.1.gombos@nokia.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 18 Jan 2010 05:41:52 +0000 (05:41 +0000)
committerlaszlo.1.gombos@nokia.com <laszlo.1.gombos@nokia.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 18 Jan 2010 05:41:52 +0000 (05:41 +0000)
        Reviewed by Simon Hausmann.

        [Qt] [Symbian] SetWindow call in npapi plugin does not happen when the cooridnates are negative
        https://bugs.webkit.org/show_bug.cgi?id=33573

        * plugins/symbian/PluginViewSymbian.cpp:
        (WebCore::PluginView::setNPWindowIfNeeded): Remove tests for negative
        coordinates for early return.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@53390 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebCore/ChangeLog
WebCore/plugins/symbian/PluginViewSymbian.cpp

index 4e39701..9a3e853 100644 (file)
@@ -1,3 +1,14 @@
+2010-01-17  Srinidhi Shreedhara  <srinidhi.shreedhara@nokia.com>
+
+        Reviewed by Simon Hausmann.
+
+        [Qt] [Symbian] SetWindow call in npapi plugin does not happen when the cooridnates are negative
+        https://bugs.webkit.org/show_bug.cgi?id=33573
+
+        * plugins/symbian/PluginViewSymbian.cpp:
+        (WebCore::PluginView::setNPWindowIfNeeded): Remove tests for negative
+        coordinates for early return.
+
 2010-01-17  Oliver Hunt  <oliver@apple.com>
 
         Reviewed by Dan Bernstein.
index 98d88ed..045314d 100644 (file)
@@ -281,8 +281,6 @@ void PluginView::setNPWindowIfNeeded()
 
     m_npWindow.width = m_windowRect.width();
     m_npWindow.height = m_windowRect.height();
-    if (m_npWindow.x < 0 || m_npWindow.y < 0 || m_npWindow.width <= 0 || m_npWindow.height <= 0)
-        return;
     
     PluginView::setCurrentPluginView(this);
     JSC::JSLock::DropAllLocks dropAllLocks(JSC::SilenceAssertionsOnly);