Bug 163244 – [iOS] REGRESSION (r197953): User gesture required to load video in iOS 9-built apps

• Status: RESOLVED
• Resolution: FIXED
• Component: WebCore Misc.

[Daniel Bates]

Following <https://trac.webkit.org/changeset/197953> (bug #155141), videos require a user gesture to load on iOS. Among other implications of this change, we dispatch a DOM loadedmetadata event at a <video> event only if a human being initiated playback of the video. This changes makes iOS more closely align with the behavior on macOS. However it is represents a backward incompatible change from the behavior in iOS 9 and earlier. We should only enable this behavioral change for apps built against iOS 10 or later.

[Daniel Bates]

<rdar://problem/27250015>

[Daniel Bates]

Created attachment 291173 [details]
Patch and layout tests

[Build Bot]

Comment on attachment 291173 [details]
Patch and layout tests

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

New failing tests:
media/loadedmetadata-fires-without-user-gesture-when-setRequiresUserGestureToLoadVideo-false.html
media/require-user-gesture-to-load-video.html

[Build Bot]

Created attachment 291176 [details]
Archive of layout-test-results from ews103 for mac-yosemite

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

[Build Bot]

Comment on attachment 291173 [details]
Patch and layout tests

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

New failing tests:
media/loadedmetadata-fires-without-user-gesture-when-setRequiresUserGestureToLoadVideo-false.html
media/require-user-gesture-to-load-video.html

[Build Bot]

Created attachment 291177 [details]
Archive of layout-test-results from ews107 for mac-yosemite-wk2

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

[Daniel Bates]

(In reply to comment #3)
> Comment on attachment 291173 [details]
> Patch and layout tests
> 
> Attachment 291173 [details] did not pass mac-ews (mac):
> Output: http://webkit-queues.webkit.org/results/2258285
> 
> New failing tests:
> media/loadedmetadata-fires-without-user-gesture-when-
> setRequiresUserGestureToLoadVideo-false.html
> media/require-user-gesture-to-load-video.html

For some reason the bots do not seem to be re-generating the Internal Settings bindings from Settings.in (why?). So, window.internals.settings.requiresUserGestureToLoadVideo is undefined.

[Daniel Bates]

(In reply to comment #7)
> (In reply to comment #3)
> > Comment on attachment 291173 [details]
> > Patch and layout tests
> > 
> > Attachment 291173 [details] did not pass mac-ews (mac):
> > Output: http://webkit-queues.webkit.org/results/2258285
> > 
> > New failing tests:
> > media/loadedmetadata-fires-without-user-gesture-when-
> > setRequiresUserGestureToLoadVideo-false.html
> > media/require-user-gesture-to-load-video.html
> 
> For some reason the bots do not seem to be re-generating the Internal
> Settings bindings from Settings.in (why?). So,
> window.internals.settings.requiresUserGestureToLoadVideo is undefined.

err, window.internals.settings.setRequiresUserGestureToLoadVideo

[Build Bot]

Comment on attachment 291173 [details]
Patch and layout tests

Attachment 291173 [details] did not pass ios-sim-ews (ios-simulator-wk2):
Output: http://webkit-queues.webkit.org/results/2258358

New failing tests:
media/loadedmetadata-fires-without-user-gesture-when-setRequiresUserGestureToLoadVideo-false.html
media/require-user-gesture-to-load-video.html

[Build Bot]

Created attachment 291188 [details]
Archive of layout-test-results from ews122 for ios-simulator-wk2

The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews122  Port: ios-simulator-wk2  Platform: Mac OS X 10.11.6

[Daniel Bates]

Created attachment 291280 [details]
Patch and layout tests

[Build Bot]

Comment on attachment 291280 [details]
Patch and layout tests

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

New failing tests:
media/loadedmetadata-fires-without-user-gesture-when-setRequiresUserGestureToLoadVideo-false.html
media/require-user-gesture-to-load-video.html

[Build Bot]

Created attachment 291289 [details]
Archive of layout-test-results from ews104 for mac-yosemite-wk2

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

[Daniel Bates]

(In reply to comment #13)
> Created attachment 291289 [details]
> Archive of layout-test-results from ews104 for mac-yosemite-wk2
> 
> The attached test failures were seen while running run-webkit-tests on the
> mac-wk2-ews.
> Bot: ews104  Port: mac-yosemite-wk2  Platform: Mac OS X 10.10.5

Same issue as remarked in comment #7. For some reason the mac-yosemite-wk2 is not re-generating the Internal Settings bindings from Settings.in (why?). Notice that the mac-yosemite EWS built and ran all the tests successfully.

[Daniel Bates]

Comment on attachment 291280 [details]
Patch and layout tests

Clearing flags on attachment: 291280

Committed r207159: <http://trac.webkit.org/changeset/207159>

[Daniel Bates]

All reviewed patches have been landed.  Closing bug.

[Daniel Bates]

Committed build fix for the Apple Internal Mac build in <http://trac.webkit.org/changeset/207171>.