Add GN build file for surface.

Add GN build file for surface.

This is needed by content.

BUG=None
TEST=gn gen out/Debug; followed by ninja -C out/Debug surface
R=brettw@chromium.org,piman@chromium.org
NOTRY=true

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=271163

[brettw, 2014-05-16 16:57:04]

lgtm

https://codereview.chromium.org/282353002/diff/1/BUILD.gn
File BUILD.gn (right):

https://codereview.chromium.org/282353002/diff/1/BUILD.gn#newcode87
BUILD.gn:87: "//ui/surface",
Is this because it depends on Skia? Can you put a comment here about why it's
excluded on Android so I know when I can remove it from this list? (I have a
patch to fix Skia on Android).

https://codereview.chromium.org/282353002/diff/1/ui/surface/BUILD.gn
File ui/surface/BUILD.gn (right):

https://codereview.chromium.org/282353002/diff/1/ui/surface/BUILD.gn#newcode30
ui/surface/BUILD.gn:30: #'include_dirs': [
Just do:
  include_dirs = [ "//third_party/khronos" ]

[piman, 2014-05-16 19:32:36]

LGTM if Brett is happy.

[tfarina, 2014-05-16 21:24:24]

https://codereview.chromium.org/282353002/diff/1/BUILD.gn
File BUILD.gn (right):

https://codereview.chromium.org/282353002/diff/1/BUILD.gn#newcode87
BUILD.gn:87: "//ui/surface",
On 2014/05/16 16:57:04, brettw wrote:
> Is this because it depends on Skia?
Yes. It is because Android GN is targeting ARM and Skia does not build at this
config, it still pending some ARM stuff.

> Can you put a comment here about why it's
> excluded on Android so I know when I can remove it from this list? (I have a
> patch to fix Skia on Android).
Done.

[tfarina, 2014-05-16 21:24:34]

https://codereview.chromium.org/282353002/diff/1/ui/surface/BUILD.gn
File ui/surface/BUILD.gn (right):

https://codereview.chromium.org/282353002/diff/1/ui/surface/BUILD.gn#newcode30
ui/surface/BUILD.gn:30: #'include_dirs': [
On 2014/05/16 16:57:04, brettw wrote:
> Just do:
>   include_dirs = [ "//third_party/khronos" ]

Done.

[tfarina, 2014-05-17 03:29:15]

The CQ bit was checked by tfarina@chromium.org

[commit-bot: I haz the power, 2014-05-17 03:31:21]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/tfarina@chromium.org/282353002/40001

[commit-bot: I haz the power, 2014-05-17 04:55:19]

Change committed as 271163