Make a content/test API for registering JNI.

Make a content/test API for registering JNI.

The components test setup needs to register browserJNI calls but this is not part of the content API. This makes a new content test wrapper API for doing this.

This change fixes the components tests "gn check" failures so these targets can now be added to the whitelist.

Cleans up the components/test DEPS file. The dependency allowances for this directory are quite wide so whitelisting every header file, as was done before, was needlessly complicated.

Committed: https://crrev.com/7ae6a21024a9d8f1e7a43714e8d79f9a4a9aa27e
Cr-Commit-Position: refs/heads/master@{#362476}

[brettw, 2015-12-01 00:27:11]

Description was changed from

==========
Make a content/test API for registering JNI.

The components test setup needs to register browserJNI calls but this is not
part of the content API. This makes a new content test wrapper API for doing
this.

This change fixes the components tests "gn check" failures so these targets can
now be added to the whitelist.
==========

to

==========
Make a content/test API for registering JNI.

The components test setup needs to register browserJNI calls but this is not
part of the content API. This makes a new content test wrapper API for doing
this.

This change fixes the components tests "gn check" failures so these targets can
now be added to the whitelist.

Cleans up the components/test DEPS file. The dependency allowances for this
directory are quite wide so whitelisting every header file, as was done before,
was needlessly complicated.
==========

[brettw, 2015-12-01 00:27:11]

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

[brettw, 2015-12-01 00:27:21]

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

[commit-bot: I haz the power, 2015-12-01 00:30:21]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1485893003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1485893003/1

[commit-bot: I haz the power, 2015-12-01 01:09:03]

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

[commit-bot: I haz the power, 2015-12-01 01:09:03]

Dry run: Try jobs failed on following builders:
  android_chromium_gn_compile_dbg on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/android_chromiu...)
  android_chromium_gn_compile_rel on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/android_chromiu...)
  android_clang_dbg_recipe on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/android_clang_d...)
  linux_android_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_android_r...)

[brettw, 2015-12-01 17:26:02]

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

[commit-bot: I haz the power, 2015-12-01 17:27:02]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1485893003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1485893003/20001

[commit-bot: I haz the power, 2015-12-01 17:43:10]

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

[commit-bot: I haz the power, 2015-12-01 17:43:11]

Dry run: Try jobs failed on following builders:
  android_clang_dbg_recipe on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/android_clang_d...)
  linux_android_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_android_r...)

[brettw, 2015-12-01 17:45:20]

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

[brettw, 2015-12-01 17:45:20]

namespace

[commit-bot: I haz the power, 2015-12-01 17:45:45]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1485893003/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1485893003/40001

[commit-bot: I haz the power, 2015-12-01 19:00:35]

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

[commit-bot: I haz the power, 2015-12-01 19:00:36]

Dry run: This issue passed the CQ dry run.

[jam, 2015-12-01 19:14:18]

lgtm

https://codereview.chromium.org/1485893003/diff/40001/content/public/test/tes...
File content/public/test/test_utils.cc (right):

https://codereview.chromium.org/1485893003/diff/40001/content/public/test/tes...
content/public/test/test_utils.cc:206: return
content::android::RegisterBrowserJni(env);
no content::

[brettw, 2015-12-01 19:15:02]

The CQ bit was checked by brettw@chromium.org

[commit-bot: I haz the power, 2015-12-01 19:16:23]

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

[commit-bot: I haz the power, 2015-12-01 19:30:01]

Description was changed from

==========
Make a content/test API for registering JNI.

The components test setup needs to register browserJNI calls but this is not
part of the content API. This makes a new content test wrapper API for doing
this.

This change fixes the components tests "gn check" failures so these targets can
now be added to the whitelist.

Cleans up the components/test DEPS file. The dependency allowances for this
directory are quite wide so whitelisting every header file, as was done before,
was needlessly complicated.
==========

to

==========
Make a content/test API for registering JNI.

The components test setup needs to register browserJNI calls but this is not
part of the content API. This makes a new content test wrapper API for doing
this.

This change fixes the components tests "gn check" failures so these targets can
now be added to the whitelist.

Cleans up the components/test DEPS file. The dependency allowances for this
directory are quite wide so whitelisting every header file, as was done before,
was needlessly complicated.
==========

[commit-bot: I haz the power, 2015-12-01 19:30:02]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-12-01 19:35:51]

Description was changed from

==========
Make a content/test API for registering JNI.

The components test setup needs to register browserJNI calls but this is not
part of the content API. This makes a new content test wrapper API for doing
this.

This change fixes the components tests "gn check" failures so these targets can
now be added to the whitelist.

Cleans up the components/test DEPS file. The dependency allowances for this
directory are quite wide so whitelisting every header file, as was done before,
was needlessly complicated.
==========

to

==========
Make a content/test API for registering JNI.

The components test setup needs to register browserJNI calls but this is not
part of the content API. This makes a new content test wrapper API for doing
this.

This change fixes the components tests "gn check" failures so these targets can
now be added to the whitelist.

Cleans up the components/test DEPS file. The dependency allowances for this
directory are quite wide so whitelisting every header file, as was done before,
was needlessly complicated.

Committed: https://crrev.com/7ae6a21024a9d8f1e7a43714e8d79f9a4a9aa27e
Cr-Commit-Position: refs/heads/master@{#362476}
==========

[commit-bot: I haz the power, 2015-12-01 19:35:52]

Patchset 3 (id:??) landed as
https://crrev.com/7ae6a21024a9d8f1e7a43714e8d79f9a4a9aa27e
Cr-Commit-Position: refs/heads/master@{#362476}