145414 2015-05-27 14:51:22 -0700 Include AVPlayerViewController_WebKitOnly.h only on iOS and Mac 2015-05-29 11:20:32 -0700 1 1 1 Unclassified WebKit Platform 528+ (Nightly build) Unspecified Unspecified RESOLVED INVALID https://bugs.webkit.org/show_bug.cgi?id=145289 InRadar P2 Normal --- 1 dbates dbates ap bshafiei eric.carlson jer.noble jonlee oldest_to_newest 1097723 0 dbates 2015-05-27 14:51:22 -0700 It is unnecessary to include AVPlayerViewController_WebKitOnly.h from AVFoundationSPI.h. 1097726 1 253802 dbates 2015-05-27 14:54:15 -0700 Created attachment 253802 Patch 1097731 2 dbates 2015-05-27 15:13:28 -0700 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. 1098137 3 jonlee 2015-05-29 11:15:08 -0700 Reopening. 1098138 4 jonlee 2015-05-29 11:15:27 -0700 <rdar://problem/21149548> 1098139 5 jonlee 2015-05-29 11:20:24 -0700 Sorry I misunderstood what's going on. Back to invalid. 253802 2015-05-27 14:54:15 -0700 2015-05-27 15:15:20 -0700 Patch bug-145414-20150527145403.patch text/plain 1341 dbates U3VidmVyc2lvbiBSZXZpc2lvbjogMTg0OTE4CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggZDhiNmI5MjI5MjUxNTZh M2UxZjkxMGFiMGMxNzE3Y2Q2NjMzMmVhOS4uOTlhMTU1ZDI3ODQ1ZTkxNWUyY2Q2YmY5YzNlYWVm MzZjNjhlZGE5MiAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE0IEBACisyMDE1LTA1LTI3ICBEYW5p ZWwgQmF0ZXMgIDxkYWJhdGVzQGFwcGxlLmNvbT4KKworICAgICAgICBSZW1vdmUgaW5jbHVkZSBv ZiBBVlBsYXllclZpZXdDb250cm9sbGVyX1dlYktpdE9ubHkuaCBmcm9tIEFWRm91bmRhdGlvblNQ SS5oCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xNDU0 MTQKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBJdCBp cyB1bm5lY2Vzc2FyeSB0byBpbmNsdWRlIEFWUGxheWVyVmlld0NvbnRyb2xsZXJfV2ViS2l0T25s eS5oIGZyb20gQVZGb3VuZGF0aW9uU1BJLmguCisKKyAgICAgICAgKiBwbGF0Zm9ybS9zcGkvbWFj L0FWRm91bmRhdGlvblNQSS5oOgorCiAyMDE1LTA1LTI3ICBFcmljIENhcmxzb24gIDxlcmljLmNh cmxzb25AYXBwbGUuY29tPgogCiAgICAgICAgIFtNYWNdIHNob3J0LWNpcmN1aXQgTUlNRSB0eXBl IGxvb2t1cCB3aGVuIHBvc3NpYmxlCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9y bS9zcGkvbWFjL0FWRm91bmRhdGlvblNQSS5oIGIvU291cmNlL1dlYkNvcmUvcGxhdGZvcm0vc3Bp L21hYy9BVkZvdW5kYXRpb25TUEkuaAppbmRleCBmOGJjYWIzMjk1NDc1ODQwMjE3YjM0OTllZmQ3 OGU0NDNiZWI4ZmFkLi5mNDE5OTA1NGFkYmIyNGQ0NmYwNjc1MzgzOGU0OWZmMTViYjc3ZTZkIDEw MDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9zcGkvbWFjL0FWRm91bmRhdGlvblNQ SS5oCisrKyBiL1NvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL3NwaS9tYWMvQVZGb3VuZGF0aW9uU1BJ LmgKQEAgLTUzLDEwICs1Myw2IEBACiAKICNpZiBQTEFURk9STShJT1MpCiAKLSNpZiBVU0UoQVBQ TEVfSU5URVJOQUxfU0RLKQotI2ltcG9ydCA8QVZLaXQvQVZQbGF5ZXJWaWV3Q29udHJvbGxlcl9X ZWJLaXRPbmx5Lmg+Ci0jZW5kaWYKLQogI2ltcG9ydCA8QVZGb3VuZGF0aW9uL0FWUGxheWVyTGF5 ZXIuaD4KIAogQGludGVyZmFjZSBBVlBsYXllckxheWVyIChBVlBsYXllckxheWVyT3B0aW1pemVk RnVsbHNjcmVlbk1vZGVTdXBwb3J0UHJpdmF0ZSkK