Support quirk for letting media autoplay if the user interacted with at least one media element.
Created attachment 316815 [details] Patch
Comment on attachment 316815 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=316815&action=review > Source/WebCore/ChangeLog:5 > + Radar? > Source/WebCore/html/MediaElementSession.cpp:194 > + if (needsDocumentLevelMediaUserGestureQuirk(element.document()) && element.document().topDocument().mediaState() & MediaProducer::HasUserInteractedWithMediaElement) Nit: may be worth swapping the order of the comparisons.
(In reply to Eric Carlson from comment #2) > Comment on attachment 316815 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=316815&action=review > > > Source/WebCore/ChangeLog:5 > > + > > Radar? Added. > > > Source/WebCore/html/MediaElementSession.cpp:194 > > + if (needsDocumentLevelMediaUserGestureQuirk(element.document()) && element.document().topDocument().mediaState() & MediaProducer::HasUserInteractedWithMediaElement) > > Nit: may be worth swapping the order of the comparisons. Done.
(In reply to Eric Carlson from comment #2) > Comment on attachment 316815 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=316815&action=review > > > Source/WebCore/ChangeLog:5 > > + > > Radar? > > > Source/WebCore/html/MediaElementSession.cpp:194 > > + if (needsDocumentLevelMediaUserGestureQuirk(element.document()) && element.document().topDocument().mediaState() & MediaProducer::HasUserInteractedWithMediaElement) > > Nit: may be worth swapping the order of the comparisons. I also put the quirk in #if PLATFORM(MAC) like we talked about.
Committed r220085: <http://trac.webkit.org/changeset/220085>
<rdar://problem/33641773>