Bug 187193 – Video sometimes flickers when playing to AppleTV

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

[Eric Carlson]

Video sometimes flickers when playing to AppleTV.

[Eric Carlson]

<rdar://problem/40153865>

[Eric Carlson]

Created attachment 343942 [details]
Patch

[Jer Noble]

Comment on attachment 343942 [details]
Patch

Nit: I'd rather see separate "scheduleUpdateSessionState()" and "updateSessionState()" methods than a single method with a default parameter.

[EWS Watchlist]

Comment on attachment 343942 [details]
Patch

Attachment 343942 [details] did not pass mac-ews (mac):
Output: https://webkit-queues.webkit.org/results/8387349

New failing tests:
platform/mac/media/audio-session-category-audio-autoplay.html

[EWS Watchlist]

Created attachment 343948 [details]
Archive of layout-test-results from ews101 for mac-sierra

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

[EWS Watchlist]

Comment on attachment 343942 [details]
Patch

Attachment 343942 [details] did not pass mac-debug-ews (mac):
Output: https://webkit-queues.webkit.org/results/8387320

New failing tests:
platform/mac/media/audio-session-category-audio-autoplay.html
http/tests/security/webaudio-render-remote-audio-blocked-no-crossorigin.html
http/tests/security/webaudio-render-remote-audio-allowed-crossorigin.html

[EWS Watchlist]

Created attachment 343953 [details]
Archive of layout-test-results from ews114 for mac-sierra

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

[EWS Watchlist]

Comment on attachment 343942 [details]
Patch

Attachment 343942 [details] did not pass mac-wk2-ews (mac-wk2):
Output: https://webkit-queues.webkit.org/results/8389699

New failing tests:
platform/mac/media/audio-session-category-audio-autoplay.html

[EWS Watchlist]

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

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

[Eric Carlson]

Created attachment 344122 [details]
Patch for landing

[WebKit Commit Bot]

Comment on attachment 344122 [details]
Patch for landing

Clearing flags on attachment: 344122

Committed r233435: <https://trac.webkit.org/changeset/233435>

[Dawei Fenton (:realdawei)]

