Bug 176907

Summary: [FreeType] Complex text is enabled too often after r221909
Product: WebKit Reporter: Carlos Garcia Campos <cgarcia>
Component: PlatformAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: bugs-noreply, buildbot, csaavedra, mmaxfield, svillar, webkit-bug-importer
Priority: P2 Keywords: Gtk, InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch svillar: review+

Description Carlos Garcia Campos 2017-09-14 08:42:38 PDT
In r221909 we enabled complex text by default following the same cocoa ifdefs, but I forgot another ifdef in the cpp file. This is going to require another rebaseline fo 2200 tests. I hope it will be the last one.
Comment 1 Carlos Garcia Campos 2017-09-14 08:44:16 PDT
Created attachment 320769 [details]
Patch
Comment 2 Carlos Garcia Campos 2017-09-15 00:08:36 PDT
Committed r222077: <http://trac.webkit.org/changeset/222077>
Comment 3 Claudio Saavedra 2017-09-15 03:06:44 PDT
After this change there's massive WPE test failure in the bots. Not sure what's going on.. https://build.webkit.org/builders/WPE%20Linux%2064-bit%20Release%20%28Tests%29/builds/2820
Comment 4 Claudio Saavedra 2017-09-15 03:07:17 PDT
(In reply to Carlos Garcia Campos from comment #0)
> In r221909 we enabled complex text by default following the same cocoa
> ifdefs, but I forgot another ifdef in the cpp file. This is going to require
> another rebaseline fo 2200 tests. I hope it will be the last one.

> This is going to require another rebaseline fo 2200 tests. I hope it will be the last one.

Ah, sorry. I should read before commenting.
Comment 5 Radar WebKit Bug Importer 2017-09-27 12:36:33 PDT
<rdar://problem/34693592>