gyp/sfntly: set SK_SFNTLY_SUBSETTER uniformly

gyp/sfntly: set SK_SFNTLY_SUBSETTER uniformly

When Skia is built into the Android framework,
SK_SFNTLY_SUBSETTER="sample/chromium/font_subsetter.h".
This sets the same value for Skia's test framework

I will eventually move away from the
   #include PREPROCESSOR_DEFINE
trick, which does not work everywhere.

No public API changes.
TBR=reed@google.com

Committed: https://skia.googlesource.com/skia/+/e37aa89c5c6f31e6ddf43d5e530dda4f127b0f5a

[hal.canary, 2016-08-24 18:13:08]

Description was changed from

==========
gyp/sfntly: set SK_SFNTLY_SUBSETTER uniformly

When Skia is built into the Android framework,
SK_SFNTLY_SUBSETTER="sample/chromium/font_subsetter.h".
This sets the same value for Skia's test framework

I will eventually move away from the
   #include PREPROCESSOR_DEFINE
trick, which does not work everywhere.

BUG=skia:
==========

to

==========
gyp/sfntly: set SK_SFNTLY_SUBSETTER uniformly

When Skia is built into the Android framework,
SK_SFNTLY_SUBSETTER="sample/chromium/font_subsetter.h".
This sets the same value for Skia's test framework

I will eventually move away from the
   #include PREPROCESSOR_DEFINE
trick, which does not work everywhere.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2274223002
==========

[hal.canary, 2016-08-24 18:14:53]

The CQ bit was checked by halcanary@google.com to run a CQ dry run

[hal.canary, 2016-08-24 18:14:59]

Patchset #1 (id:1) has been deleted

[commit-bot: I haz the power, 2016-08-24 18:14:59]

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-08-24 19:01:07]

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

[commit-bot: I haz the power, 2016-08-24 19:01:09]

Dry run: This issue passed the CQ dry run.

[hal.canary, 2016-08-24 19:15:01]

halcanary@google.com changed reviewers:
+ mtklein@google.com

[hal.canary, 2016-08-24 19:15:01]

ptal

[hal.canary, 2016-08-24 19:21:15]

The CQ bit was checked by halcanary@google.com

[commit-bot: I haz the power, 2016-08-24 19:21:17]

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-08-24 19:21:19]

Note for Reviewers:
The CQ is waiting for an approval.
If you believe that the CL is not ready yet, or if you would like to L-G-T-M
with comments then please uncheck the CQ checkbox.

Waiting for LGTM from valid reviewer(s) till 2016-08-25 01:21 UTC

[mtklein, 2016-08-24 19:22:28]

lgtm

[commit-bot: I haz the power, 2016-08-24 19:22:47]

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

[commit-bot: I haz the power, 2016-08-24 19:22:48]

Try jobs failed on following builders:
  skia_presubmit-Trybot on master.client.skia.fyi (JOB_FAILED,
http://build.chromium.org/p/client.skia.fyi/builders/skia_presubmit-Trybot/bu...)

[hal.canary, 2016-08-24 19:25:24]

Description was changed from

==========
gyp/sfntly: set SK_SFNTLY_SUBSETTER uniformly

When Skia is built into the Android framework,
SK_SFNTLY_SUBSETTER="sample/chromium/font_subsetter.h".
This sets the same value for Skia's test framework

I will eventually move away from the
   #include PREPROCESSOR_DEFINE
trick, which does not work everywhere.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2274223002
==========

to

==========
gyp/sfntly: set SK_SFNTLY_SUBSETTER uniformly

When Skia is built into the Android framework,
SK_SFNTLY_SUBSETTER="sample/chromium/font_subsetter.h".
This sets the same value for Skia's test framework

I will eventually move away from the
   #include PREPROCESSOR_DEFINE
trick, which does not work everywhere.

No public API changes.
TBR=reed@google.com
==========

[hal.canary, 2016-08-24 19:25:31]

The CQ bit was checked by halcanary@google.com

[commit-bot: I haz the power, 2016-08-24 19:25:40]

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-08-24 19:28:42]

Description was changed from

==========
gyp/sfntly: set SK_SFNTLY_SUBSETTER uniformly

When Skia is built into the Android framework,
SK_SFNTLY_SUBSETTER="sample/chromium/font_subsetter.h".
This sets the same value for Skia's test framework

I will eventually move away from the
   #include PREPROCESSOR_DEFINE
trick, which does not work everywhere.

No public API changes.
TBR=reed@google.com
==========

to

==========
gyp/sfntly: set SK_SFNTLY_SUBSETTER uniformly

When Skia is built into the Android framework,
SK_SFNTLY_SUBSETTER="sample/chromium/font_subsetter.h".
This sets the same value for Skia's test framework

I will eventually move away from the
   #include PREPROCESSOR_DEFINE
trick, which does not work everywhere.

No public API changes.
TBR=reed@google.com

Committed:
https://skia.googlesource.com/skia/+/e37aa89c5c6f31e6ddf43d5e530dda4f127b0f5a
==========

[commit-bot: I haz the power, 2016-08-24 19:28:43]

Committed patchset #1 (id:20001) as
https://skia.googlesource.com/skia/+/e37aa89c5c6f31e6ddf43d5e530dda4f127b0f5a