Fix a bug that progress notification can linger after download completes

Fix a bug that progress notification can linger after download completes

If the size of the download is unknwon, percentage completed is
always -1.
The current code doesn't add the download into SharedPreference in that case.
As a result, the SharedPref will not be found when download completes.
This causes notifyDownloadSuccess() to early return.
This CL fix the issue by always adding the download entry into the
sharedprefs.

TBR=tedchoc@chromium.org
BUG=608091
Review-Url: https://codereview.chromium.org/1939593002
Cr-Commit-Position: refs/heads/master@{#390970}
(cherry picked from commit f0e99eea9df4ba893cb307960f3bc7194e469bd7)

Committed: https://chromium.googlesource.com/chromium/src/+/ab9b8915bbd8eb8d21c24de8a4929fd020b93c8f

[qinmin, 2016-05-02 21:01:06]

Description was changed from

==========
Fix a bug that progress notification can linger after download completes

If the size of the download is unknwon, percentage completed is
always -1.
The current code doesn't add the download into SharedPreference in that case.
As a result, the SharedPref will not be found when download completes.
This causes notifyDownloadSuccess() to early return.
This CL fix the issue by always adding the download entry into the
sharedprefs.

BUG=608091

Review-Url: https://codereview.chromium.org/1939593002
Cr-Commit-Position: refs/heads/master@{#390970}
(cherry picked from commit f0e99eea9df4ba893cb307960f3bc7194e469bd7)
==========

to

==========
Fix a bug that progress notification can linger after download completes

If the size of the download is unknwon, percentage completed is
always -1.
The current code doesn't add the download into SharedPreference in that case.
As a result, the SharedPref will not be found when download completes.
This causes notifyDownloadSuccess() to early return.
This CL fix the issue by always adding the download entry into the
sharedprefs.

TBR=tedchoc@chromium.org
BUG=608091

Review-Url: https://codereview.chromium.org/1939593002
Cr-Commit-Position: refs/heads/master@{#390970}
(cherry picked from commit f0e99eea9df4ba893cb307960f3bc7194e469bd7)
==========

[qinmin, 2016-05-02 21:04:33]

Description was changed from

==========
Fix a bug that progress notification can linger after download completes

If the size of the download is unknwon, percentage completed is
always -1.
The current code doesn't add the download into SharedPreference in that case.
As a result, the SharedPref will not be found when download completes.
This causes notifyDownloadSuccess() to early return.
This CL fix the issue by always adding the download entry into the
sharedprefs.

TBR=tedchoc@chromium.org
BUG=608091

Review-Url: https://codereview.chromium.org/1939593002
Cr-Commit-Position: refs/heads/master@{#390970}
(cherry picked from commit f0e99eea9df4ba893cb307960f3bc7194e469bd7)
==========

to

==========
Fix a bug that progress notification can linger after download completes

If the size of the download is unknwon, percentage completed is
always -1.
The current code doesn't add the download into SharedPreference in that case.
As a result, the SharedPref will not be found when download completes.
This causes notifyDownloadSuccess() to early return.
This CL fix the issue by always adding the download entry into the
sharedprefs.

TBR=tedchoc@chromium.org
BUG=608091

Review-Url: https://codereview.chromium.org/1939593002
Cr-Commit-Position: refs/heads/master@{#390970}
(cherry picked from commit f0e99eea9df4ba893cb307960f3bc7194e469bd7)

Committed:
https://chromium.googlesource.com/chromium/src/+/ab9b8915bbd8eb8d21c24de8a492...
==========

[qinmin, 2016-05-02 21:04:34]

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