Title: [277447] trunk/Source/WebKit
Revision
277447
Author
[email protected]
Date
2021-05-13 12:45:02 -0700 (Thu, 13 May 2021)

Log Message

Process suspension may be delayed by up to 5 seconds after there is no longer any database activity
https://bugs.webkit.org/show_bug.cgi?id=225774

Reviewed by Geoffrey Garen.

Use a 1 second HysteresisActivity in WebSQLiteDatabaseTracker instead of a 5 seconds one.
I frequently see our processes' suspension getting delayed by 5 seconds when homing out of
MobileSafari due to database activity. This change should help with that.

* Shared/WebSQLiteDatabaseTracker.cpp:
(WebKit::WebSQLiteDatabaseTracker::WebSQLiteDatabaseTracker):

Modified Paths

Diff

Modified: trunk/Source/WebKit/ChangeLog (277446 => 277447)


--- trunk/Source/WebKit/ChangeLog	2021-05-13 19:43:37 UTC (rev 277446)
+++ trunk/Source/WebKit/ChangeLog	2021-05-13 19:45:02 UTC (rev 277447)
@@ -1,5 +1,19 @@
 2021-05-13  Chris Dumez  <[email protected]>
 
+        Process suspension may be delayed by up to 5 seconds after there is no longer any database activity
+        https://bugs.webkit.org/show_bug.cgi?id=225774
+
+        Reviewed by Geoffrey Garen.
+
+        Use a 1 second HysteresisActivity in WebSQLiteDatabaseTracker instead of a 5 seconds one.
+        I frequently see our processes' suspension getting delayed by 5 seconds when homing out of
+        MobileSafari due to database activity. This change should help with that.
+
+        * Shared/WebSQLiteDatabaseTracker.cpp:
+        (WebKit::WebSQLiteDatabaseTracker::WebSQLiteDatabaseTracker):
+
+2021-05-13  Chris Dumez  <[email protected]>
+
         Introduce FileSystem::hardLinkCount()
         https://bugs.webkit.org/show_bug.cgi?id=225767
 

Modified: trunk/Source/WebKit/Shared/WebSQLiteDatabaseTracker.cpp (277446 => 277447)


--- trunk/Source/WebKit/Shared/WebSQLiteDatabaseTracker.cpp	2021-05-13 19:43:37 UTC (rev 277446)
+++ trunk/Source/WebKit/Shared/WebSQLiteDatabaseTracker.cpp	2021-05-13 19:45:02 UTC (rev 277447)
@@ -37,7 +37,7 @@
 
 WebSQLiteDatabaseTracker::WebSQLiteDatabaseTracker(IsHoldingLockedFilesHandler&& isHoldingLockedFilesHandler)
     : m_isHoldingLockedFilesHandler(WTFMove(isHoldingLockedFilesHandler))
-    , m_hysteresis([this](PAL::HysteresisState state) { setIsHoldingLockedFiles(state == PAL::HysteresisState::Started); })
+    , m_hysteresis([this](PAL::HysteresisState state) { setIsHoldingLockedFiles(state == PAL::HysteresisState::Started); }, 1_s)
 {
     ASSERT(RunLoop::isMain());
     SQLiteDatabaseTracker::setClient(this);
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to