Change a data_deps to deps temporarily since it doesn't seem to be working transitively.

Change a data_deps to deps temporarily since it doesn't seem to be working transitively.

The "mojo" target depended on //ui/gl:gl transitively through a few dependencies. However when building it, libEGL etc don't get built. This works with deps but not data_deps.

BUG=477104
Committed: https://crrev.com/f7986b663e1132e9e83f95391ba99c99c338eccf
Cr-Commit-Position: refs/heads/master@{#325506}

[jam, 2015-04-16 19:26:57]

jam@chromium.org changed reviewers:
+ brettw@chromium.org

[brettw, 2015-04-16 19:33:30]

lgtm

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

https://codereview.chromium.org/1059963006/diff/1/ui/gl/BUILD.gn#newcode203
ui/gl/BUILD.gn:203: deps += [
Can you put "# TODO(brettw) bug 477104: this should be data_deps" above this?

[jam, 2015-04-16 20:01:11]

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

https://codereview.chromium.org/1059963006/diff/1/ui/gl/BUILD.gn#newcode203
ui/gl/BUILD.gn:203: deps += [
On 2015/04/16 19:33:30, brettw wrote:
> Can you put "# TODO(brettw) bug 477104: this should be data_deps" above this?

Done.

[jam, 2015-04-16 20:01:39]

The CQ bit was checked by jam@chromium.org

[jam, 2015-04-16 20:01:39]

The patchset sent to the CQ was uploaded after l-g-t-m from brettw@chromium.org
Link to the patchset: https://codereview.chromium.org/1059963006/#ps20001
(title: " ")

[commit-bot: I haz the power, 2015-04-16 20:03:55]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1059963006/20001

[commit-bot: I haz the power, 2015-04-16 20:28:13]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-04-16 20:29:06]

Patchset 2 (id:??) landed as
https://crrev.com/f7986b663e1132e9e83f95391ba99c99c338eccf
Cr-Commit-Position: refs/heads/master@{#325506}