Title: [273054] trunk/Source/WebKit
Revision
273054
Author
hironori.fu...@sony.com
Date
2021-02-17 17:46:51 -0800 (Wed, 17 Feb 2021)

Log Message

Unreviewed, reverting r273019.

GTK port layout tests are timint out.

Reverted changeset:

"[CoordinatedGraphics] The whole content is unnecessarily
repainted by animations in non-AC mode pages"
https://bugs.webkit.org/show_bug.cgi?id=221391
https://commits.webkit.org/r273019

Modified Paths

Diff

Modified: trunk/Source/WebKit/ChangeLog (273053 => 273054)


--- trunk/Source/WebKit/ChangeLog	2021-02-18 01:24:41 UTC (rev 273053)
+++ trunk/Source/WebKit/ChangeLog	2021-02-18 01:46:51 UTC (rev 273054)
@@ -1,3 +1,16 @@
+2021-02-17  Fujii Hironori  <hironori.fu...@sony.com>
+
+        Unreviewed, reverting r273019.
+
+        GTK port layout tests are timint out.
+
+        Reverted changeset:
+
+        "[CoordinatedGraphics] The whole content is unnecessarily
+        repainted by animations in non-AC mode pages"
+        https://bugs.webkit.org/show_bug.cgi?id=221391
+        https://commits.webkit.org/r273019
+
 2021-02-17  Aditya Keerthi  <akeer...@apple.com>
 
         [iOS][FCR] Add new datalist dropdown for text inputs

Modified: trunk/Source/WebKit/WebProcess/WebPage/CoordinatedGraphics/DrawingAreaCoordinatedGraphics.cpp (273053 => 273054)


--- trunk/Source/WebKit/WebProcess/WebPage/CoordinatedGraphics/DrawingAreaCoordinatedGraphics.cpp	2021-02-18 01:24:41 UTC (rev 273053)
+++ trunk/Source/WebKit/WebProcess/WebPage/CoordinatedGraphics/DrawingAreaCoordinatedGraphics.cpp	2021-02-18 01:46:51 UTC (rev 273054)
@@ -356,7 +356,7 @@
     if (m_layerTreeHost)
         m_layerTreeHost->scheduleLayerFlush();
     else
-        scheduleDisplay();
+        setNeedsDisplay();
 }
 
 #if USE(COORDINATED_GRAPHICS)
@@ -473,9 +473,6 @@
 
     m_isWaitingForDidUpdate = false;
 
-    if (m_dirtyRegion.isEmpty())
-        return;
-
     // Display if needed. We call displayTimerFired here since it will throttle updates to 60fps.
     displayTimerFired();
 }
@@ -686,6 +683,9 @@
     if (m_isPaintingSuspended)
         return;
 
+    if (m_dirtyRegion.isEmpty())
+        return;
+
     if (m_displayTimer.isActive())
         return;
 
@@ -709,6 +709,9 @@
     if (m_isPaintingSuspended)
         return;
 
+    if (m_dirtyRegion.isEmpty())
+        return;
+
     if (m_shouldSendDidUpdateBackingStoreState) {
         sendDidUpdateBackingStoreState();
         return;
@@ -763,9 +766,6 @@
     if (m_layerTreeHost)
         return;
 
-    if (m_dirtyRegion.isEmpty())
-        return;
-
     updateInfo.viewSize = m_webPage.size();
     updateInfo.deviceScaleFactor = m_webPage.corePage()->deviceScaleFactor();
 
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to