(In reply to WebKit Commit Bot from comment #11)
> Comment on attachment 344122 [details]
> Patch for landing
> 
> Clearing flags on attachment: 344122
> 
> Committed r233435: <https://trac.webkit.org/changeset/233435>

Seeing crashes in iOS after this revision

Sample Crash Log:
https://build.webkit.org/builders/Apple%20iOS%2011%20Simulator%20Release%20WK2%20%28Tests%29/builds/5969/steps/run-api-tests/logs/stdio

 TestWebKitAPI.WebKitLegacy.AudioSessionCategoryIOS
        2018-07-03 10:12:34.215 TestWebKitAPI[54158:225799115] *** Warning: <AVPlayerViewController: 0x7f9ef1804200> is trying to enter full screen, but is not in its view's window's view controller hierarchy. This results in undefined behavior.
        -[AVPlayerViewController exitFullScreenAnimated:completionHandler:] failed with error View needs to be loaded before you can call -[AVPlayerViewController (0x7f9ef1804200) exitFullScreenAnimated:completionHandler:]!
        -[AVPlayerViewController exitFullScreenAnimated:completionHandler:] failed with error View needs to be loaded before you can call -[AVPlayerViewController (0x7f9ef1804200) exitFullScreenAnimated:completionHandler:]!
        2018-07-03 10:12:34.351 TestWebKitAPI[54158:225799115] *** Warning: <AVPlayerViewController: 0x7f9ef7815000> is trying to enter full screen, but is not in its view's window's view controller hierarchy. This results in undefined behavior.
        -[AVPlayerViewController exitFullScreenAnimated:completionHandler:] failed with error View needs to be loaded before you can call -[AVPlayerViewController (0x7f9ef7815000) exitFullScreenAnimated:completionHandler:]!
        -[AVPlayerViewController exitFullScreenAnimated:completionHandler:] failed with error View needs to be loaded before you can call -[AVPlayerViewController (0x7f9ef7815000) exitFullScreenAnimated:completionHandler:]!
        2018-07-03 10:12:34.527 TestWebKitAPI[54158:225799115] *** Warning: <AVPlayerViewController: 0x7f9ef7014000> is trying to enter full screen, but is not in its view's window's view controller hierarchy. This results in undefined behavior.
        -[AVPlayerViewController exitFullScreenAnimated:completionHandler:] failed with error View needs to be loaded before you can call -[AVPlayerViewController (0x7f9ef7014000) exitFullScreenAnimated:completionHandler:]!
        -[AVPlayerViewController exitFullScreenAnimated:completionHandler:] failed with error View needs to be loaded before you can call -[AVPlayerViewController (0x7f9ef7014000) exitFullScreenAnimated:completionHandler:]!
        >>>> frameSizeChanged = 4096
        >>>> frameSizeChanged = 4096
        2018-07-03 10:12:34.713 TestWebKitAPI[54158:225799115] *** Warning: <AVPlayerViewController: 0x7f9ef7029600> is trying to enter full screen, but is not in its view's window's view controller hierarchy. This results in undefined behavior.
        -[AVPlayerViewController exitFullScreenAnimated:completionHandler:] failed with error View needs to be loaded before you can call -[AVPlayerViewController (0x7f9ef7029600) exitFullScreenAnimated:completionHandler:]!
        -[AVPlayerViewController exitFullScreenAnimated:completionHandler:] failed with error View needs to be loaded before you can call -[AVPlayerViewController (0x7f9ef7029600) exitFullScreenAnimated:completionHandler:]!
        2018-07-03 10:12:34.898 TestWebKitAPI[54158:225799115] *** Warning: <AVPlayerViewController: 0x7f9ef781d600> is trying to enter full screen, but is not in its view's window's view controller hierarchy. This results in undefined behavior.
        -[AVPlayerViewController exitFullScreenAnimated:completionHandler:] failed with error View needs to be loaded before you can call -[AVPlayerViewController (0x7f9ef781d600) exitFullScreenAnimated:completionHandler:]!
        
        /Volumes/Data/slave/ios-simulator-11-release/build/Tools/TestWebKitAPI/Tests/WebKitLegacy/ios/AudioSessionCategoryIOS.mm:87
        Value of: [[getAVAudioSessionClass() sharedInstance] category]
          Actual: "AVAudioSessionCategoryPlayback"
        Expected: getAVAudioSessionCategoryAmbient()
        Which is: "AVAudioSessionCategoryAmbient"
        
        
        /Volumes/Data/slave/ios-simulator-11-release/build/Tools/TestWebKitAPI/Tests/WebKitLegacy/ios/AudioSessionCategoryIOS.mm:95
        Value of: [[getAVAudioSessionClass() sharedInstance] category]
          Actual: "AVAudioSessionCategoryPlayback"
        Expected: getAVAudioSessionCategoryAmbient()
        Which is: "AVAudioSessionCategoryAmbient"
        
        
        /Volumes/Data/slave/ios-simulator-11-release/build/Tools/TestWebKitAPI/Tests/WebKitLegacy/ios/AudioSessionCategoryIOS.mm:103
        Value of: [[getAVAudioSessionClass() sharedInstance] category]
          Actual: "AVAudioSessionCategoryPlayback"
        Expected: getAVAudioSessionCategoryAmbient()
        Which is: "AVAudioSessionCategoryAmbient"
        
        
        /Volumes/Data/slave/ios-simulator-11-release/build/Tools/TestWebKitAPI/Tests/WebKitLegacy/ios/AudioSessionCategoryIOS.mm:111
        Value of: [[getAVAudioSessionClass() sharedInstance] category]
          Actual: "AVAudioSessionCategoryPlayback"
        Expected: getAVAudioSessionCategoryAmbient()
        Which is: "AVAudioSessionCategoryAmbient"

[Eric Carlson]

(In reply to David Fenton (:realdawei) from comment #12)
> (In reply to WebKit Commit Bot from comment #11)
> > Comment on attachment 344122 [details]
> > Patch for landing
> > 
> > Clearing flags on attachment: 344122
> > 
> > Committed r233435: <https://trac.webkit.org/changeset/233435>
> 
> Seeing crashes in iOS after this revision
> 
> Sample Crash Log:
> https://build.webkit.org/builders/
> Apple%20iOS%2011%20Simulator%20Release%20WK2%20%28Tests%29/builds/5969/steps/
> run-api-tests/logs/stdio
> 
  This log shows that TestWebKitAPI.WebKitLegacy.AudioSessionCategoryIOS is failing, but I don't see any indication of a crash.

  I will fix the failing test.

[Dawei Fenton (:realdawei)]

(In reply to Eric Carlson from comment #13)
> (In reply to David Fenton (:realdawei) from comment #12)
> > (In reply to WebKit Commit Bot from comment #11)
> > > Comment on attachment 344122 [details]
> > > Patch for landing
> > > 
> > > Clearing flags on attachment: 344122
> > > 
> > > Committed r233435: <https://trac.webkit.org/changeset/233435>
> > 
> > Seeing crashes in iOS after this revision
> > 
> > Sample Crash Log:
> > https://build.webkit.org/builders/
> > Apple%20iOS%2011%20Simulator%20Release%20WK2%20%28Tests%29/builds/5969/steps/
> > run-api-tests/logs/stdio
> > 
>   This log shows that TestWebKitAPI.WebKitLegacy.AudioSessionCategoryIOS is
> failing, but I don't see any indication of a crash.
> 
>   I will fix the failing test.

Thanks! sorry about that I mislabeled it as "crash"

[Eric Carlson]

Created attachment 344208 [details]
Fix broken test.

[WebKit Commit Bot]

Comment on attachment 344208 [details]
Fix broken test.

Clearing flags on attachment: 344208

Committed r233478: <https://trac.webkit.org/changeset/233478>

[Dawei Fenton (:realdawei)]

(In reply to WebKit Commit Bot from comment #16)
> Comment on attachment 344208 [details]
> Fix broken test.
> 
> Clearing flags on attachment: 344208
> 
> Committed r233478: <https://trac.webkit.org/changeset/233478>

still failing after this revision:

https://build.webkit.org/builders/Apple%20iOS%2011%20Simulator%20Release%20WK2%20%28Tests%29/builds/5980/steps/run-api-tests/logs/stdio


Failed

    TestWebKitAPI.WebKitLegacy.AudioSessionCategoryIOS
        2018-07-03 15:32:45.284 TestWebKitAPI[48251:227456308] *** Warning: <AVPlayerViewController: 0x7ffbe480f600> is trying to enter full screen, but is not in its view's window's view controller hierarchy. This results in undefined behavior.
        -[AVPlayerViewController exitFullScreenAnimated:completionHandler:] failed with error View needs to be loaded before you can call -[AVPlayerViewController (0x7ffbe480f600) exitFullScreenAnimated:completionHandler:]!
        -[AVPlayerViewController exitFullScreenAnimated:completionHandler:] failed with error View needs to be loaded before you can call -[AVPlayerViewController (0x7ffbe480f600) exitFullScreenAnimated:completionHandler:]!
        2018-07-03 15:32:45.416 TestWebKitAPI[48251:227456308] *** Warning: <AVPlayerViewController: 0x7ffbe880a200> is trying to enter full screen, but is not in its view's window's view controller hierarchy. This results in undefined behavior.
        -[AVPlayerViewController exitFullScreenAnimated:completionHandler:] failed with error View needs to be loaded before you can call -[AVPlayerViewController (0x7ffbe880a200) exitFullScreenAnimated:completionHandler:]!
        -[AVPlayerViewController exitFullScreenAnimated:completionHandler:] failed with error View needs to be loaded before you can call -[AVPlayerViewController (0x7ffbe880a200) exitFullScreenAnimated:completionHandler:]!
        2018-07-03 15:32:55.766 TestWebKitAPI[48251:227456308] *** Warning: <AVPlayerViewController: 0x7ffbe880ba00> is trying to enter full screen, but is not in its view's window's view controller hierarchy. This results in undefined behavior.
        -[AVPlayerViewController exitFullScreenAnimated:completionHandler:] failed with error View needs to be loaded before you can call -[AVPlayerViewController (0x7ffbe880ba00) exitFullScreenAnimated:completionHandler:]!
        -[AVPlayerViewController exitFullScreenAnimated:completionHandler:] failed with error View needs to be loaded before you can call -[AVPlayerViewController (0x7ffbe880ba00) exitFullScreenAnimated:completionHandler:]!
        >>>> frameSizeChanged = 4096
        >>>> frameSizeChanged = 4096
        2018-07-03 15:33:06.064 TestWebKitAPI[48251:227456308] *** Warning: <AVPlayerViewController: 0x7ffbe8819800> is trying to enter full screen, but is not in its view's window's view controller hierarchy. This results in undefined behavior.
        -[AVPlayerViewController exitFullScreenAnimated:completionHandler:] failed with error View needs to be loaded before you can call -[AVPlayerViewController (0x7ffbe8819800) exitFullScreenAnimated:completionHandler:]!
        -[AVPlayerViewController exitFullScreenAnimated:completionHandler:] failed with error View needs to be loaded before you can call -[AVPlayerViewController (0x7ffbe8819800) exitFullScreenAnimated:completionHandler:]!
        2018-07-03 15:33:06.377 TestWebKitAPI[48251:227456308] *** Warning: <AVPlayerViewController: 0x7ffbe3826e00> is trying to enter full screen, but is not in its view's window's view controller hierarchy. This results in undefined behavior.
        -[AVPlayerViewController exitFullScreenAnimated:completionHandler:] failed with error View needs to be loaded before you can call -[AVPlayerViewController (0x7ffbe3826e00) exitFullScreenAnimated:completionHandler:]!
        
        /Volumes/Data/slave/ios-simulator-11-release/build/Tools/TestWebKitAPI/Tests/WebKitLegacy/ios/AudioSessionCategoryIOS.mm:99
        Value of: [[getAVAudioSessionClass() sharedInstance] category]
          Actual: "AVAudioSessionCategoryPlayback"
        Expected: getAVAudioSessionCategoryAmbient()
        Which is: "AVAudioSessionCategoryAmbient"
        
        
        /Volumes/Data/slave/ios-simulator-11-release/build/Tools/TestWebKitAPI/Tests/WebKitLegacy/ios/AudioSessionCategoryIOS.mm:108
        Value of: [[getAVAudioSessionClass() sharedInstance] category]
          Actual: "AVAudioSessionCategoryPlayback"
        Expected: getAVAudioSessionCategoryAmbient()
        Which is: "AVAudioSessionCategoryAmbient"

[Eric Carlson]

Created attachment 344348 [details]
Fix the broken test

[WebKit Commit Bot]

Comment on attachment 344348 [details]
Fix the broken test

Clearing flags on attachment: 344348

Committed r233535: <https://trac.webkit.org/changeset/233535>