Allow apps built before SDK with file URL XHR header restriction to work. In <https://trac.webkit.org/changeset/222795> (bug #177824) only Dashboard widgets were exempt from XHR header restrictions. As it turns out this broke Cardiogram and we subsequently exempted Cardiogram in the patch for bug #181693. Nike is another app that is affect by bug #177824. Let's exempt all Mac and iOS apps that were built with an SDK before the patch for bug bug #177824 to allow developers time to update their app(s).
<rdar://problem/36533447>
Created attachment 332977 [details] Patch
Comment on attachment 332977 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=332977&action=review > Source/WTF/wtf/spi/darwin/dyldSPI.h:46 > +#ifndef DYLD_MACOSX_VERSION_10_13_4 > +#define DYLD_MACOSX_VERSION_10_13_4 0x000A0D04 > +#endif I thought macOS only updated dyld version for a major release?
Comment on attachment 332977 [details] Patch r=me assuming that the check actually works. I think Wenson and I concluded that it didn't for clipboard work though...
Comment on attachment 332977 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=332977&action=review > Source/WebCore/ChangeLog:11 > + (WebCore::IOSApplication::isCardiogram): Deleted. You might wanna clarify that we already allow this legacy behavior on all iOS apps linked SDK before iOS 11.3 and refer to the revision in which you made that change.
(In reply to Ryosuke Niwa from comment #4) > Comment on attachment 332977 [details] > Patch > > r=me assuming that the check actually works. I think Wenson and I concluded > that it didn't for clipboard work though... (FYI: <rdar://problem/35484255> is what I had in mind).
Brent Fulgham and I talked about this bug today (02/05/2018). We feel it is sufficient to just exempt the Nike app (See bug #182491) for now. If we discover more fallout from <https://trac.webkit.org/changeset/222795> (bug #177824) then we can consider this patch again.
<rdar://problem/37240400>