79756 2012-02-27 23:56:05 -0800 [GTK] css3/flexbox/inline-flexbox.html fails 2025-02-13 05:49:34 -0800 1 1 1 Unclassified WebKit WebKitGTK 528+ (Nightly build) Unspecified Unspecified NEW P2 Normal --- 1 pnormand webkit-unassigned bugs-noreply deleted_user mrobinson zimmermann oldest_to_newest 566437 0 pnormand 2012-02-27 23:56:05 -0800 Since it's been added, it seems. http://build.webkit.org/results/GTK%20Linux%2064-bit%20Release/r109083%20(18907)/css3/flexbox/inline-flexbox-diffs.html I'll skip it for now. 566859 1 ojan 2012-02-28 10:47:48 -0800 Looks like it's only failing on the GTK bots. Looking at the toggling expected/actual image result at http://build.webkit.org/results/GTK%20Linux%2032-bit%20Release/r109021%20(21857)/results.html it appears to be a text kerning bug in the GTK port. 566946 2 mrobinson 2012-02-28 11:59:43 -0800 Leaving my comments here as well as #webkit, just in case ojan is away. Assuming that the "actual image" output is the flexbox example, it seems that flexbox is somehow triggering the complex text rendering path. I say this because the kerning errors that Ojan describes are actually ligatures. When complex text is active an 'f' character and an 'l' character combine into one ligature. It makes sense that text rendered via the complex text path and the simple text path look different. I'm surprised it passes on other platforms. Unless of course, this bug only occurs on GTK+. An easy fix for this is to force the reference HTML to use complex text rendering, but perhaps it's a bug that flexbox text is rendered via the complex text path. 567167 3 mrobinson 2012-02-28 15:33:02 -0800 This change seems to fix this test and several others and to produce much more consistent results. I'm afraid we should probably revert this change and rebaseline one more time. --- a/Tools/DumpRenderTree/gtk/DumpRenderTree.cpp +++ b/Tools/DumpRenderTree/gtk/DumpRenderTree.cpp @@ -161,11 +161,11 @@ static void initializeGtkFontSettings(const char* testURL) ASSERT(screen); const cairo_font_options_t* screenOptions = gdk_screen_get_font_options(screen); ASSERT(screenOptions); - cairo_font_options_t* options = cairo_font_options_copy(screenOptions); - // Turn off text metrics hinting, which quantizes metrics to pixels in device space. - cairo_font_options_set_hint_metrics(options, CAIRO_HINT_METRICS_OFF); - gdk_screen_set_font_options(screen, options); - cairo_font_options_destroy(options); + //cairo_font_options_t* options = cairo_font_options_copy(screenOptions); + //// Turn off text metrics hinting, which quantizes metrics to pixels in device space. + //cairo_font_options_set_hint_metrics(options, CAIRO_HINT_METRICS_OFF); + //gdk_screen_set_font_options(screen, options); + //cairo_font_options_destroy(options); } 567173 4 mrobinson 2012-02-28 15:36:18 -0800 Ccing Nikolas Zimmmermann who might be able to provide more context here. 2094444 5 deleted_user 2025-02-13 05:49:34 -0800 <inline-flexbox.html> doesn't exist anymore: https://wpt.fyi/results/css?label=master&product=chrome%5Bexperimental%5D&product=edge%5Bexperimental%5D&product=firefox%5Bexperimental%5D&product=safari%5Bexperimental%5D&product=webkitgtk&aligned&q=inline-flexbox