Title: [246047] branches/safari-607-branch/Source/WebCore
- Revision
- 246047
- Author
- [email protected]
- Date
- 2019-06-03 14:04:07 -0700 (Mon, 03 Jun 2019)
Log Message
Revert r246001. rdar://problem/50871594
Modified Paths
Diff
Modified: branches/safari-607-branch/Source/WebCore/ChangeLog (246046 => 246047)
--- branches/safari-607-branch/Source/WebCore/ChangeLog 2019-06-03 20:58:44 UTC (rev 246046)
+++ branches/safari-607-branch/Source/WebCore/ChangeLog 2019-06-03 21:04:07 UTC (rev 246047)
@@ -1,3 +1,7 @@
+2019-06-03 Alan Coon <[email protected]>
+
+ Revert r246001. rdar://problem/50871594
+
2019-05-31 Alan Coon <[email protected]>
Apply patch. rdar://problem/50871594
Modified: branches/safari-607-branch/Source/WebCore/Modules/mediastream/MediaStreamTrack.cpp (246046 => 246047)
--- branches/safari-607-branch/Source/WebCore/Modules/mediastream/MediaStreamTrack.cpp 2019-06-03 20:58:44 UTC (rev 246046)
+++ branches/safari-607-branch/Source/WebCore/Modules/mediastream/MediaStreamTrack.cpp 2019-06-03 21:04:07 UTC (rev 246047)
@@ -331,16 +331,16 @@
m_promise = WTFMove(promise);
auto weakThis = makeWeakPtr(*this);
- auto failureHandler = [weakThis] (const String& failedConstraint, const String& message) {
- if (!weakThis || !weakThis->m_promise)
+ auto failureHandler = [weakThis, this] (const String& failedConstraint, const String& message) {
+ if (!weakThis || !m_promise)
return;
- weakThis->m_promise->rejectType<IDLInterface<OverconstrainedError>>(OverconstrainedError::create(failedConstraint, message).get());
+ m_promise->rejectType<IDLInterface<OverconstrainedError>>(OverconstrainedError::create(failedConstraint, message).get());
};
- auto successHandler = [weakThis, constraints] () {
- if (!weakThis || !weakThis->m_promise)
+ auto successHandler = [weakThis, this, constraints] () {
+ if (!weakThis || !m_promise)
return;
- weakThis->m_promise->resolve();
- weakThis->m_constraints = constraints.valueOr(MediaTrackConstraints { });
+ m_promise->resolve();
+ m_constraints = constraints.valueOr(MediaTrackConstraints { });
};
m_private->applyConstraints(createMediaConstraints(constraints), WTFMove(successHandler), WTFMove(failureHandler));
}
Modified: branches/safari-607-branch/Source/WebCore/html/HTMLMediaElement.h (246046 => 246047)
--- branches/safari-607-branch/Source/WebCore/html/HTMLMediaElement.h 2019-06-03 20:58:44 UTC (rev 246046)
+++ branches/safari-607-branch/Source/WebCore/html/HTMLMediaElement.h 2019-06-03 21:04:07 UTC (rev 246047)
@@ -126,10 +126,11 @@
: public HTMLElement
, public ActiveDOMObject
, public MediaControllerInterface
+ , public CanMakeWeakPtr<HTMLMediaElement>
, public PlatformMediaSessionClient
, private MediaCanStartListener
, private MediaPlayerClient
- , public MediaProducer
+ , private MediaProducer
, private VisibilityChangeClient
, private ApplicationStateChangeListener
#if ENABLE(VIDEO_TRACK)
@@ -187,6 +188,8 @@
MediaPlayerEnums::VideoGravity videoFullscreenGravity() const { return m_videoFullscreenGravity; }
#endif
+ using CanMakeWeakPtr<HTMLMediaElement>::weakPtrFactory;
+
void scheduleCheckPlaybackTargetCompatability();
void checkPlaybackTargetCompatablity();
void scheduleResolvePendingPlayPromises();
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
