Bug 145414

Summary: Include AVPlayerViewController_WebKitOnly.h only on iOS and Mac
Product: WebKit Reporter: Daniel Bates <dbates>
Component: PlatformAssignee: Daniel Bates <dbates>
Status: RESOLVED INVALID    
Severity: Normal CC: ap, bshafiei, eric.carlson, jer.noble, jonlee
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=145289
Attachments:
Description Flags
Patch none

Daniel Bates
Reported 2015-05-27 14:51:22 PDT
It is unnecessary to include AVPlayerViewController_WebKitOnly.h from AVFoundationSPI.h.
Attachments
Patch (1.31 KB, patch)
2015-05-27 14:54 PDT, Daniel Bates 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Daniel Bates
Comment 1 2015-05-27 14:54:15 PDT
Created attachment 253802 [details] Patch
Daniel Bates
Comment 2 2015-05-27 15:13:28 PDT
After talking with Jer Noble in person today, future work will make use of AVPlayerViewController_WebKitOnly.h and hence we should conditionally include AVPlayerViewController_WebKitOnly.h when we have AVKit.
Jon Lee
Comment 3 2015-05-29 11:15:08 PDT
Reopening.
Jon Lee
Comment 4 2015-05-29 11:15:27 PDT
<rdar://problem/21149548>
Jon Lee
Comment 5 2015-05-29 11:20:24 PDT
Sorry I misunderstood what's going on. Back to invalid.
Note You need to log in before you can comment on or make changes to this bug.