We have removed networking access from the WebContent process (see Bug 183192 and related), and no longer need the UIProcess to ask the WebContent process about networking-related things. This bug removes some left-over WebCookieStore API stuff that still asks the WebContent process about things it no longer knows anything about.
<rdar://problem/38191450>
Created attachment 335163 [details] Patch
Created attachment 335164 [details] Patch
Created attachment 335166 [details] Patch
Created attachment 335196 [details] Patch
Comment on attachment 335196 [details] Patch r=me don't break gtk
Comment on attachment 335196 [details] Patch Attachment 335196 [details] did not pass win-ews (win): Output: http://webkit-queues.webkit.org/results/6841105 New failing tests: http/wpt/resource-timing/rt-initiatorType-media.html
Created attachment 335199 [details] Archive of layout-test-results from ews204 for win-future The attached test failures were seen while running run-webkit-tests on the win-ews. Bot: ews204 Port: win-future Platform: CYGWIN_NT-6.1-2.9.0-0.318-5-3-x86_64-64bit
Created attachment 335239 [details] Patch
Created attachment 335298 [details] Patch for landing
Comment on attachment 335298 [details] Patch for landing Rejecting attachment 335298 [details] from commit-queue. Number of test failures exceeded the failure limit. Full output: http://webkit-queues.webkit.org/results/6857355
Created attachment 335302 [details] Archive of layout-test-results from webkit-cq-02 for mac-sierra The attached test failures were seen while running run-webkit-tests on the commit-queue. Bot: webkit-cq-02 Port: mac-sierra Platform: Mac OS X 10.12.6
(In reply to WebKit Commit Bot from comment #12) > Created attachment 335302 [details] > Archive of layout-test-results from webkit-cq-02 for mac-sierra > > The attached test failures were seen while running run-webkit-tests on the > commit-queue. > Bot: webkit-cq-02 Port: mac-sierra Platform: Mac OS X 10.12.6 These failures do not reproduce locally, and don't seem to be related to dormant cookie code in the WebContent process.
Committed r229426: <https://trac.webkit.org/changeset/229426>
(In reply to Brent Fulgham from comment #14) > Committed r229426: <https://trac.webkit.org/changeset/229426> It seems this has broken both GTK and WPE builds: https://build.webkit.org/builders/WPE%20Linux%2064-bit%20Release%20%28Build%29/builds/9314
(In reply to Carlos Alberto Lopez Perez from comment #15) > (In reply to Brent Fulgham from comment #14) > > Committed r229426: <https://trac.webkit.org/changeset/229426> > > It seems this has broken both GTK and WPE builds: > https://build.webkit.org/builders/WPE%20Linux%2064- > bit%20Release%20%28Build%29/builds/9314 Yes, I've been trying for two days on IRC to get a GTK person to take a look. Can anyone help fix the GTK build?
WPE (and GTK) Failure looks like this: [668/857] Linking CXX executable bin/WPENetworkProcess FAILED: bin/WPENetworkProcess : && /usr/lib/ccache/c++ -fdiagnostics-color=always -Wno-attributes -Wno-maybe-uninitialized -Wwrite-strings -Wundef -Wpointer-arith -Wmissing-format-attribute -Wformat-security -Wcast-align -Wextra -Wall -fno-strict-aliasing -fno-exceptions -std=c++14 -fno-rtti -O3 -DNDEBUG -L/home/buildbot/wpe/wpe-linux-64-release/build/WebKitBuild/DependenciesWPE/Root/lib -fuse-ld=gold -Wl,--disable-new-dtags Source/WebKit/CMakeFiles/NetworkProcess.dir/NetworkProcess/EntryPoint/unix/NetworkProcessMain.cpp.o -o bin/WPENetworkProcess -Wl,-rpath,/home/buildbot/wpe/wpe-linux-64-release/build/WebKitBuild/Release/lib: -rdynamic lib/libWPEWebKit.so.1.0.0 lib/libWebCore.a lib/libPAL.a lib/libJavaScriptCore.a lib/libWTF.a lib/libbmalloc.a -ldl ../DependenciesWPE/Root/lib/libWPEBackend.so && : lib/libWPEWebKit.so.1.0.0: error: undefined reference to 'non-virtual thunk to WebKit::WebCookieManager::didReceiveMessage(IPC::Connection&, IPC::Decoder&)' lib/libWPEWebKit.so.1.0.0: error: undefined reference to 'WebKit::WebCookieManager::didReceiveMessage(IPC::Connection&, IPC::Decoder&)' The WebCookieManager message handling doesn't seem to be getting generated, even though it appears in the CMakeLists.txt and SourcesWPE.txt/SourcesGTK.txt files.
Committed r229442: <https://trac.webkit.org/changeset/229442>
(In reply to Brent Fulgham from comment #16) > Yes, I've been trying for two days on IRC to get a GTK person to take a > look. Can anyone help fix the GTK build? Problem is that most of our team is only available on IRC during European business hours. I'm the exception, but I'm IP banned from freenode since I use a proxy service. Probably the easiest way to get our attention is to just leave a comment here on Bugzilla and CC us, either individually, or better our bugs-noreply@webkitgtk.org email.