media: Remove unneeded RegisterNatives() call

media: Remove unneeded RegisterNatives() call

BUG=603936
Committed: https://crrev.com/8699a9de980b74681697b8d8eabe1b050f6b1978
Cr-Commit-Position: refs/heads/master@{#409283}

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

Description was changed from

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

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

to

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

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

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

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

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

watk@ ptal!

see patchset dependency and bug for more info

[no sievers, 2016-08-01 22:42:00]

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:42:39]

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:55:51]

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

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

Dry run: This issue passed the CQ dry run.

[watk, 2016-08-01 23:59:41]

cool, lgtm

https://codereview.chromium.org/2199913003/diff/1/media/base/android/media_jn...
File media/base/android/media_jni_registrar.cc (right):

https://codereview.chromium.org/2199913003/diff/1/media/base/android/media_jn...
media/base/android/media_jni_registrar.cc:13: #include
"media/base/android/media_codec_util.h"
delete?

https://codereview.chromium.org/2199913003/diff/1/media/base/android/media_jn...
media/base/android/media_jni_registrar.cc:17: #include
"media/base/android/sdk_media_codec_bridge.h"
delete?

https://codereview.chromium.org/2199913003/diff/1/media/capture/video/android...
File media/capture/video/android/capture_jni_registrar.cc (right):

https://codereview.chromium.org/2199913003/diff/1/media/capture/video/android...
media/capture/video/android/capture_jni_registrar.cc:11: #include
"media/capture/video/android/photo_capabilities.h"
delete?

https://codereview.chromium.org/2199913003/diff/1/media/capture/video/android...
media/capture/video/android/capture_jni_registrar.cc:13: #include
"media/capture/video/android/video_capture_device_factory_android.h"
delete?

https://codereview.chromium.org/2199913003/diff/1/media/midi/midi_jni_registr...
File media/midi/midi_jni_registrar.cc (right):

https://codereview.chromium.org/2199913003/diff/1/media/midi/midi_jni_registr...
media/midi/midi_jni_registrar.cc:11: #include "media/midi/midi_device_android.h"
delete?

[no sievers, 2016-08-02 00:03:20]

thanks!

https://codereview.chromium.org/2199913003/diff/1/media/base/android/media_jn...
File media/base/android/media_jni_registrar.cc (right):

https://codereview.chromium.org/2199913003/diff/1/media/base/android/media_jn...
media/base/android/media_jni_registrar.cc:13: #include
"media/base/android/media_codec_util.h"
On 2016/08/01 23:59:40, watk wrote:
> delete?

Done.

https://codereview.chromium.org/2199913003/diff/1/media/base/android/media_jn...
media/base/android/media_jni_registrar.cc:17: #include
"media/base/android/sdk_media_codec_bridge.h"
On 2016/08/01 23:59:40, watk wrote:
> delete?

Done.

https://codereview.chromium.org/2199913003/diff/1/media/capture/video/android...
File media/capture/video/android/capture_jni_registrar.cc (right):

https://codereview.chromium.org/2199913003/diff/1/media/capture/video/android...
media/capture/video/android/capture_jni_registrar.cc:11: #include
"media/capture/video/android/photo_capabilities.h"
On 2016/08/01 23:59:41, watk wrote:
> delete?

Done.

https://codereview.chromium.org/2199913003/diff/1/media/capture/video/android...
media/capture/video/android/capture_jni_registrar.cc:13: #include
"media/capture/video/android/video_capture_device_factory_android.h"
On 2016/08/01 23:59:41, watk wrote:
> delete?

Done.

[no sievers, 2016-08-02 18:17:19]

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

[commit-bot: I haz the power, 2016-08-02 18:18:04]

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-02 19:30:05]

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

[commit-bot: I haz the power, 2016-08-02 19:30:06]

Dry run: This issue passed the CQ dry run.

[no sievers, 2016-08-02 19:46:46]

The CQ bit was checked by sievers@chromium.org

[no sievers, 2016-08-02 19:46:47]

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

[commit-bot: I haz the power, 2016-08-02 19:47:17]

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 19:52:28]

Description was changed from

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

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

to

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

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

[commit-bot: I haz the power, 2016-08-02 19:52:30]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-08-02 19:54:18]

Description was changed from

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

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

to

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

BUG=603936

Committed: https://crrev.com/8699a9de980b74681697b8d8eabe1b050f6b1978
Cr-Commit-Position: refs/heads/master@{#409283}
==========

[commit-bot: I haz the power, 2016-08-02 19:54:19]

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