Create a gypi that contains all the SK_API header files for use by clients (i.e. chromium).

Create a gypi that contains all the SK_API header files for use by clients (i.e. chromium).

R=robertphillips@google.com, torne@chromium.org

Committed: https://code.google.com/p/skia/source/detail?r=10637

[djsollen, 2013-08-07 20:57:39]

torne, is this the format that you need or should I have it wrapped in a
variables section?

[Torne, 2013-08-07 21:33:32]

On 2013/08/07 20:57:39, djsollen wrote:
> torne, is this the format that you need or should I have it wrapped in a
> variables section?

I believe the top level of a .gyp[i] file must always be a dictionary, and it's
probably best to have it be {'variables': {'header_filenames':[]}} so that I can
just include it at the top level.

[djsollen, 2013-08-08 12:08:29]

torne, you might want to drop this on your machine and test it out, but I think
it should be good to do.

The next step on our side will be to have our housekeeping bots autogenerate
this file if we add or remove SK_API from a file.

[robertphillips, 2013-08-08 12:17:57]

lgtm

[Torne, 2013-08-08 12:22:23]

lgtm

[robertphillips, 2013-08-08 12:23:47]

Derek: do we already have an issue with the Skia Infrastructure team for the
automation?

[djsollen, 2013-08-08 12:31:09]

Committed patchset #2 manually as r10637 (presubmit successful).

[djsollen, 2013-08-08 12:36:24]

We have an issue for the automation filed at...

https://code.google.com/p/skia/issues/detail?id=1489

[tfarina, 2013-08-08 23:47:04]

can we take a "different" approach here?

I'd like to follow the same pattern as of Blink,  i.e.,
third_party/WebKit/public/blink.gyp:blink, so we could do the same here
third_party/skia/public/skia.gyp:skia

This is not very related to the work done here though, since this is just a gypi
include file.

[Torne, 2013-08-09 14:19:24]

Hm, I've just tested this more carefully and it doesn't actually work; the list
doesn't include SkDraw, SkSize or SkThread.h from include/core, all of which are
included by files in src/skia/ext in Chromium..