WebKit Bugzilla
New
Browse
Search+
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
185147
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-
Details
Formatted Diff
Diff
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
Details
View All
Add attachment
proposed patch, testcase, etc.
Michael Saboff
Comment 1
2018-04-30 15:32:31 PDT
Created
attachment 339155
[details]
Patch
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
Committed
r231190
: <
https://trac.webkit.org/changeset/231190
>
Radar WebKit Bug Importer
Comment 7
2018-04-30 18:21:18 PDT
<
rdar://problem/39857210
>
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug