REGRESSION?(r202466): http/tests/security/canvas-remote-read-remote-video-redirect.html failing on Sierra
<rdar://problem/27030025>
Created attachment 282173 [details] Patch
Created attachment 282176 [details] Patch
Comment on attachment 282176 [details] Patch LGTM. R=me.
Comment on attachment 282176 [details] Patch Clearing flags on attachment: 282176 Committed r202520: <http://trac.webkit.org/changeset/202520>
All reviewed patches have been landed. Closing bug.
Re-opened since this is blocked by bug 159185
(In reply to comment #7) > Re-opened since this is blocked by bug 159185 /Volumes/Data/slave/elcapitan-32bit-release/build/Source/WebCore/platform/network/cocoa/WebCoreNSURLSession.mm:74:17: error: property 'hasSingleSecurityOrigin' requires method 'hasSingleSecurityOrigin' to be defined - use @synthesize, @dynamic or provide a method implementation in this class implementation [-Werror,-Wobjc-property-implementation] https://build.webkit.org/builders/Apple%20El%20Capitan%20Release%20%2832-bit%20Build%29/builds/7249
Comment on attachment 282176 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=282176&action=review > Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm:2252 > + if ([session respondsToSelector:@selector(hasSingleSecurityOrigin)]) > + return session.hasSingleSecurityOrigin; El Capitan compilers didn't like this. Does it need to be [session hasSingleSecurityOrigin]?
(In reply to comment #9) > Comment on attachment 282176 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=282176&action=review > > > Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm:2252 > > + if ([session respondsToSelector:@selector(hasSingleSecurityOrigin)]) > > + return session.hasSingleSecurityOrigin; > > El Capitan compilers didn't like this. Does it need to be [session > hasSingleSecurityOrigin]? No, the error message is correct. It needs a @synthesize.
Committed r202546: <http://trac.webkit.org/changeset/202546>