RESOLVED FIXED185147
Eliminate WebProcessShim.dylib
https://bugs.webkit.org/show_bug.cgi?id=185147
Summary Eliminate WebProcessShim.dylib
Michael Saboff
Reported 2018-04-30 15:26:34 PDT
This shim library contains WebKit/Shared/mac/CookieStorageShimLibrary.cpp and SecItemShim code. The Cookie Storage Shim library is being removed in <https://bugs.webkit.org/show_bug.cgi?id=185146> "Remove unused mac/CookieStorageShimLibrary". This bug is to eliminate the creation and use of WebProcessShim.dylib as the WebContent process doesn't reference any of the APIs in the WebProcessShim library. The SecItemShim code is still referenced by the Network process, but that is done via SecItemShim.dylib.
Attachments
Patch (11.76 KB, patch)
2018-04-30 15:32 PDT, Michael Saboff
rniwa: review+
ews-watchlist: commit-queue-
Archive of layout-test-results from ews202 for win-future (12.83 MB, application/zip)
2018-04-30 17:42 PDT, EWS Watchlist
no flags
Michael Saboff
Comment 1 2018-04-30 15:32:31 PDT
Alexey Proskuryakov
Comment 2 2018-04-30 15:54:49 PDT
Comment on attachment 339155 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=339155&action=review > Source/WebKit/WebKit.xcodeproj/project.pbxproj:-11908 > - baseConfigurationReference = A1EDD2DD1884B9E300BBFE98 /* WebProcessShim.xcconfig */; The WebProcessShim.xcconfig file can be now removed too I think.
Michael Saboff
Comment 3 2018-04-30 17:16:39 PDT
(In reply to Alexey Proskuryakov from comment #2) > Comment on attachment 339155 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=339155&action=review > > > Source/WebKit/WebKit.xcodeproj/project.pbxproj:-11908 > > - baseConfigurationReference = A1EDD2DD1884B9E300BBFE98 /* WebProcessShim.xcconfig */; > > The WebProcessShim.xcconfig file can be now removed too I think. Yup. I'll do that before landing.
EWS Watchlist
Comment 4 2018-04-30 17:42:22 PDT
Comment on attachment 339155 [details] Patch Attachment 339155 [details] did not pass win-ews (win): Output: http://webkit-queues.webkit.org/results/7517050 New failing tests: http/tests/misc/resource-timing-resolution.html
EWS Watchlist
Comment 5 2018-04-30 17:42:34 PDT
Created attachment 339166 [details] Archive of layout-test-results from ews202 for win-future The attached test failures were seen while running run-webkit-tests on the win-ews. Bot: ews202 Port: win-future Platform: CYGWIN_NT-6.1-2.9.0-0.318-5-3-x86_64-64bit
Michael Saboff
Comment 6 2018-04-30 18:19:18 PDT
Radar WebKit Bug Importer
Comment 7 2018-04-30 18:21:18 PDT
Note You need to log in before you can comment on or make changes to this bug.