jni_helpers: Optimize IsNull()

jni_helpers: Optimize IsNull()

The current implementation is unnecessary expensive - we create a local reference frame for creating new Java objects and then create a new local reference. It's cheaper to just do jni->IsSameObject(obj, nullptr).

R=perkj@webrtc.org

Committed: https://crrev.com/ffdd41ecf2180c4f52f003915a4bd514bb5e5736
Cr-Commit-Position: refs/heads/master@{#11825}

[magjed_webrtc, 2016-02-26 10:00:34]

Description was changed from

==========
jni_helpers: Optimize IsNull()

The current implementation is unnecessary expensive - we create a local
reference frame for creating new Java objects and then create a new local
reference. It's cheaper to just do jni->IsSameObject(obj, nullptr).
==========

to

==========
jni_helpers: Optimize IsNull()

The current implementation is unnecessary expensive - we create a local
reference frame for creating new Java objects and then create a new local
reference. It's cheaper to just do jni->IsSameObject(obj, nullptr).
==========

[magjed_webrtc, 2016-02-26 10:00:34]

magjed@webrtc.org changed reviewers:
+ perkj@webrtc.org

[magjed_webrtc, 2016-02-26 10:45:51]

perkj - Please take a look.

[perkj_webrtc, 2016-02-26 11:58:40]

lgtm

[magjed_webrtc, 2016-02-29 08:43:20]

The CQ bit was checked by magjed@webrtc.org

[commit-bot: I haz the power, 2016-02-29 08:43:23]

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

[commit-bot: I haz the power, 2016-02-29 08:53:53]

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

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

Exceeded global retry quota

[magjed_webrtc, 2016-03-01 09:10:15]

Description was changed from

==========
jni_helpers: Optimize IsNull()

The current implementation is unnecessary expensive - we create a local
reference frame for creating new Java objects and then create a new local
reference. It's cheaper to just do jni->IsSameObject(obj, nullptr).
==========

to

==========
jni_helpers: Optimize IsNull()

The current implementation is unnecessary expensive - we create a local
reference frame for creating new Java objects and then create a new local
reference. It's cheaper to just do jni->IsSameObject(obj, nullptr).

R=perkj@webrtc.org

Committed:
https://chromium.googlesource.com/external/webrtc/+/ffdd41ecf2180c4f52f003915...
==========

[magjed_webrtc, 2016-03-01 09:10:17]

Committed patchset #1 (id:1) manually as
ffdd41ecf2180c4f52f003915a4bd514bb5e5736 (presubmit successful).

[commit-bot: I haz the power, 2016-03-01 09:10:17]

Description was changed from

==========
jni_helpers: Optimize IsNull()

The current implementation is unnecessary expensive - we create a local
reference frame for creating new Java objects and then create a new local
reference. It's cheaper to just do jni->IsSameObject(obj, nullptr).

R=perkj@webrtc.org

Committed:
https://chromium.googlesource.com/external/webrtc/+/ffdd41ecf2180c4f52f003915...
==========

to

==========
jni_helpers: Optimize IsNull()

The current implementation is unnecessary expensive - we create a local
reference frame for creating new Java objects and then create a new local
reference. It's cheaper to just do jni->IsSameObject(obj, nullptr).

R=perkj@webrtc.org

Committed: https://crrev.com/ffdd41ecf2180c4f52f003915a4bd514bb5e5736
Cr-Commit-Position: refs/heads/master@{#11825}
==========

[commit-bot: I haz the power, 2016-03-01 09:10:18]

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