Title: [218712] trunk/Source/WebKit2
- Revision
- 218712
- Author
- [email protected]
- Date
- 2017-06-22 12:41:08 -0700 (Thu, 22 Jun 2017)
Log Message
Add release assertion in CallbackMap::invalidate()
https://bugs.webkit.org/show_bug.cgi?id=173722
Reviewed by Brady Eidson.
Add release assertion in CallbackMap::invalidate() to make sure it is called on the main thread.
* UIProcess/GenericCallback.h:
(WebKit::CallbackMap::invalidate):
Modified Paths
Diff
Modified: trunk/Source/WebKit2/ChangeLog (218711 => 218712)
--- trunk/Source/WebKit2/ChangeLog 2017-06-22 19:38:39 UTC (rev 218711)
+++ trunk/Source/WebKit2/ChangeLog 2017-06-22 19:41:08 UTC (rev 218712)
@@ -1,3 +1,15 @@
+2017-06-22 Chris Dumez <[email protected]>
+
+ Add release assertion in CallbackMap::invalidate()
+ https://bugs.webkit.org/show_bug.cgi?id=173722
+
+ Reviewed by Brady Eidson.
+
+ Add release assertion in CallbackMap::invalidate() to make sure it is called on the main thread.
+
+ * UIProcess/GenericCallback.h:
+ (WebKit::CallbackMap::invalidate):
+
2017-06-22 Brady Eidson <[email protected]>
Add some thread safety guards to GenericCallback.
Modified: trunk/Source/WebKit2/UIProcess/GenericCallback.h (218711 => 218712)
--- trunk/Source/WebKit2/UIProcess/GenericCallback.h 2017-06-22 19:38:39 UTC (rev 218711)
+++ trunk/Source/WebKit2/UIProcess/GenericCallback.h 2017-06-22 19:41:08 UTC (rev 218712)
@@ -221,6 +221,7 @@
void invalidate(CallbackBase::Error error)
{
+ RELEASE_ASSERT(isMainThread());
invalidateCallbackMap(m_map, error);
}
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
