Title: [214376] trunk/Source/WebKit2
- Revision
- 214376
- Author
- [email protected]
- Date
- 2017-03-24 14:14:37 -0700 (Fri, 24 Mar 2017)
Log Message
Fix assertions after r214358.
https://bugs.webkit.org/show_bug.cgi?id=169167
* UIProcess/API/APIContentExtensionStore.cpp:
(API::ContentExtensionStore::compileContentExtension):
AtomicString::init must be called for the first time from the main thread.
We were calling it for the first time from a non-main thread sometimes.
Call it from the main thread before initiating compiling on another thread.
Modified Paths
Diff
Modified: trunk/Source/WebKit2/ChangeLog (214375 => 214376)
--- trunk/Source/WebKit2/ChangeLog 2017-03-24 21:13:40 UTC (rev 214375)
+++ trunk/Source/WebKit2/ChangeLog 2017-03-24 21:14:37 UTC (rev 214376)
@@ -1 +1,12 @@
+2017-03-24 Alex Christensen <[email protected]>
+
+ Fix assertions after r214358.
+ https://bugs.webkit.org/show_bug.cgi?id=169167
+
+ * UIProcess/API/APIContentExtensionStore.cpp:
+ (API::ContentExtensionStore::compileContentExtension):
+ AtomicString::init must be called for the first time from the main thread.
+ We were calling it for the first time from a non-main thread sometimes.
+ Call it from the main thread before initiating compiling on another thread.
+
== Rolled over to ChangeLog-2017-03-23 ==
Modified: trunk/Source/WebKit2/UIProcess/API/APIContentExtensionStore.cpp (214375 => 214376)
--- trunk/Source/WebKit2/UIProcess/API/APIContentExtensionStore.cpp 2017-03-24 21:13:40 UTC (rev 214375)
+++ trunk/Source/WebKit2/UIProcess/API/APIContentExtensionStore.cpp 2017-03-24 21:14:37 UTC (rev 214376)
@@ -380,6 +380,7 @@
void ContentExtensionStore::compileContentExtension(const WTF::String& identifier, WTF::String&& json, Function<void(RefPtr<API::ContentExtension>, std::error_code)> completionHandler)
{
+ AtomicString::init();
m_compileQueue->dispatch([protectedThis = makeRef(*this), identifier = identifier.isolatedCopy(), json = json.isolatedCopy(), storePath = m_storePath.isolatedCopy(), completionHandler = WTFMove(completionHandler)] () mutable {
auto path = constructedPath(storePath, identifier);
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
