blimp: Remove unneeded RegisterNatives() call

blimp: Remove unneeded RegisterNatives() call

BUG=603936
Committed: https://crrev.com/38d2b2cbd951ca3e54365b1559cc984401631d5a
Cr-Commit-Position: refs/heads/master@{#409227}

[no sievers, 2016-08-01 22:13:50]

Description was changed from

==========
blimp: Remove unneeded RegisterNatives() call

BUG=603936
==========

to

==========
blimp: Remove unneeded RegisterNatives() call

BUG=603936
==========

[no sievers, 2016-08-01 22:13:50]

sievers@chromium.org changed reviewers:
+ dtrainor@chromium.org

[no sievers, 2016-08-01 22:14:33]

dtrainor: ptal

Making it so that you don't need to call RegisterNatives() anymore if there are
not Java->native methods (but only vice versa).

[nyquist, 2016-08-01 22:22:04]

nyquist@chromium.org changed reviewers:
+ nyquist@chromium.org

[nyquist, 2016-08-01 22:22:05]

https://codereview.chromium.org/2194403003/diff/1/blimp/client/core/android/b...
File blimp/client/core/android/blimp_jni_registrar.cc (left):

https://codereview.chromium.org/2194403003/diff/1/blimp/client/core/android/b...
blimp/client/core/android/blimp_jni_registrar.cc:24: //
//blimp/client/public/android/blimp_jni_registrar.h, and either this function
Why did you change this comment?

[no sievers, 2016-08-01 22:25:57]

sievers@chromium.org changed reviewers:
- dtrainor@chromium.org

[no sievers, 2016-08-01 22:26:21]

https://codereview.chromium.org/2194403003/diff/1/blimp/client/core/android/b...
File blimp/client/core/android/blimp_jni_registrar.cc (left):

https://codereview.chromium.org/2194403003/diff/1/blimp/client/core/android/b...
blimp/client/core/android/blimp_jni_registrar.cc:24: //
//blimp/client/public/android/blimp_jni_registrar.h, and either this function
On 2016/08/01 22:22:05, nyquist wrote:
> Why did you change this comment?

Oh that's an artefact from when I attempted to remove the other file, which then
I did differently because of this compile time magic it does.

Reverted.

[nyquist, 2016-08-01 22:28:32]

lgtm

[no sievers, 2016-08-01 22:41:16]

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

[commit-bot: I haz the power, 2016-08-01 22:41:41]

Dry run: 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, 2016-08-01 23:46:07]

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

[commit-bot: I haz the power, 2016-08-01 23:46:08]

Dry run: This issue passed the CQ dry run.

[no sievers, 2016-08-02 17:40:12]

The CQ bit was checked by sievers@chromium.org

[commit-bot: I haz the power, 2016-08-02 17:41:00]

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, 2016-08-02 17:45:32]

Description was changed from

==========
blimp: Remove unneeded RegisterNatives() call

BUG=603936
==========

to

==========
blimp: Remove unneeded RegisterNatives() call

BUG=603936
==========

[commit-bot: I haz the power, 2016-08-02 17:45:33]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-08-02 17:48:44]

Description was changed from

==========
blimp: Remove unneeded RegisterNatives() call

BUG=603936
==========

to

==========
blimp: Remove unneeded RegisterNatives() call

BUG=603936

Committed: https://crrev.com/38d2b2cbd951ca3e54365b1559cc984401631d5a
Cr-Commit-Position: refs/heads/master@{#409227}
==========

[commit-bot: I haz the power, 2016-08-02 17:48:46]

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