Title: [271383] trunk/Source/WebCore
Revision
271383
Author
[email protected]
Date
2021-01-11 15:30:27 -0800 (Mon, 11 Jan 2021)

Log Message

Use sendWithAsyncReply instead of dataCallback for icon loading
https://bugs.webkit.org/show_bug.cgi?id=220381

* loader/DocumentLoader.cpp:
(WebCore::DocumentLoader::stopLoading):
Use the original version of the patch that didn't break WK1 tests.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (271382 => 271383)


--- trunk/Source/WebCore/ChangeLog	2021-01-11 22:57:05 UTC (rev 271382)
+++ trunk/Source/WebCore/ChangeLog	2021-01-11 23:30:27 UTC (rev 271383)
@@ -1,3 +1,12 @@
+2021-01-11  Alex Christensen  <[email protected]>
+
+        Use sendWithAsyncReply instead of dataCallback for icon loading
+        https://bugs.webkit.org/show_bug.cgi?id=220381
+
+        * loader/DocumentLoader.cpp:
+        (WebCore::DocumentLoader::stopLoading):
+        Use the original version of the patch that didn't break WK1 tests.
+
 2021-01-11  Julian Gonzalez  <[email protected]>
 
         Relax assertion in Element::dispatchFocusOutEvent() for non-web process case

Modified: trunk/Source/WebCore/loader/DocumentLoader.cpp (271382 => 271383)


--- trunk/Source/WebCore/loader/DocumentLoader.cpp	2021-01-11 22:57:05 UTC (rev 271382)
+++ trunk/Source/WebCore/loader/DocumentLoader.cpp	2021-01-11 23:30:27 UTC (rev 271383)
@@ -323,8 +323,9 @@
             m_frame->loader().stopLoading(UnloadEventPolicy::None);
     }
 
-    for (auto& callback : std::exchange(m_iconLoaders, { }).values())
+    for (auto& callback : m_iconLoaders.values())
         callback(nullptr);
+    m_iconLoaders.clear();
     m_iconsPendingLoadDecision.clear();
     
 #if ENABLE(APPLICATION_MANIFEST)
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to