2011-04-08 Misha Tyutyunik <michael.tyutyunik@nokia.com>
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 8 Apr 2011 17:58:45 +0000 (17:58 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 8 Apr 2011 17:58:45 +0000 (17:58 +0000)
        Reviewed by Laszlo Gombos.

        [Qt] Plugin is not scrolled together with page content or jumping
        when content is rendered using cache (backing store).
        https://bugs.webkit.org/show_bug.cgi?id=56130

        This is a Symbian portion of the fix. For Linux it looks like
        it's enough to call set geometry which should be addressed in
        https://bugs.webkit.org/show_bug.cgi?id=57179

        No new tests required. This can be tested with manual tests from
        WebCore/manual-tests/qt and WebCore/manual-tests/plugins

        * plugins/symbian/PluginViewSymbian.cpp:
        (WebCore::PluginView::updatePluginWidget):

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

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

index bcffba4..5a92209 100644 (file)
@@ -1,3 +1,21 @@
+2011-04-08  Misha Tyutyunik  <michael.tyutyunik@nokia.com>
+
+        Reviewed by Laszlo Gombos.
+
+        [Qt] Plugin is not scrolled together with page content or jumping 
+        when content is rendered using cache (backing store).
+        https://bugs.webkit.org/show_bug.cgi?id=56130
+        
+        This is a Symbian portion of the fix. For Linux it looks like
+        it's enough to call set geometry which should be addressed in
+        https://bugs.webkit.org/show_bug.cgi?id=57179
+
+        No new tests required. This can be tested with manual tests from 
+        WebCore/manual-tests/qt and WebCore/manual-tests/plugins
+
+        * plugins/symbian/PluginViewSymbian.cpp:
+        (WebCore::PluginView::updatePluginWidget):
+
 2011-04-08  Abhishek Arya  <inferno@chromium.org>
 
         Unreviewed. Remove accidental printf added in r83306.
index 1a9a9ca..13aa60f 100644 (file)
@@ -102,12 +102,7 @@ void PluginView::updatePluginWidget()
     if (m_windowRect == oldWindowRect && m_clipRect == oldClipRect)
         return;
 
-    // in order to move/resize the plugin window at the same time as the rest of frame
-    // during e.g. scrolling, we set the mask and geometry in the paint() function, but
-    // as paint() isn't called when the plugin window is outside the frame which can
-    // be caused by a scroll, we need to move/resize immediately.
-    if (!m_windowRect.intersects(frameView->frameRect()))
-        setNPWindowIfNeeded();
+    setNPWindowIfNeeded();
 }
 
 void PluginView::setFocus(bool focused)