Changeset 73195 in webkit

Timestamp:

Dec 2, 2010 1:49:11 PM (13 years ago)

Author:

andersca@apple.com

Message:

Flash videos on YouTube flicker
​https://bugs.webkit.org/show_bug.cgi?id=50409
<rdar://problem/8668757>

Reviewed by Sam Weinig.

When painting from a backing store, we need to add the source location
to the destination point.

• WebProcess/Plugins/PluginProxy.cpp:

(WebKit::PluginProxy::paint):
(WebKit::PluginProxy::update):

Location:

trunk/WebKit2

Files:

• ChangeLog (modified) (1 diff)
• WebProcess/Plugins/PluginProxy.cpp (modified) (2 diffs)

trunk/WebKit2/ChangeLog

@@ +1 @@
+2010-12-02  Anders Carlsson  <andersca@apple.com>
+
+        Reviewed by Sam Weinig.
+
+        Flash videos on YouTube flicker
+        https://bugs.webkit.org/show_bug.cgi?id=50409
+        <rdar://problem/8668757>
+
+        When painting from a backing store, we need to add the source location
+        to the destination point.
+
+        * WebProcess/Plugins/PluginProxy.cpp:
+        (WebKit::PluginProxy::paint):
+        (WebKit::PluginProxy::update):
+
 2010-12-02  Sam Weinig  <sam@webkit.org>
 

trunk/WebKit2/WebProcess/Plugins/PluginProxy.cpp

@@ -132 +132 @@
     dirtyRectInPluginCoordinates.move(-m_frameRect.x(), -m_frameRect.y());
 
-    m_backingStore->paint(*graphicsContext, m_frameRect.location(), dirtyRectInPluginCoordinates);
+    m_backingStore->paint(*graphicsContext, dirtyRect.location(), dirtyRectInPluginCoordinates);
 
     if (m_waitingForPaintInResponseToUpdate) {
@@ -420 +420 @@
         OwnPtr<GraphicsContext> graphicsContext = m_backingStore->createGraphicsContext();
 
-        m_pluginBackingStore->paint(*graphicsContext, IntPoint(), paintedRectPluginCoordinates);
+        m_pluginBackingStore->paint(*graphicsContext, paintedRectPluginCoordinates.location(), 
+                                    paintedRectPluginCoordinates);
     }