Attachment 343854 Details for Bug 187156 – patch

[patch] patch

memory-cache-client-call-post-resume-2.patch (text/plain), 2.41 KB, created by Antti Koivisto on 2018-06-28 14:31:41 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Antti Koivisto

Created: 2018-06-28 14:31:41 PDT

Size: 2.41 KB

patch

obsolete

>Index: Source/WebCore/ChangeLog
>===================================================================
>--- Source/WebCore/ChangeLog	(revision 233328)
>+++ Source/WebCore/ChangeLog	(working copy)
>@@ -1,3 +1,20 @@
>+2018-06-28  Antti Koivisto  <antti@apple.com>
>+
>+        REGRESSION (233281): fast/dom/location-new-window-no-crash.html and some other tests are timing out
>+        https://bugs.webkit.org/show_bug.cgi?id=187156
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        We need still need to re-enable memory cache client calls even when not doing other post-resolution callbacks.
>+
>+        * style/StyleTreeResolver.cpp:
>+        (WebCore::Style::memoryCacheClientCallsResumeQueue):
>+
>+        Add a separate queue for this.
>+
>+        (WebCore::Style::suspendMemoryCacheClientCalls):
>+        (WebCore::Style::PostResolutionCallbackDisabler::~PostResolutionCallbackDisabler):
>+
> 2018-06-28  Antoine Quint  <graouts@apple.com>
> 
>         [Web Animations] Make imported/mozilla/css-animations/test_animation-starttime.html pass reliably
>Index: Source/WebCore/style/StyleTreeResolver.cpp
>===================================================================
>--- Source/WebCore/style/StyleTreeResolver.cpp	(revision 233307)
>+++ Source/WebCore/style/StyleTreeResolver.cpp	(working copy)
>@@ -579,6 +579,12 @@ static Vector<Function<void ()>>& postRe
>     return vector;
> }
> 
>+static Vector<Function<void ()>>& memoryCacheClientCallsResumeQueue()
>+{
>+    static NeverDestroyed<Vector<Function<void ()>>> vector;
>+    return vector;
>+}
>+
> void queuePostResolutionCallback(Function<void ()>&& callback)
> {
>     postResolutionCallbackQueue().append(WTFMove(callback));
>@@ -592,7 +598,7 @@ static void suspendMemoryCacheClientCall
> 
>     page->setMemoryCacheClientCallsEnabled(false);
> 
>-    postResolutionCallbackQueue().append([protectedMainFrame = makeRef(page->mainFrame())] {
>+    memoryCacheClientCallsResumeQueue().append([protectedMainFrame = makeRef(page->mainFrame())] {
>         if (Page* page = protectedMainFrame->page())
>             page->setMemoryCacheClientCallsEnabled(true);
>     });
>@@ -622,6 +628,12 @@ PostResolutionCallbackDisabler::~PostRes
>                 queue[i]();
>             queue.clear();
>         }
>+
>+        auto& queue = memoryCacheClientCallsResumeQueue();
>+        for (size_t i = 0; i < queue.size(); ++i)
>+            queue[i]();
>+        queue.clear();
>+
>         platformStrategies()->loaderStrategy()->resumePendingRequests();
>     }
>

Flags:

zalan: review+

Actions: View | Formatted Diff | Diff

Attachments on bug 187156: 343852 | 343854 | 343856 | 343870