Don't force the download notifier to unbind

Don't force the download notifier to unbind

The DownloadNotificationService tells the SystemDownloadNotifier to
unbind from it when it has no notifications to show and wants to shut
itself down.  This isn't needed because the SystemDownloadNotifier will
unbind when it is a safe time for the service to die.  This patch
removes the Observer method that tells the SystemDownloadNotifier to
unbind itself.

BUG=703212
Review-Url: https://codereview.chromium.org/2771353002
Cr-Commit-Position: refs/heads/master@{#460641}
Committed: https://chromium.googlesource.com/chromium/src/+/008b3fe04b6cefb33e44ebb30c4a2fc5881d6d40

[David Trainor- moved to gerrit, 2017-03-28 22:43:43]

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

[David Trainor- moved to gerrit, 2017-03-28 22:43:44]

ptal thanks!

[qinmin, 2017-03-29 16:27:47]

lgtm

[David Trainor- moved to gerrit, 2017-03-29 17:45:13]

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

[commit-bot: I haz the power, 2017-03-29 17:45:38]

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-03-29 17:48:40]

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

[commit-bot: I haz the power, 2017-03-29 17:48:41]

Dry run: Try jobs failed on following builders:
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  ios-simulator-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-xco...)

[David Trainor- moved to gerrit, 2017-03-30 01:31:22]

The CQ bit was checked by dtrainor@chromium.org

[David Trainor- moved to gerrit, 2017-03-30 01:31:22]

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

[commit-bot: I haz the power, 2017-03-30 01:33: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, 2017-03-30 03:26:04]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1490837482241790,
"parent_rev": "b46e8921b65ccb41c397bfac915240c0fd194474", "commit_rev":
"008b3fe04b6cefb33e44ebb30c4a2fc5881d6d40"}

[commit-bot: I haz the power, 2017-03-30 03:27:07]

Description was changed from

==========
Don't force the download notifier to unbind

The DownloadNotificationService tells the SystemDownloadNotifier to
unbind from it when it has no notifications to show and wants to shut
itself down.  This isn't needed because the SystemDownloadNotifier will
unbind when it is a safe time for the service to die.  This patch
removes the Observer method that tells the SystemDownloadNotifier to
unbind itself.

BUG=703212
==========

to

==========
Don't force the download notifier to unbind

The DownloadNotificationService tells the SystemDownloadNotifier to
unbind from it when it has no notifications to show and wants to shut
itself down.  This isn't needed because the SystemDownloadNotifier will
unbind when it is a safe time for the service to die.  This patch
removes the Observer method that tells the SystemDownloadNotifier to
unbind itself.

BUG=703212

Review-Url: https://codereview.chromium.org/2771353002
Cr-Commit-Position: refs/heads/master@{#460641}
Committed:
https://chromium.googlesource.com/chromium/src/+/008b3fe04b6cefb33e44ebb30c4a...
==========

[commit-bot: I haz the power, 2017-03-30 03:27:08]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/008b3fe04b6cefb33e44ebb30c4a...