Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1334)

Issue 2795583002: Don't force the download notifier to unbind (Closed)

Created:
3 years, 8 months ago by David Trainor- moved to gerrit
Modified:
3 years, 8 months ago
Reviewers:
CC:
chromium-reviews
Target Ref:
refs/branch-heads/3029
Project:
chromium
Visibility:
Public.

Description

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} (cherry picked from commit 008b3fe04b6cefb33e44ebb30c4a2fc5881d6d40) Review-Url: https://codereview.chromium.org/2795583002 . Cr-Commit-Position: refs/branch-heads/3029@{#531} Cr-Branched-From: 939b32ee5ba05c396eef3fd992822fcca9a2e262-refs/heads/master@{#454471} Committed: https://chromium.googlesource.com/chromium/src/+/6b74cf25e32daf095f452c397353ff23aa89d09a

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -20 lines) Patch
M chrome/android/java/src/org/chromium/chrome/browser/download/DownloadNotificationService.java View 2 chunks +5 lines, -10 lines 0 comments Download
M chrome/android/java/src/org/chromium/chrome/browser/download/SystemDownloadNotifier.java View 2 chunks +0 lines, -10 lines 0 comments Download

Messages

Total messages: 2 (1 generated)
David Trainor- moved to gerrit
3 years, 8 months ago (2017-04-02 16:50:51 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
6b74cf25e32daf095f452c397353ff23aa89d09a (tree was closed).

Powered by Google App Engine
This is Rietveld 408576698