Bug 185706 - [iOS] Update AirPlay route monitoring
Summary: [iOS] Update AirPlay route monitoring
Status: ASSIGNED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Media (show other bugs)
Version: Other
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Eric Carlson
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2018-05-16 17:26 PDT by Eric Carlson
Modified: 2018-05-17 14:48 PDT (History)
3 users (show)

See Also:


Attachments
Patch (15.60 KB, patch)
2018-05-17 09:23 PDT, Eric Carlson
jer.noble: review+
Details | Formatted Diff | Diff
Patch for landing (15.83 KB, patch)
2018-05-17 11:26 PDT, Eric Carlson
no flags Details | Formatted Diff | Diff
Build fix. (3.86 KB, patch)
2018-05-17 14:10 PDT, Eric Carlson
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Eric Carlson 2018-05-16 17:26:46 PDT
Update the API used to monitor AirPlay routes.
Comment 1 Eric Carlson 2018-05-16 17:28:00 PDT
<rdar://problem/40230677>
Comment 2 Eric Carlson 2018-05-17 09:23:46 PDT
Created attachment 340591 [details]
Patch
Comment 3 Jer Noble 2018-05-17 10:27:42 PDT
Comment on attachment 340591 [details]
Patch

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

> Source/WebCore/platform/audio/ios/MediaSessionManagerIOS.mm:463
> +        if (!protectedSelf->_callback)
>              return;
>  
> -        _callback->applicationWillEnterForeground(isSuspendedUnderLock);
> +        protectedSelf->_callback->applicationWillEnterForeground(isSuspendedUnderLock);

Nit: this and the next 4 instances of this pattern could have a reversed if case:

if (auto* callback = protectedSelf->callback)
    callback->applicationWillEnterForeground(isSuspendedUnderLock);
Comment 4 Eric Carlson 2018-05-17 11:26:56 PDT
Created attachment 340610 [details]
Patch for landing
Comment 5 WebKit Commit Bot 2018-05-17 11:44:38 PDT
Comment on attachment 340610 [details]
Patch for landing

Clearing flags on attachment: 340610

Committed r231913: <https://trac.webkit.org/changeset/231913>
Comment 6 Eric Carlson 2018-05-17 14:10:05 PDT
Created attachment 340638 [details]
Build fix.
Comment 7 WebKit Commit Bot 2018-05-17 14:48:54 PDT
Comment on attachment 340638 [details]
Build fix.

Clearing flags on attachment: 340638

Committed r231921: <https://trac.webkit.org/changeset/231921>