Add a build flag for FontLoader support (disabled by default).
Created attachment 191166 [details] Patch
What is Font Loader? The name is a little generic, we already load custom fonts :)
We are implementing the font load events spec'd in the CSS Font Load Events Module Level 3 [1]. FontLoader is the name of the interface which provides ways to detect font loading. Please see the tracking bug [2] for details. [1] http://dev.w3.org/csswg/css-font-load-events-3/ [2] https://bugs.webkit.org/show_bug.cgi?id=98395
(In reply to comment #3) > We are implementing the font load events spec'd in the CSS Font Load Events Module Level 3 [1]. > FontLoader is the name of the interface which provides ways to detect font loading. > Please see the tracking bug [2] for details. > > [1] http://dev.w3.org/csswg/css-font-load-events-3/ > [2] https://bugs.webkit.org/show_bug.cgi?id=98395 Then, in my opinion, the flag should be named ENABLE_FONT_LOAD_EVENTS. This spec looks like a nice idea but...you should start by announcing it on webkit-dev. See http://trac.webkit.org/wiki/AddingFeatures
https://lists.webkit.org/pipermail/webkit-dev/2013-January/023541.html was the announcement :)
(In reply to comment #5) > https://lists.webkit.org/pipermail/webkit-dev/2013-January/023541.html was the announcement :) Oops, sorry. I completely forgot about that. (in my defense, that was a month ago :) )
Created attachment 191172 [details] Patch2, renamed it to ENABLE_FONT_LOAD_EVENTS
Comment on attachment 191172 [details] Patch2, renamed it to ENABLE_FONT_LOAD_EVENTS View in context: https://bugs.webkit.org/attachment.cgi?id=191172&action=review > Source/cmake/WebKitFeatures.cmake:53 > + WEBKIT_OPTION_DEFINE(ENABLE_FONT_LOAD_EVENTS "Toggle FontLoader support" OFF) "Toggle Font Load Events support"? > Tools/Scripts/webkitperl/FeatureList.pm:262 > + { option => "font-loader", desc => "Toggle FontLoader support", ditto
Created attachment 191178 [details] Patch3, fixed flag descriptions
Comment on attachment 191178 [details] Patch3, fixed flag descriptions LGTM
Thank you for the review!
Comment on attachment 191178 [details] Patch3, fixed flag descriptions Clearing flags on attachment: 191178 Committed r144614: <http://trac.webkit.org/changeset/144614>
All reviewed patches have been landed. Closing bug.