Chromium Code Reviews
Help | Chromium Project | Gerrit Changes | Sign in
(61)

Issue 2300973002: NOT FOR LANDING Update FreeType to upstream 57a6733dcf7828fe3db9254edab33fda7c9f6a10. (Closed)

Created:
1 year, 3 months ago by drott
Modified:
8 months, 2 weeks ago
Reviewers:
bungeman-skia
CC:
chromium-reviews
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

NOT FOR LANDING Update FreeType to upstream 2.7.1 Update the local FreeType version that is statically linked for layout tests to 2.7.1. BUG=274030

Patch Set 1 #

Total comments: 12

Patch Set 2 : Addressing review comments, simplified top level config #

Patch Set 3 : Re-add BDF since fontconfig for cast_shell needs it #

Patch Set 4 : Update to latest on 12/22 including Variation fixes 7209110345206, 992e40e9d9afb #

Patch Set 5 : experimentally change rendering mode #

Patch Set 6 : Experimentally changing FreeType render mode #

Patch Set 7 : Rebase, see trybot results again #

Unified diffs Side-by-side diffs Delta from patch set Stats (+196 lines, -288 lines) Patch
M DEPS View 1 2 3 4 5 6 1 chunk +1 line, -1 line 0 comments Download
M build/linux/BUILD.gn View 1 1 chunk +7 lines, -16 lines 0 comments Download
M third_party/freetype2/BUILD.gn View 1 2 4 chunks +6 lines, -10 lines 0 comments Download
M third_party/freetype2/README.chromium View 1 2 1 chunk +31 lines, -38 lines 0 comments Download
D third_party/freetype2/freetype2.patch View 1 chunk +0 lines, -28 lines 0 comments Download
M third_party/freetype2/include/ftconfig.h View 14 chunks +136 lines, -180 lines 0 comments Download
M third_party/freetype2/include/ftmodule.h View 1 2 1 chunk +5 lines, -5 lines 0 comments Download
A + third_party/freetype2/patches/freetype2_symbols_visibility.patch View 1 2 chunks +9 lines, -9 lines 0 comments Download
M third_party/harfbuzz-ng/BUILD.gn View 1 1 chunk +1 line, -1 line 0 comments Download
Trybot results:  android_n5x_swarming_rel   ios-simulator   win_clang   win_chromium_x64_rel_ng   win_chromium_rel_ng   win_chromium_compile_dbg_ng   mac_chromium_rel_ng   mac_chromium_compile_dbg_ng   ios-simulator-xcode-clang   ios-simulator   ios-device-xcode-clang   ios-device   linux_chromium_tsan_rel_ng   linux_chromium_rel_ng   linux_chromium_compile_dbg_ng   linux_chromium_chromeos_rel_ng   linux_chromium_chromeos_ozone_rel_ng   linux_chromium_asan_rel_ng   chromium_presubmit   chromeos_daisy_chromium_compile_only_ng   chromeos_amd64-generic_chromium_compile_only_ng   linux_android_rel_ng   cast_shell_linux   cast_shell_android   android_n5x_swarming_rel   android_cronet   android_compile_dbg   android_clang_dbg_recipe   android_arm64_dbg_recipe   linux_chromium_rel_ng   win_clang   win_chromium_x64_rel_ng   win_chromium_compile_dbg_ng   win_chromium_rel_ng   ios-simulator-xcode-clang   mac_chromium_rel_ng   mac_chromium_compile_dbg_ng   ios-simulator   ios-device-xcode-clang   linux_android_rel_ng   ios-device   android_n5x_swarming_rel   cast_shell_android   android_cronet   android_compile_dbg   android_clang_dbg_recipe   android_arm64_dbg_recipe   linux_chromium_rel_ng   linux_chromium_compile_dbg_ng   linux_chromium_chromeos_rel_ng   linux_chromium_asan_rel_ng   linux_chromium_chromeos_ozone_rel_ng   chromium_presubmit   chromeos_daisy_chromium_compile_only_ng   chromeos_amd64-generic_chromium_compile_only_ng   cast_shell_linux 

Messages

Total messages: 46 (40 generated)
bungeman-skia
Just some build comments as things to keep in mind. If Chromium doesn't need to ...
1 year, 3 months ago (2016-09-01 15:08:33 UTC) #6
drott
Addressing review comments, simplified top level config
11 months, 3 weeks ago (2016-12-21 13:53:06 UTC) #7
drott
Not ready for review yet, I need to ensure that we keep the previous rendering ...
11 months, 3 weeks ago (2016-12-22 13:39:12 UTC) #26
drott
experimentally change rendering mode
11 months, 2 weeks ago (2016-12-28 13:02:01 UTC) #27
drott
Experimentally changing FreeType render mode
11 months, 2 weeks ago (2016-12-28 15:49:12 UTC) #32
drott
11 months, 1 week ago (2017-01-05 14:54:20 UTC) #37
Rebase, see trybot results again

Powered by Google App Engine
This is Rietveld 0eb02b776