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

Issue 1517033004: Revert of Roll third_party/freetype-android/src to VER-2-6-2. (Closed)

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

Description

Revert of Roll third_party/freetype-android/src to VER-2-6-2. (patchset #2 id:20001 of https://codereview.chromium.org/1518643003/ ) Reason for revert: This is still breaking the WebKit Android (Nexus4) bot: https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Android%20%28Nexus4%29/builds/42306 Failing tests (in case link goes dead): webkit_tests unexpected_failures: css1/classification/list_style.html svg/W3C-SVG-1.1-SE/coords-dom-02-f.svg css1/formatting_model/horizontal_formatting.html css1/formatting_model/canvas.html css1/cascade/important.html css1/font_properties/font_style.html css1/font_properties/font_variant.html css1/color_and_background/background_color.html css1/classification/list_style_position.html svg/batik/text/textAnchor3.svg fast/text/chromium-linux-fontconfig-renderstyle.html svg/canvas/canvas-pattern-svg.html css1/color_and_background/color.html fast/hidpi/clip-text-in-hidpi.html editing/caret/caret-color.html css1/classification/list_style_image.html css1/text_properties/text_align.html fast/invalid/009.html fast/compact/002.html Original issue's description: > Roll third_party/freetype-android/src to VER-2-6-2. > > https://chromium.googlesource.com/chromium/src/third_party/freetype2.git/+log/30fe5e762e06261..66cf29b1bc869b > > This updates from VER-2-6-1 + 1 to VER-2-6-2. > > It is expected that this will change the weight of CFF fonts. > If this becomes undesireable, there is a patch which can be applied. > > BUG=chromium:551028 > > Committed: https://crrev.com/a7a845a523c9f2f3c481ec99c0c35646f9e5ea7f > Cr-Commit-Position: refs/heads/master@{#364438} > > Committed: https://crrev.com/89e75f4d6c033ba5904cc808feb1292e8f211120 > Cr-Commit-Position: refs/heads/master@{#364723} TBR=jshin@chromium.org,wangxianzhu@chromium.org,bungeman@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=chromium:551028 Committed: https://crrev.com/08d4b38ff9d09a22b439eb2d4a5a9921767ec986 Cr-Commit-Position: refs/heads/master@{#364769}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+10 lines, -50 lines) Patch
M DEPS View 1 chunk +1 line, -1 line 0 comments Download
M third_party/WebKit/LayoutTests/TestExpectations View 1 chunk +0 lines, -20 lines 0 comments Download
M third_party/freetype-android/README.chromium View 2 chunks +4 lines, -6 lines 0 comments Download
M third_party/freetype-android/include/freetype-android-config/ftoption.h View 4 chunks +5 lines, -23 lines 0 comments Download

Messages

Total messages: 6 (2 generated)
nyquist
Created Revert of Roll third_party/freetype-android/src to VER-2-6-2.
5 years ago (2015-12-11 19:14:57 UTC) #1
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1517033004/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1517033004/1
5 years ago (2015-12-11 19:16:09 UTC) #2
commit-bot: I haz the power
Committed patchset #1 (id:1)
5 years ago (2015-12-11 19:17:57 UTC) #4
commit-bot: I haz the power
5 years ago (2015-12-11 19:18:59 UTC) #6
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/08d4b38ff9d09a22b439eb2d4a5a9921767ec986
Cr-Commit-Position: refs/heads/master@{#364769}

Powered by Google App Engine
This is Rietveld 408576698