Download Notifications : Fixed a crash

Download Notifications : Fixed a crash

Tapping on a download notification after chrome has been killed leads
to a crash. This is because we are trying to access the native part of
DownloadManagerService before the native has been loaded. This CL fixes
the crash by adding a check for existence of the DownloadManagerService
object before accessing it.

BUG=715442
Review-Url: https://codereview.chromium.org/2842163005
Cr-Commit-Position: refs/heads/master@{#467460}
Committed: https://chromium.googlesource.com/chromium/src/+/96216ffca1555b05e3f42c359966d6181e6467c9

[shaktisahu, 2017-04-26 19:46:44]

Patchset #1 (id:1) has been deleted

[shaktisahu, 2017-04-26 19:47:07]

shaktisahu@chromium.org changed reviewers:
+ qinmin@chromium.org

[shaktisahu, 2017-04-26 19:47:07]

PTAL

[shaktisahu, 2017-04-26 19:47:29]

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

[qinmin, 2017-04-26 19:48:18]

lgtm

[qinmin, 2017-04-26 19:48:19]

lgtm

[commit-bot: I haz the power, 2017-04-26 19:48:28]

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, 2017-04-26 21:05:30]

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

[commit-bot: I haz the power, 2017-04-26 21:05:31]

Dry run: This issue passed the CQ dry run.

[shaktisahu, 2017-04-26 21:09:08]

The CQ bit was checked by shaktisahu@chromium.org

[commit-bot: I haz the power, 2017-04-26 21:10:13]

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-04-26 21:22:37]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1493240948105510,
"parent_rev": "89bf4588a7238e2da23c119ad363edab624e73e4", "commit_rev":
"96216ffca1555b05e3f42c359966d6181e6467c9"}

[commit-bot: I haz the power, 2017-04-26 21:22:46]

Description was changed from

==========
Download Notifications : Fixed a crash

Tapping on a download notification after chrome has been killed leads
to a crash. This is because we are trying to access the native part of
DownloadManagerService before the native has been loaded. This CL fixes
the crash by adding a check for existence of the DownloadManagerService
object before accessing it.

BUG=715442
==========

to

==========
Download Notifications : Fixed a crash

Tapping on a download notification after chrome has been killed leads
to a crash. This is because we are trying to access the native part of
DownloadManagerService before the native has been loaded. This CL fixes
the crash by adding a check for existence of the DownloadManagerService
object before accessing it.

BUG=715442

Review-Url: https://codereview.chromium.org/2842163005
Cr-Commit-Position: refs/heads/master@{#467460}
Committed:
https://chromium.googlesource.com/chromium/src/+/96216ffca1555b05e3f42c359966...
==========

[commit-bot: I haz the power, 2017-04-26 21:22:47]

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