Fix content_browsertests on Linux GN build.

Fix content_browsertests on Linux GN build.

BUG=461044
Committed: https://crrev.com/b3dd04980de9b7bb42c5ca198b3152cbf485c956
Cr-Commit-Position: refs/heads/master@{#318078}

[jam, 2015-02-25 07:16:51]

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

[brettw, 2015-02-25 16:40:38]

lgtm

https://codereview.chromium.org/954873002/diff/20001/content/test/BUILD.gn
File content/test/BUILD.gn (right):

https://codereview.chromium.org/954873002/diff/20001/content/test/BUILD.gn#ne...
content/test/BUILD.gn:416: deps += [ "//ppapi:ppapi_tests" ]
Can you make this data_deps instead? This is really a runtime dependency.

https://codereview.chromium.org/954873002/diff/20001/ppapi/shared_impl/BUILD.gn
File ppapi/shared_impl/BUILD.gn (left):

https://codereview.chromium.org/954873002/diff/20001/ppapi/shared_impl/BUILD....
ppapi/shared_impl/BUILD.gn:171: #}
Delete this line too

[jam, 2015-02-25 16:55:24]

https://codereview.chromium.org/954873002/diff/20001/content/test/BUILD.gn
File content/test/BUILD.gn (right):

https://codereview.chromium.org/954873002/diff/20001/content/test/BUILD.gn#ne...
content/test/BUILD.gn:416: deps += [ "//ppapi:ppapi_tests" ]
On 2015/02/25 16:40:38, brettw wrote:
> Can you make this data_deps instead? This is really a runtime dependency.

Done.

https://codereview.chromium.org/954873002/diff/20001/ppapi/shared_impl/BUILD.gn
File ppapi/shared_impl/BUILD.gn (left):

https://codereview.chromium.org/954873002/diff/20001/ppapi/shared_impl/BUILD....
ppapi/shared_impl/BUILD.gn:171: #}
On 2015/02/25 16:40:38, brettw wrote:
> Delete this line too

Done.

[jam, 2015-02-25 16:55:27]

The CQ bit was checked by jam@chromium.org

[jam, 2015-02-25 16:55:27]

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/954873002/#ps40001 (title:
" ")

[commit-bot: I haz the power, 2015-02-25 16:56:00]

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

[commit-bot: I haz the power, 2015-02-25 17:14:16]

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

[commit-bot: I haz the power, 2015-02-25 17:14:17]

Try jobs failed on following builders:
  linux_chromium_gn_dbg on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_gn_rel on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[jam, 2015-02-25 17:18:31]

The CQ bit was checked by jam@chromium.org

[jam, 2015-02-25 17:18:32]

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/954873002/#ps60001 (title:
" ")

[commit-bot: I haz the power, 2015-02-25 17:19:03]

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

[commit-bot: I haz the power, 2015-02-25 18:23:07]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-02-25 18:23:47]

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