Title: [288880] trunk/Source/WebKit
Revision
288880
Author
[email protected]
Date
2022-02-01 08:49:53 -0800 (Tue, 01 Feb 2022)

Log Message

Minor cleanup in RemoteLayerBackingStoreCollection
https://bugs.webkit.org/show_bug.cgi?id=235936

Reviewed by Tim Horton.

Do time math once. Insert a blank line for clarity.

* Shared/RemoteLayerTree/RemoteLayerBackingStoreCollection.mm:
(WebKit::RemoteLayerBackingStoreCollection::markBackingStoreVolatile):
(WebKit::RemoteLayerBackingStoreCollection::backingStoreBecameUnreachable):

Modified Paths

Diff

Modified: trunk/Source/WebKit/ChangeLog (288879 => 288880)


--- trunk/Source/WebKit/ChangeLog	2022-02-01 16:41:43 UTC (rev 288879)
+++ trunk/Source/WebKit/ChangeLog	2022-02-01 16:49:53 UTC (rev 288880)
@@ -1,5 +1,18 @@
 2022-02-01  Simon Fraser  <[email protected]>
 
+        Minor cleanup in RemoteLayerBackingStoreCollection
+        https://bugs.webkit.org/show_bug.cgi?id=235936
+
+        Reviewed by Tim Horton.
+
+        Do time math once. Insert a blank line for clarity.
+
+        * Shared/RemoteLayerTree/RemoteLayerBackingStoreCollection.mm:
+        (WebKit::RemoteLayerBackingStoreCollection::markBackingStoreVolatile):
+        (WebKit::RemoteLayerBackingStoreCollection::backingStoreBecameUnreachable):
+
+2022-02-01  Simon Fraser  <[email protected]>
+
         RemoteRenderingBackendProxy should not be a IPC::MessageSender
         https://bugs.webkit.org/show_bug.cgi?id=235935
 

Modified: trunk/Source/WebKit/Shared/RemoteLayerTree/RemoteLayerBackingStoreCollection.mm (288879 => 288880)


--- trunk/Source/WebKit/Shared/RemoteLayerTree/RemoteLayerBackingStoreCollection.mm	2022-02-01 16:41:43 UTC (rev 288879)
+++ trunk/Source/WebKit/Shared/RemoteLayerTree/RemoteLayerBackingStoreCollection.mm	2022-02-01 16:49:53 UTC (rev 288880)
@@ -122,8 +122,9 @@
 
 bool RemoteLayerBackingStoreCollection::markBackingStoreVolatile(RemoteLayerBackingStore& backingStore, MonotonicTime now)
 {
-    if (now - backingStore.lastDisplayTime() < volatileBackingStoreAgeThreshold) {
-        if (now - backingStore.lastDisplayTime() >= volatileSecondaryBackingStoreAgeThreshold)
+    auto timeSinceLastDisplay = now - backingStore.lastDisplayTime();
+    if (timeSinceLastDisplay < volatileBackingStoreAgeThreshold) {
+        if (timeSinceLastDisplay >= volatileSecondaryBackingStoreAgeThreshold)
             backingStore.setBufferVolatility(RemoteLayerBackingStore::BufferType::SecondaryBack, true);
 
         return false;
@@ -139,6 +140,7 @@
     auto backingStoreIter = m_liveBackingStore.find(&backingStore);
     if (backingStoreIter == m_liveBackingStore.end())
         return;
+
     m_unparentedBackingStore.add(&backingStore);
     m_liveBackingStore.remove(backingStoreIter);
 
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to