Reorganized sfntly build targets.

Reorganized sfntly build targets.

- Split sfntly target into smaller pieces
- Add option to build sfntly sample programs
- Add new OWNERS

Committed: https://crrev.com/83db57815bc84d57d03aa2b2d99d14c00426454f
Cr-Commit-Position: refs/heads/master@{#413609}

[Lei Zhang, 2016-08-20 04:55:33]

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

[commit-bot: I haz the power, 2016-08-20 04:55:38]

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

[Lei Zhang, 2016-08-20 05:31:06]

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

[commit-bot: I haz the power, 2016-08-20 05:31:21]

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

[Lei Zhang, 2016-08-20 06:23:12]

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

[commit-bot: I haz the power, 2016-08-20 06:23:18]

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

[Lei Zhang, 2016-08-20 06:25:23]

thestig@chromium.org changed reviewers:
+ behdad@chromium.org, vandebo@chromium.org

[Lei Zhang, 2016-08-20 06:25:24]

behdad: WDYT?
halcanary: Wanna be in OWNERS?
vandebo: OWNERS stamp

[commit-bot: I haz the power, 2016-08-20 07:30:38]

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

[commit-bot: I haz the power, 2016-08-20 07:30:39]

Dry run: This issue passed the CQ dry run.

[vandebo (ex-Chrome), 2016-08-21 17:39:40]

On 2016/08/20 06:25:24, Lei Zhang wrote:
> behdad: WDYT?
> halcanary: Wanna be in OWNERS?
> vandebo: OWNERS stamp

OWNERS LGTM

[hal.canary, 2016-08-22 14:41:52]

On 2016/08/20 06:25:24, Lei Zhang wrote:
> halcanary: Wanna be in OWNERS?

Sure.  LGTM  

Am I the only caller of this code?

[hal.canary, 2016-08-22 14:54:10]

On 2016/08/22 14:41:52, Hal Canary wrote:
> Am I the only caller of this code?

should `sfntly_chromium` depend on `sfntly` instead of vice versa? (you would
have to change skia/BUILD.gn)

[Lei Zhang, 2016-08-22 17:50:27]

On 2016/08/22 14:41:52, Hal Canary wrote:
> On 2016/08/20 06:25:24, Lei Zhang wrote:
> > halcanary: Wanna be in OWNERS?
> 
> Sure.  LGTM  

Ok, will add you to OWNERS.

> Am I the only caller of this code?

AFAICT, yes.

[Lei Zhang, 2016-08-22 17:53:09]

On 2016/08/22 14:54:10, Hal Canary wrote:
> On 2016/08/22 14:41:52, Hal Canary wrote:
> > Am I the only caller of this code?
> 
> should `sfntly_chromium` depend on `sfntly` instead of vice versa? (you would
> have to change skia/BUILD.gn)

Meh. I don't think it's that confusing. I mostly wanted to break the sfntly
target apart because the sfntly_chromium files are in
src/cpp/src/sample/chromium, which should not be needed for all the sfntly
sample targets.

[Lei Zhang, 2016-08-22 18:51:44]

On 2016/08/22 14:41:52, Hal Canary wrote:
> On 2016/08/20 06:25:24, Lei Zhang wrote:
> > halcanary: Wanna be in OWNERS?
> 
> Sure.  LGTM  

Added you in patch set 4. Waiting for behdad.

[Lei Zhang, 2016-08-22 21:34:56]

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

[Lei Zhang, 2016-08-22 21:35:10]

thestig@chromium.org changed reviewers:
+ halcanary@google.com, jshin@chromium.org

[commit-bot: I haz the power, 2016-08-22 21:35:57]

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

[jungshik at Google, 2016-08-22 21:44:40]

LGTM

[commit-bot: I haz the power, 2016-08-23 00:34:35]

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

[commit-bot: I haz the power, 2016-08-23 00:34:36]

Dry run: This issue passed the CQ dry run.

[Lei Zhang, 2016-08-23 00:35:22]

The CQ bit was checked by thestig@chromium.org

[Lei Zhang, 2016-08-23 00:35:23]

The patchset sent to the CQ was uploaded after l-g-t-m from
vandebo@chromium.org, halcanary@google.com
Link to the patchset: https://codereview.chromium.org/2257423004/#ps80001
(title: "add jshin")

[commit-bot: I haz the power, 2016-08-23 00:36:22]

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-23 00:43:06]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-08-23 00:45:15]

Description was changed from

==========
Reorganized sfntly build targets.

- Split sfntly target into smaller pieces
- Add option to build sfntly sample programs
- Add new OWNERS
==========

to

==========
Reorganized sfntly build targets.

- Split sfntly target into smaller pieces
- Add option to build sfntly sample programs
- Add new OWNERS

Committed: https://crrev.com/83db57815bc84d57d03aa2b2d99d14c00426454f
Cr-Commit-Position: refs/heads/master@{#413609}
==========

[commit-bot: I haz the power, 2016-08-23 00:45:15]

Patchset 5 (id:??) landed as
https://crrev.com/83db57815bc84d57d03aa2b2d99d14c00426454f
Cr-Commit-Position: refs/heads/master@{#413609}