Changeset 87642 in webkit

Timestamp:

May 29, 2011 1:06:10 PM (13 years ago)

Author:

andersca@apple.com

Message:

2011-05-29 Anders Carlsson <​andersca@apple.com>

Reviewed by Darin Adler.

REGRESSION (87637): Entering WebKit Full Screen mode causes an assertion in DrawingAreaImpl::layerHostDidFlushLayers
​https://bugs.webkit.org/show_bug.cgi?id=61706
<rdar://problem/9521970>

If we've already let the UI process know that we're in compositing mode, we just need to force a repaint.

• WebProcess/WebPage/DrawingAreaImpl.cpp: (WebKit::DrawingAreaImpl::forceRepaint):

Location:

trunk/Source/WebKit2

Files:

• ChangeLog (modified) (1 diff)
• WebProcess/WebPage/DrawingAreaImpl.cpp (modified) (1 diff)

trunk/Source/WebKit2/ChangeLog

@@ +1 @@
+2011-05-29  Anders Carlsson  <andersca@apple.com>
+
+        Reviewed by Darin Adler.
+
+        REGRESSION (87637): Entering WebKit Full Screen mode causes an assertion in DrawingAreaImpl::layerHostDidFlushLayers
+        https://bugs.webkit.org/show_bug.cgi?id=61706
+        <rdar://problem/9521970>
+
+        If we've already let the UI process know that we're in compositing mode, we just need to force a repaint.
+
+        * WebProcess/WebPage/DrawingAreaImpl.cpp:
+        (WebKit::DrawingAreaImpl::forceRepaint):
+
 2011-05-29  Beth Dakin  <bdakin@apple.com>
 

trunk/Source/WebKit2/WebProcess/WebPage/DrawingAreaImpl.cpp

@@ -159 +159 @@
         // FIXME: We need to do the same work as the layerHostDidFlushLayers function here,
         // but clearly it doesn't make sense to call the function with that name.
-        // Consider renaming it.
-        layerHostDidFlushLayers();
+        // Consider refactoring and renaming it.
+        if (m_compositingAccordingToProxyMessages)
+            m_layerTreeHost->forceRepaint();
+        else
+            layerHostDidFlushLayers();
         if (!m_layerTreeHost->participatesInDisplay())
             return;