../../Source/WebKit2/UIProcess/API/gtk/WebKitLoaderClient.cpp:44:10: error: ‘void LoaderClient::didStartProvisionalLoadForFrame(WebKit::WebPageProxy&, WebKit::WebFrameProxy&, uint64_t, API::Object*)’ marked override, but does not override void didStartProvisionalLoadForFrame(WebPageProxy&, WebFrameProxy& frame, uint64_t /* navigationID */, API::Object* /* userData */) override ^ ../../Source/WebKit2/UIProcess/API/gtk/WebKitLoaderClient.cpp:51:10: error: ‘void LoaderClient::didReceiveServerRedirectForProvisionalLoadForFrame(WebKit::WebPageProxy&, WebKit::WebFrameProxy&, uint64_t, API::Object*)’ marked override, but does not override void didReceiveServerRedirectForProvisionalLoadForFrame(WebPageProxy&, WebFrameProxy& frame, uint64_t /* navigationID */, API::Object* /* userData */) override ^ ../../Source/WebKit2/UIProcess/API/gtk/WebKitLoaderClient.cpp:58:10: error: ‘void LoaderClient::didFailProvisionalLoadWithErrorForFrame(WebKit::WebPageProxy&, WebKit::WebFrameProxy&, uint64_t, const WebCore::ResourceError&, API::Object*)’ marked override, but does not override void didFailProvisionalLoadWithErrorForFrame(WebPageProxy&, WebFrameProxy& frame, uint64_t /* navigationID */, const ResourceError& resourceError, API::Object* /* userData */) override ^ ../../Source/WebKit2/UIProcess/API/gtk/WebKitLoaderClient.cpp:71:10: error: ‘void LoaderClient::didCommitLoadForFrame(WebKit::WebPageProxy&, WebKit::WebFrameProxy&, uint64_t, API::Object*)’ marked override, but does not override void didCommitLoadForFrame(WebPageProxy&, WebFrameProxy& frame, uint64_t /* navigationID */, API::Object* /* userData */) override ^ ../../Source/WebKit2/UIProcess/API/gtk/WebKitLoaderClient.cpp:78:10: error: ‘void LoaderClient::didFinishLoadForFrame(WebKit::WebPageProxy&, WebKit::WebFrameProxy&, uint64_t, API::Object*)’ marked override, but does not override void didFinishLoadForFrame(WebPageProxy&, WebFrameProxy& frame, uint64_t /* navigationID */, API::Object* /* userData */) override ^ ../../Source/WebKit2/UIProcess/API/gtk/WebKitLoaderClient.cpp:85:10: error: ‘void LoaderClient::didFailLoadWithErrorForFrame(WebKit::WebPageProxy&, WebKit::WebFrameProxy&, uint64_t, const WebCore::ResourceError&, API::Object*)’ marked override, but does not override void didFailLoadWithErrorForFrame(WebPageProxy&, WebFrameProxy& frame, uint64_t /* navigationID */, const ResourceError& resourceError, API::Object* /* userData */) override ^
Created attachment 244582 [details] Patch
Thanks for the patch. If this patch contains new public API please make sure it follows the guidelines for new WebKit2 GTK+ API. See http://trac.webkit.org/wiki/WebKitGTK/AddingNewWebKit2API
It is a wrong patch,, I have to look more,,
I have just landed a speculative fix for this one.
I haven't fixed this one though: Source/WebKit2/CMakeFiles/WebKit2.dir/UIProcess/API/gtk/WebKitUserContentManager.cpp.o -c ../../Source/WebKit2/UIProcess/API/gtk/WebKitUserContentManager.cpp ../../Source/WebKit2/UIProcess/API/gtk/WebKitUserContentManager.cpp: In constructor ‘_WebKitUserContentManagerPrivate::_WebKitUserContentManagerPrivate()’: ../../Source/WebKit2/UIProcess/API/gtk/WebKitUserContentManager.cpp:37:33: error: ‘create’ is not a member of ‘WebKit::WebUserContentControllerProxy’ : userContentController(WebUserContentControllerProxy::create()) ^ This was broken by https://bugs.webkit.org/show_bug.cgi?id=140414. I'll let you take a look at it as I don't have a GTK build.
For reference, here are my speculative GTK build fixes: https://trac.webkit.org/r178395 https://trac.webkit.org/r178408 https://trac.webkit.org/r178409
Created attachment 244585 [details] Patch
Comment on attachment 244585 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=244585&action=review > Source/WebKit2/ChangeLog:3 > + [GTK] Fix build after r178375 do you mean r178385?
Comment on attachment 244585 [details] Patch rs=me with fixing the changelog
(In reply to comment #8) > Comment on attachment 244585 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=244585&action=review > > > Source/WebKit2/ChangeLog:3 > > + [GTK] Fix build after r178375 > > do you mean r178385? Oops,, Yes , I mean r178385. Thanks,
Created attachment 244587 [details] Patch
Somebody else fixed by http://trac.webkit.org/changeset/178412
Could you coordinate in the future to avoid parallel working? The simplest way is to add the new bug url to the original bug.
Sorry for duplicated work. I didn't know that build fix was in progress.
Thank you all for the build fixes.
(In reply to comment #11) > Created attachment 244587 [details] > Patch Could you re-apply this patch to the trunk? I mis-readed the code. APIObject is refcounted, so it is correct to use adoptRef for WebUserContentManager.
(In reply to comment #16) > (In reply to comment #11) > > Created attachment 244587 [details] > > Patch > > Could you re-apply this patch to the trunk? > I mis-readed the code. APIObject is refcounted, so it is correct to use > adoptRef for WebUserContentManager. Shouldn't we add the create method again instead? I think it was removed only because mac no uses a different way to allocate the object but we still need it
Reopening to land the right fix
Comment on attachment 244587 [details] Patch Rejecting attachment 244587 [details] from commit-queue. Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.appspot.com', '--bot-id=webkit-cq-02', 'apply-attachment', '--no-update', '--non-interactive', 244587, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit Last 500 characters of output: orce']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit Parsed 2 diffs from patch file(s). patching file Source/WebKit2/ChangeLog Hunk #1 succeeded at 1 with fuzz 3. patching file Source/WebKit2/UIProcess/API/gtk/WebKitUserContentManager.cpp Hunk #1 FAILED at 34. 1 out of 1 hunk FAILED -- saving rejects to file Source/WebKit2/UIProcess/API/gtk/WebKitUserContentManager.cpp.rej Failed to run "[u'/Volumes/Data/EWS/WebKit/Tools/Scripts/svn-apply', '--force']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit Full output: http://webkit-queues.appspot.com/results/4847328368263168
Committed r178417: <http://trac.webkit.org/changeset/178417>