Remove the speical case that we don't push mojom.js data deps to device on Android.

Remove the speical case that we don't push mojom.js data deps to device on Android.

The mojom("foo") GN template now only marks generated mojom.js files as
data deps for the "foo_js_data_deps" target. So it is no longer an
issue.

For example, now
$grep '\.mojom\.js'
out/Debug/gen/content/test/content_unittests_apk.native.runtimedeps | wc
-l
2

(The two mojom.js files are from //mojo/public/js/bindings, which is
going to be removed soon.)

BUG=603212
Review-Url: https://codereview.chromium.org/2952223002
Cr-Commit-Position: refs/heads/master@{#481623}
Committed: https://chromium.googlesource.com/chromium/src/+/c8b216bbd914a0e91b66d13b2030735c2ee2d964

[yzshen1, 2017-06-22 17:45:30]

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

[commit-bot: I haz the power, 2017-06-22 17:45:48]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[yzshen1, 2017-06-22 17:46:09]

yzshen@chromium.org changed reviewers:
+ agrieve@chromium.org

[yzshen1, 2017-06-22 17:46:09]

Hi,

Would you please take a look? Thanks!

[agrieve, 2017-06-22 18:10:00]

On 2017/06/22 17:46:09, yzshen1 wrote:
> Hi,
> 
> Would you please take a look? Thanks!

lgtm. Great to have this fixed up!

[yzshen1, 2017-06-22 18:15:28]

The CQ bit was unchecked by yzshen@chromium.org

[yzshen1, 2017-06-22 18:15:28]

The CQ bit was checked by yzshen@chromium.org

[commit-bot: I haz the power, 2017-06-22 18:15:34]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-06-22 19:22:14]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1498155328748130, "parent_rev":
"d6f6e8e707132d83ab89f7e039b2b4c6b0942803", "commit_rev":
"c8b216bbd914a0e91b66d13b2030735c2ee2d964"}

[commit-bot: I haz the power, 2017-06-22 19:22:29]

Description was changed from

==========
Remove the speical case that we don't push mojom.js data deps to device on
Android.

The mojom("foo") GN template now only marks generated mojom.js files as
data deps for the "foo_js_data_deps" target. So it is no longer an
issue.

For example, now
$grep '\.mojom\.js'
out/Debug/gen/content/test/content_unittests_apk.native.runtimedeps | wc
-l
2

(The two mojom.js files are from //mojo/public/js/bindings, which is
going to be removed soon.)

BUG=603212
==========

to

==========
Remove the speical case that we don't push mojom.js data deps to device on
Android.

The mojom("foo") GN template now only marks generated mojom.js files as
data deps for the "foo_js_data_deps" target. So it is no longer an
issue.

For example, now
$grep '\.mojom\.js'
out/Debug/gen/content/test/content_unittests_apk.native.runtimedeps | wc
-l
2

(The two mojom.js files are from //mojo/public/js/bindings, which is
going to be removed soon.)

BUG=603212

Review-Url: https://codereview.chromium.org/2952223002
Cr-Commit-Position: refs/heads/master@{#481623}
Committed:
https://chromium.googlesource.com/chromium/src/+/c8b216bbd914a0e91b66d13b2030...
==========

[commit-bot: I haz the power, 2017-06-22 19:22:30]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/c8b216bbd914a0e91b66d13b2030...