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

Description Daniel Bates 2015-05-27 14:51:22 PDT
It is unnecessary to include AVPlayerViewController_WebKitOnly.h from AVFoundationSPI.h.
Comment 1 Daniel Bates 2015-05-27 14:54:15 PDT
Created attachment 253802 [details]
Patch
Comment 2 Daniel Bates 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.
Comment 3 Jon Lee 2015-05-29 11:15:08 PDT
Reopening.
Comment 4 Jon Lee 2015-05-29 11:15:27 PDT
<rdar://problem/21149548>
Comment 5 Jon Lee 2015-05-29 11:20:24 PDT
Sorry I misunderstood what's going on. Back to invalid.