Summary: | Add WebKitPrivateFrameworkStubs for iOS 11 | ||||||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Jonathan Bedard <jbedard> | ||||||||||||||||||||
Component: | Tools / Tests | Assignee: | Jonathan Bedard <jbedard> | ||||||||||||||||||||
Status: | RESOLVED FIXED | ||||||||||||||||||||||
Severity: | Normal | CC: | ap, buildbot, commit-queue, ddkilzer, keith_miller, lforschler, mark.lam, msaboff, rniwa, saam, webkit-bug-importer | ||||||||||||||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||||||||||||||
Version: | WebKit Nightly Build | ||||||||||||||||||||||
Hardware: | Unspecified | ||||||||||||||||||||||
OS: | Unspecified | ||||||||||||||||||||||
Attachments: |
|
Description
Jonathan Bedard
2017-06-29 13:38:01 PDT
Created attachment 314160 [details]
Patch
Created attachment 314162 [details]
Patch
Created attachment 314167 [details]
Manually edited patch
svn-create-patch is not handling deletion from directories, we have a bug tracking this. In the mean time, this manually curated patch should be correct.
Created attachment 314171 [details]
Manually edited patch
Comment on attachment 314171 [details] Manually edited patch Attachment 314171 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.webkit.org/results/4022165 New failing tests: media/video-no-audio.html Created attachment 314181 [details]
Archive of layout-test-results from ews107 for mac-elcapitan-wk2
The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews107 Port: mac-elcapitan-wk2 Platform: Mac OS X 10.11.6
Created attachment 314187 [details]
Patch
Created attachment 314268 [details]
Patch
Created attachment 314534 [details]
Patch
Comment on attachment 314534 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=314534&action=review r=me > Source/WebCore/Configurations/WebCore.xcconfig:33 > + Nit: Why add a blank line here? Should be consistent across all *.xcconfig files. > Source/WebCore/PAL/Configurations/PAL.xcconfig:32 > + Nit: Why add a blank line here? Should be consistent across all *.xcconfig files. > Source/WebKit2/Configurations/BaseTarget.xcconfig:30 > + Nit: Why add a blank line here? Should be consistent across all *.xcconfig files. > Source/WebKit/mac/Configurations/WebKitLegacy.xcconfig:37 > + Nit: Why add a blank line here? Should be consistent across all *.xcconfig files. > WebKitLibraries/WebKitPrivateFrameworkStubs/iOS/11/AppSupport.framework/AppSupport.tbd:15 > +exports: > + - > + archs: > + - armv7 > + - armv7s > + - arm64 > + - i386 > + - x86_64 Weird indention here for x86_64. Does it matter? Created attachment 314616 [details]
Patch for landing
Comment on attachment 314616 [details] Patch for landing Clearing flags on attachment: 314616 Committed r219140: <http://trac.webkit.org/changeset/219140> All reviewed patches have been landed. Closing bug. |