Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(156)

Issue 1283923002: Add Unicode Variation Selector support on Linux official builds (Closed)

Created:
5 years, 4 months ago by kojii
Modified:
5 years, 4 months ago
Reviewers:
drott, eae
CC:
blink-reviews, krit, drott+blinkwatch_chromium.org, Rik, dshwang, jbroman, Justin Novosad, danakj, pdr+graphicswatchlist_chromium.org, f(malita), Stephen Chennney, rwlbuis
Base URL:
https://chromium.googlesource.com/chromium/blink.git@master
Target Ref:
refs/heads/master
Project:
blink
Visibility:
Public.

Description

Add Unicode Variation Selector support on Linux official builds Most of the work to support Unicode Variation Selector was done in a separate CL[1], but it was not enabled for Linux due to crbug.com/462689. This patch enables the Unicode Variation Selector support on Linux official builds. The CL to enable on all Linux[2] was reverted because chromeos=1 is still bound to the harfbuzz in the system and thus unable to use hb_ot_font_set_funcs() on Trusty. [1] https://codereview.chromium.org/1149693002/ [2] https://codereview.chromium.org/1209043010/ BUG=383580 Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=200427

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -3 lines) Patch
M Source/platform/fonts/shaping/HarfBuzzFace.cpp View 1 chunk +4 lines, -3 lines 0 comments Download

Messages

Total messages: 10 (2 generated)
kojii
PTAL. Another try, just for OFFICIAL_BUILD. It does not help devs to use bundled HB, ...
5 years, 4 months ago (2015-08-12 05:15:43 UTC) #2
eae
LGTM Thanks for sticking with this problem!
5 years, 4 months ago (2015-08-12 20:24:00 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1283923002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1283923002/1
5 years, 4 months ago (2015-08-12 20:24:11 UTC) #5
commit-bot: I haz the power
Committed patchset #1 (id:1) as https://src.chromium.org/viewvc/blink?view=rev&revision=200427
5 years, 4 months ago (2015-08-12 21:22:03 UTC) #6
fukino
On 2015/08/12 21:22:03, commit-bot: I haz the power wrote: > Committed patchset #1 (id:1) as ...
5 years, 4 months ago (2015-08-13 03:08:39 UTC) #7
kojii
On 2015/08/13 03:08:39, fukino wrote: > On 2015/08/12 21:22:03, commit-bot: I haz the power wrote: ...
5 years, 4 months ago (2015-08-13 03:29:40 UTC) #8
rsadam
On 2015/08/13 03:29:40, kojii wrote: > On 2015/08/13 03:08:39, fukino wrote: > > On 2015/08/12 ...
5 years, 4 months ago (2015-08-18 19:42:30 UTC) #9
rsadam
5 years, 4 months ago (2015-08-18 19:45:16 UTC) #10
Message was sent while issue was closed.
A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/1292583006/ by rsadam@chromium.org.

The reason for reverting is: This CL prevents building official Chrome images on
Linux.

../../third_party/WebKit/Source/platform/fonts/shaping/HarfBuzzFace.cpp:176:13:
error: use of undeclared identifier 'hb_ot_font_set_funcs'

The proposed workaround doesn't work, so reverting this CL..

Powered by Google App Engine
This is Rietveld 408576698