Bug 207056

Summary: [macOS] AirPlay sometimes stops after 60 minutes of playback
Product: WebKit Reporter: Eric Carlson <eric.carlson>
Component: MediaAssignee: Eric Carlson <eric.carlson>
Status: ASSIGNED ---    
Severity: Normal CC: commit-queue, ews-watchlist, glenn, jer.noble, jonlee, philipj, sergio, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: Other   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Updated patch.
none
Patch
jer.noble: review+
Patch for landing.
commit-queue: commit-queue-
Patch for landing. none

Description Eric Carlson 2020-01-31 10:25:25 PST 
AirPlay sometimes stops after 60 minutes of playback

<rdar://problem/53649508>
Comment 1 Eric Carlson 2020-01-31 11:08:20 PST 
Created attachment 389381 [details]
Patch
Comment 2 Jon Lee 2020-01-31 11:39:59 PST 
Comment on attachment 389381 [details]
Patch

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

> Source/WebCore/ChangeLog:15
> +        AirPlay can't actually be active is an item doesn't have audio or video, so always claim

if* an item

> Source/WebCore/ChangeLog:16
> +        to have both during AirPlay.

any side effects if the video has no audio tracks?
Comment 3 Eric Carlson 2020-01-31 13:45:59 PST 
Comment on attachment 389381 [details]
Patch

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

>> Source/WebCore/ChangeLog:15
>> +        AirPlay can't actually be active is an item doesn't have audio or video, so always claim
> 
> if* an item

Oops!

>> Source/WebCore/ChangeLog:16
>> +        to have both during AirPlay.
> 
> any side effects if the video has no audio tracks?

No.
Comment 4 Eric Carlson 2020-01-31 13:46:35 PST 
Created attachment 389407 [details]
Updated patch.
Comment 5 Eric Carlson 2020-02-03 08:48:47 PST 
Created attachment 389519 [details]
Patch
Comment 6 Eric Carlson 2020-02-03 09:51:39 PST 
Created attachment 389531 [details]
Patch for landing.
Comment 7 WebKit Commit Bot 2020-02-03 10:26:46 PST 
Comment on attachment 389531 [details]
Patch for landing.

Rejecting attachment 389531 [details] from commit-queue.

Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-01', 'validate-changelog', '--check-oops', '--non-interactive', 389531, '--port=mac']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit

ChangeLog entry in Source/WebKit/ChangeLog contains OOPS!.

Full output: https://webkit-queues.webkit.org/results/13316107
Comment 8 Eric Carlson 2020-02-03 11:34:58 PST 
Created attachment 389542 [details]
Patch for landing.
Comment 9 WebKit Commit Bot 2020-02-03 12:19:50 PST 
Comment on attachment 389542 [details]
Patch for landing.

Clearing flags on attachment: 389542

Committed r255581: <https://trac.webkit.org/changeset/255581>