Summary: | Fix macros for wireless playback | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Jon Lee <jonlee> | ||||
Component: | Media | Assignee: | Jon Lee <jonlee> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | andersca, eric.carlson, jer.noble, mitz, webkit-bug-importer | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | All | ||||||
OS: | All | ||||||
Bug Depends on: | 145314, 145315, 145316, 145477 | ||||||
Bug Blocks: | |||||||
Attachments: |
|
Description
Jon Lee
2015-05-20 18:34:38 PDT
Created attachment 253764 [details]
Patch
Comment on attachment 253764 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=253764&action=review > Source/WebKit2/UIProcess/API/Cocoa/WKWebViewConfiguration.mm:350 > + return _allowsAirPlayForMediaPlayback; > +} > + > +- (void)setMediaPlaybackAllowsAirPlay:(BOOL)allowed > +{ > + _allowsAirPlayForMediaPlayback = allowed; > +} > + > +- (BOOL)mediaPlaybackRequiresUserAction > +{ > + return _requiresUserActionForMediaPlayback; > +} > + > +- (void)setMediaPlaybackRequiresUserAction:(BOOL)required > +{ > + _requiresUserActionForMediaPlayback = required; I think this should just call into the API methods instead of dealing with ivars directly. Comment on attachment 253764 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=253764&action=review >> Source/WebKit2/UIProcess/API/Cocoa/WKWebViewConfiguration.mm:350 >> + _requiresUserActionForMediaPlayback = required; > > I think this should just call into the API methods instead of dealing with ivars directly. Ok, I will make that adjustment when I check in the patch. Committed r184946: <http://trac.webkit.org/changeset/184946> |