Bug 175831 – Opt all Mac clients into document-level media user gesture quirks

• Status: RESOLVED
• Resolution: FIXED
• Component: Media

[Matt Rajca]

This un-breaks many of the sites that take a long time to load video ad elements on-demand.

[Matt Rajca]

Created attachment 318765 [details]
Patch

[Matt Rajca]

Not to be reviewed.

[Build Bot]

Comment on attachment 318765 [details]
Patch

Attachment 318765 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.webkit.org/results/4362089

New failing tests:
http/tests/media/autoplay-if-audio-is-playing.html

[Build Bot]

Created attachment 318777 [details]
Archive of layout-test-results from ews106 for mac-elcapitan-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews106  Port: mac-elcapitan-wk2  Platform: Mac OS X 10.11.6

[Build Bot]

Comment on attachment 318765 [details]
Patch

Attachment 318765 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.webkit.org/results/4362270

New failing tests:
http/tests/media/autoplay-if-audio-is-playing.html

[Build Bot]

Created attachment 318778 [details]
Archive of layout-test-results from ews100 for mac-elcapitan

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews100  Port: mac-elcapitan  Platform: Mac OS X 10.11.6

[Build Bot]

Comment on attachment 318765 [details]
Patch

Attachment 318765 [details] did not pass mac-debug-ews (mac):
Output: http://webkit-queues.webkit.org/results/4362276

New failing tests:
http/tests/media/autoplay-if-audio-is-playing.html

[Build Bot]

Created attachment 318779 [details]
Archive of layout-test-results from ews117 for mac-elcapitan

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews117  Port: mac-elcapitan  Platform: Mac OS X 10.11.6

[Matt Rajca]

Created attachment 318790 [details]
Patch

[Matt Rajca]

<rdar://problem/33996723>

[Build Bot]

Comment on attachment 318790 [details]
Patch

Attachment 318790 [details] did not pass mac-debug-ews (mac):
Output: http://webkit-queues.webkit.org/results/4364207

New failing tests:
media/track/track-element-load-event.html

[Build Bot]

Created attachment 318821 [details]
Archive of layout-test-results from ews116 for mac-elcapitan

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews116  Port: mac-elcapitan  Platform: Mac OS X 10.11.6

[Matt Rajca]

(In reply to Build Bot from comment #12)
> Created attachment 318821 [details]
> Archive of layout-test-results from ews116 for mac-elcapitan
> 
> The attached test failures were seen while running run-webkit-tests on the
> mac-debug-ews.
> Bot: ews116  Port: mac-elcapitan  Platform: Mac OS X 10.11.6

The Mac debug failure looks like an unrelated assertion failure in updating text track displays: ASSERT(m_mediaControlsHost);

[Matt Rajca]

Created attachment 318873 [details]
Patch

[Matt Rajca]

(In reply to Matt Rajca from comment #13)
> (In reply to Build Bot from comment #12)
> > Created attachment 318821 [details]
> > Archive of layout-test-results from ews116 for mac-elcapitan
> > 
> > The attached test failures were seen while running run-webkit-tests on the
> > mac-debug-ews.
> > Bot: ews116  Port: mac-elcapitan  Platform: Mac OS X 10.11.6
> 
> The Mac debug failure looks like an unrelated assertion failure in updating
> text track displays: ASSERT(m_mediaControlsHost);

Re-running EWS fixed it.

[Eric Carlson]

Comment on attachment 318873 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=318873&action=review

> Source/WebCore/ChangeLog:3
> +        Opt all Mac clients into document-level media user gesture quirks.

"all Mac clients" is not accurate, can't any host app set needsSiteSpecificQuirks to true?

[Matt Rajca]

(In reply to Eric Carlson from comment #16)
> Comment on attachment 318873 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=318873&action=review
> 
> > Source/WebCore/ChangeLog:3
> > +        Opt all Mac clients into document-level media user gesture quirks.
> 
> "all Mac clients" is not accurate, can't any host app set
> needsSiteSpecificQuirks to true?

This quirk is under #if PLATFORM(MAC) so it doesn't affect iOS.

[Eric Carlson]

Comment on attachment 318873 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=318873&action=review

>>> Source/WebCore/ChangeLog:3
>>> +        Opt all Mac clients into document-level media user gesture quirks.
>> 
>> "all Mac clients" is not accurate, can't any host app set needsSiteSpecificQuirks to true?
> 
> This quirk is under #if PLATFORM(MAC) so it doesn't affect iOS.

Yes, but won't this change the behavior of a WebKit app on a non-macOS platform that calls settings.setNeedsSiteSpecificQuirks(true)?

[Matt Rajca]

(In reply to Eric Carlson from comment #18)
> Comment on attachment 318873 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=318873&action=review
> 
> >>> Source/WebCore/ChangeLog:3
> >>> +        Opt all Mac clients into document-level media user gesture quirks.
> >> 
> >> "all Mac clients" is not accurate, can't any host app set needsSiteSpecificQuirks to true?
> > 
> > This quirk is under #if PLATFORM(MAC) so it doesn't affect iOS.
> 
> Yes, but won't this change the behavior of a WebKit app on a non-macOS
> platform that calls settings.setNeedsSiteSpecificQuirks(true)?

No. It only applies to Mac clients that opt-into quirks. Do we want it to run on non-macOS platforms as well?

[Matt Rajca]

Committed r221386: <http://trac.webkit.org/changeset/221386>