NOT FOR LANDING Update FreeType to upstream 57a6733dcf7828fe3db9254edab33fda7c9f6a10.

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

[drott, 2016-09-01 12:18:56]

The CQ bit was checked by drott@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-01 12:19:07]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-01 14:56:53]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-01 14:56:54]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[bungeman-skia, 2016-09-01 15:08:31]

bungeman@google.com changed reviewers:
+ bungeman@google.com

[bungeman-skia, 2016-09-01 15:08:33]

Just some build comments as things to keep in mind. If Chromium doesn't need to
support anything other than opentype there's a lot of code and modules that
don't need to be compiled in. I'm not sure what Pdfium relies on actually
existing, but I'm pretty sure there's no need for a Chromium specific build of
FreeType to include FNT and PFR. Also, the 'validator' is just dead weight. It
was an attempt to do something like ots, but is much less.

https://codereview.chromium.org/2300973002/diff/1/third_party/freetype2/BUILD.gn
File third_party/freetype2/BUILD.gn (right):

https://codereview.chromium.org/2300973002/diff/1/third_party/freetype2/BUILD...
third_party/freetype2/BUILD.gn:19: "src/builds/unix/ftsystem.c",
Is there a good reason for this over src/base/ftsystem.c? It would seem a good
justification would be needed, seeing as this blocks future use of FreeType on
other platforms and I believe breaks memory tracking (Skia always sets the
allocator to use, I think this breaks that?)

https://codereview.chromium.org/2300973002/diff/1/third_party/freetype2/BUILD...
third_party/freetype2/BUILD.gn:23: "src/src/base/ftbdf.c",
No longer used.

https://codereview.chromium.org/2300973002/diff/1/third_party/freetype2/BUILD...
third_party/freetype2/BUILD.gn:35: "src/src/base/ftotval.c",
We neither need nor want this.

https://codereview.chromium.org/2300973002/diff/1/third_party/freetype2/BUILD...
third_party/freetype2/BUILD.gn:36: "src/src/base/ftpfr.c",
No longer used.

https://codereview.chromium.org/2300973002/diff/1/third_party/freetype2/BUILD...
third_party/freetype2/BUILD.gn:39: "src/src/base/fttype1.c",
No longer used.

https://codereview.chromium.org/2300973002/diff/1/third_party/freetype2/BUILD...
third_party/freetype2/BUILD.gn:40: "src/src/base/ftwinfnt.c",
No longer used.

https://codereview.chromium.org/2300973002/diff/1/third_party/freetype2/BUILD...
third_party/freetype2/BUILD.gn:41: "src/src/bdf/bdf.c",
No longer used.

https://codereview.chromium.org/2300973002/diff/1/third_party/freetype2/BUILD...
third_party/freetype2/BUILD.gn:43: "src/src/cid/type1cid.c",
No longer used.

https://codereview.chromium.org/2300973002/diff/1/third_party/freetype2/BUILD...
third_party/freetype2/BUILD.gn:46: "src/src/pcf/pcf.c",
No longer used.

https://codereview.chromium.org/2300973002/diff/1/third_party/freetype2/BUILD...
third_party/freetype2/BUILD.gn:47: "src/src/pfr/pfr.c",
No longer used.

https://codereview.chromium.org/2300973002/diff/1/third_party/freetype2/BUILD...
third_party/freetype2/BUILD.gn:55: "src/src/type1/type1.c",
No longer used.

https://codereview.chromium.org/2300973002/diff/1/third_party/freetype2/BUILD...
third_party/freetype2/BUILD.gn:57: "src/src/winfonts/winfnt.c",
No longer used.

[drott, 2016-12-21 13:53:06]

Addressing review comments, simplified top level config

[drott, 2016-12-21 13:53:06]

The CQ bit was checked by drott@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-21 13:53:15]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[drott, 2016-12-21 13:57:43]

Description was changed from

==========
Update Freetype to master@f44ddfda45eaded9

Work in progress - We might want to use the freetype-android directory instead
and make it point a very up to upstream version?

BUG=555855,274030
==========

to

==========
NOT FOR LANDING Statically link upstream FreeType on Linux

Update FreeType to upstream 57a6733dcf7828fe3db9254edab33fda7c9f6a10.

Statically link our own FreeType on Linux. This is needed since newer
fixes related to OpenType Variable fonts are coming to ToT in upstream
FreeType, without which we can't support the feature reliably. Also,
FreeType versions newer than 2.5.2 (which is standard on Ubuntu 14.04)
have better color emoji support.

BUG=274030
==========

[commit-bot: I haz the power, 2016-12-21 14:19:22]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-12-21 14:19:23]

Dry run: Try jobs failed on following builders:
  cast_shell_linux on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)

[drott, 2016-12-21 14:42:15]

The CQ bit was checked by drott@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-21 14:42:27]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[drott, 2016-12-21 14:42:31]

Patchset #3 (id:40001) has been deleted

[drott, 2016-12-21 14:43:06]

The CQ bit was checked by drott@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-21 14:43:17]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-21 17:32:18]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-12-21 17:32:19]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[drott, 2016-12-22 07:52:05]

The CQ bit was checked by drott@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-22 07:52:15]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[drott, 2016-12-22 10:40:31]

The CQ bit was checked by drott@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-22 10:40:47]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-22 13:27:24]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-12-22 13:27:25]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[drott, 2016-12-22 13:39:12]

Not ready for review yet, I need to ensure that we keep the previous rendering
settings on Linux, currently, Linux rebaselines would be too huge.

[drott, 2016-12-28 13:02:01]

experimentally change rendering mode

[drott, 2016-12-28 13:02:02]

The CQ bit was checked by drott@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-28 13:02:13]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-28 13:13:44]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-12-28 13:13:45]

Dry run: Try jobs failed on following builders:
  chromeos_daisy_chromium_compile_only_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_daisy_...)

[drott, 2016-12-28 15:49:12]

Experimentally changing FreeType render mode

[drott, 2016-12-28 15:49:12]

The CQ bit was checked by drott@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-28 15:49:23]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-28 18:41:01]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-12-28 18:41:02]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[drott, 2017-01-05 14:54:20]

Rebase, see trybot results again

[drott, 2017-01-05 14:54:21]

The CQ bit was checked by drott@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-05 14:54:31]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-05 17:32:39]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-01-05 17:32:40]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[drott, 2017-01-19 15:07:42]

Description was changed from

==========
NOT FOR LANDING Statically link upstream FreeType on Linux

Update FreeType to upstream 57a6733dcf7828fe3db9254edab33fda7c9f6a10.

Statically link our own FreeType on Linux. This is needed since newer
fixes related to OpenType Variable fonts are coming to ToT in upstream
FreeType, without which we can't support the feature reliably. Also,
FreeType versions newer than 2.5.2 (which is standard on Ubuntu 14.04)
have better color emoji support.

BUG=274030
==========

to

==========
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
==========

[drott, 2017-02-02 19:16:10]

The CQ bit was checked by drott@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-02-02 19:16:42]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-02 20:53:44]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-02-02 20:53:45]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)