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

Issue 1474673003: Mark AlwaysUseComplexText as stable (Closed)

Created:
5 years ago by eae
Modified:
5 years ago
CC:
blink-reviews, chromium-reviews
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Mark AlwaysUseComplexText as stable Turn on the AlwaysUseComplexText runtime feature by default, thus always using the complex text path. It's been experimental for quite awhile now without any major problems. Flipping the switch to turn it on by default now allows maximum amount of time before the next release branch is cut. BUG=404597 R=leviw@chromium.org Committed: https://crrev.com/fa7fc32c5940dfd3d734ed3231b1295da4c3303e Cr-Commit-Position: refs/heads/master@{#361474}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M third_party/WebKit/Source/platform/RuntimeEnabledFeatures.in View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 9 (4 generated)
leviw_travelin_and_unemployed
F-YEAH! thisPatch = LGTM * LayoutUnit::max();
5 years ago (2015-11-24 21:26:41 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1474673003/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1474673003/1
5 years ago (2015-11-24 21:27:07 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1474673003/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1474673003/1
5 years ago (2015-11-24 21:29:55 UTC) #7
commit-bot: I haz the power
Committed patchset #1 (id:1)
5 years ago (2015-11-24 22:39:35 UTC) #8
commit-bot: I haz the power
5 years ago (2015-11-24 22:40:36 UTC) #9
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/fa7fc32c5940dfd3d734ed3231b1295da4c3303e
Cr-Commit-Position: refs/heads/master@{#361474}

Powered by Google App Engine
This is Rietveld 408576698