Title: [227033] branches/safari-605-branch/Source/WebCore
- Revision
- 227033
- Author
- [email protected]
- Date
- 2018-01-16 21:04:39 -0800 (Tue, 16 Jan 2018)
Log Message
Cherry-pick r226951. rdar://problem/36568098
Modified Paths
Diff
Modified: branches/safari-605-branch/Source/WebCore/ChangeLog (227032 => 227033)
--- branches/safari-605-branch/Source/WebCore/ChangeLog 2018-01-17 05:04:37 UTC (rev 227032)
+++ branches/safari-605-branch/Source/WebCore/ChangeLog 2018-01-17 05:04:39 UTC (rev 227033)
@@ -1,5 +1,23 @@
2018-01-16 Jason Marcell <[email protected]>
+ Cherry-pick r226951. rdar://problem/36568098
+
+ 2018-01-15 Youenn Fablet <[email protected]>
+
+ RealtimeMediaSource should be ThreadSafeRefCounted
+ https://bugs.webkit.org/show_bug.cgi?id=181649
+
+ Reviewed by Eric Carlson.
+
+ Difficult to write a test as this is really racy.
+ RealtimeIncomingVideoSourceCocoa::OnFrame is taking a reference on a background thread
+ to send a task to the main thread.
+ This requires it to be thread safe ref counted.
+
+ * platform/mediastream/RealtimeMediaSource.h:
+
+2018-01-16 Jason Marcell <[email protected]>
+
Cherry-pick r226930. rdar://problem/36567972
2018-01-12 Myles C. Maxfield <[email protected]>
Modified: branches/safari-605-branch/Source/WebCore/platform/mediastream/RealtimeMediaSource.h (227032 => 227033)
--- branches/safari-605-branch/Source/WebCore/platform/mediastream/RealtimeMediaSource.h 2018-01-17 05:04:37 UTC (rev 227032)
+++ branches/safari-605-branch/Source/WebCore/platform/mediastream/RealtimeMediaSource.h 2018-01-17 05:04:39 UTC (rev 227033)
@@ -41,7 +41,7 @@
#include "MediaSample.h"
#include "PlatformLayer.h"
#include "RealtimeMediaSourceCapabilities.h"
-#include <wtf/RefCounted.h>
+#include <wtf/ThreadSafeRefCounted.h>
#include <wtf/Vector.h>
#include <wtf/WeakPtr.h>
#include <wtf/text/WTFString.h>
@@ -62,7 +62,7 @@
struct CaptureSourceOrError;
-class WEBCORE_EXPORT RealtimeMediaSource : public RefCounted<RealtimeMediaSource> {
+class WEBCORE_EXPORT RealtimeMediaSource : public ThreadSafeRefCounted<RealtimeMediaSource> {
public:
class Observer {
public:
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
