RESOLVED FIXED 173877
[GTK][WPE] Implement API::IconLoadingClient and rework WebKitFaviconDatabase to use IconDatabase directly 
https://bugs.webkit.org/show_bug.cgi?id=173877
Summary [GTK][WPE] Implement API::IconLoadingClient and rework WebKitFaviconDatabase ...
Carlos Garcia Campos
Reported 2017-06-27 09:09:51 PDT
This way IconDatabase and its related classes can be removed from WebCore, as well as the WebKit2 implementation and C API.
Attachments
Patch (125.68 KB, patch)
2017-06-27 09:15 PDT, Carlos Garcia Campos 		no flags
DetailsFormatted DiffDiff
Patch (126.60 KB, patch)
2017-06-27 09:39 PDT, Carlos Garcia Campos 		beidson: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Carlos Garcia Campos
Comment 1 2017-06-27 09:15:02 PDT
Created attachment 313916 [details] Patch
Carlos Garcia Campos
Comment 2 2017-06-27 09:39:14 PDT
Created attachment 313919 [details] Patch Fix WPE build, I forgot to update the WPE makefile. We should move common glib api files to a shared makefile at some point.
Brady Eidson
Comment 3 2017-06-27 13:14:56 PDT
Comment on attachment 313919 [details] Patch lgtm
Build Bot
Comment 4 2017-06-27 13:32:31 PDT
Attachment 313919 [details] did not pass style-queue: ERROR: Source/WebKit2/UIProcess/API/glib/WebKitIconLoadingClient.cpp:36: Extra space before ( in function call [whitespace/parens] [4] Total errors found: 1 in 14 files If any of these errors are false positives, please file a bug against check-webkit-style.
Carlos Garcia Campos
Comment 5 2017-06-27 23:04:35 PDT
(In reply to Brady Eidson from comment #3) > Comment on attachment 313919 [details] > Patch > > lgtm Thanks, could you also review bug #173874 that is blocking this one?
Carlos Garcia Campos
Comment 6 2017-06-29 03:00:36 PDT
Committed r218922: <http://trac.webkit.org/changeset/218922>
Note You need to log in before you can comment on or make changes to this bug.