Fix null ref in NetworkMonitorAutoDetect if Connectivity Manager service is unavailable

Fix null ref in NetworkMonitorAutoDetect if Connectivity Manager service is unavailable

BUG=webrtc:7917
TBR=magjed@webrtc.org

Review-Url: https://codereview.webrtc.org/2963363002
Cr-Commit-Position: refs/heads/master@{#18906}
Committed: https://chromium.googlesource.com/external/webrtc/+/6aa95117d8d6270d8ef4da8b33cbdd67b4043ba8

[bdodson, 2017-06-30 20:37:14]

bdodson@google.com changed reviewers:
+ deadbeef@chromium.org, zhihuang@webrtc.org

[Zhi Huang, 2017-06-30 20:51:10]

On 2017/06/30 20:37:14, bdodson wrote:

LGTM.

Taylor and I don't have power here. You might need approval from
magjed@webrtc.org. 

If this is a blocker, you could TBR=magjed@webrtc.org to submit it.

[bdodson, 2017-06-30 21:05:50]

Description was changed from

==========
Fix null ref in NetworkMonitorAutoDetect if Connectivity Manager service is
unavailable

BUG=webrtc:7917
==========

to

==========
Fix null ref in NetworkMonitorAutoDetect if Connectivity Manager service is
unavailable

BUG=webrtc:7917
==========

[bdodson, 2017-06-30 21:05:50]

bdodson@google.com changed reviewers:
+ magjed@google.com

[Taylor Brandstetter, 2017-07-05 16:30:11]

deadbeef@webrtc.org changed reviewers:
+ deadbeef@webrtc.org

[Taylor Brandstetter, 2017-07-05 16:30:12]

lgtm

[Taylor Brandstetter, 2017-07-05 16:32:52]

Description was changed from

==========
Fix null ref in NetworkMonitorAutoDetect if Connectivity Manager service is
unavailable

BUG=webrtc:7917
==========

to

==========
Fix null ref in NetworkMonitorAutoDetect if Connectivity Manager service is
unavailable

BUG=webrtc:7917
TBR=magjed@webrtc.org
==========

[Taylor Brandstetter, 2017-07-05 16:32:56]

The CQ bit was checked by deadbeef@webrtc.org

[commit-bot: I haz the power, 2017-07-05 16:32:59]

CQ is trying da patch.

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

[commit-bot: I haz the power, 2017-07-05 16:55:10]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1499272376378460, "parent_rev":
"e4f63a1929615825c065f801955440acc4ef4b05", "commit_rev":
"6aa95117d8d6270d8ef4da8b33cbdd67b4043ba8"}

[commit-bot: I haz the power, 2017-07-05 16:55:15]

Description was changed from

==========
Fix null ref in NetworkMonitorAutoDetect if Connectivity Manager service is
unavailable

BUG=webrtc:7917
TBR=magjed@webrtc.org
==========

to

==========
Fix null ref in NetworkMonitorAutoDetect if Connectivity Manager service is
unavailable

BUG=webrtc:7917
TBR=magjed@webrtc.org

Review-Url: https://codereview.webrtc.org/2963363002
Cr-Commit-Position: refs/heads/master@{#18906}
Committed:
https://chromium.googlesource.com/external/webrtc/+/6aa95117d8d6270d8ef4da8b3...
==========

[commit-bot: I haz the power, 2017-07-05 16:55:16]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/external/webrtc/+/6aa95117d8d6270d8ef4da8b3...

[magjed_webrtc, 2017-07-07 11:49:06]

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

[magjed_webrtc, 2017-07-07 11:49:07]

https://codereview.webrtc.org/2963363002/diff/1/webrtc/sdk/android/api/org/we...
File webrtc/sdk/android/api/org/webrtc/NetworkMonitorAutoDetect.java (right):

https://codereview.webrtc.org/2963363002/diff/1/webrtc/sdk/android/api/org/we...
webrtc/sdk/android/api/org/webrtc/NetworkMonitorAutoDetect.java:596: return
null;
Shouldn't we still add possible entries from wifiDirectManagerDelegate here?