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.

BUG=608091
Committed: https://crrev.com/f0e99eea9df4ba893cb307960f3bc7194e469bd7
Cr-Commit-Position: refs/heads/master@{#390970}

[qinmin, 2016-04-30 00:27:08]

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

[qinmin, 2016-04-30 00:27:09]

PTAL

[Ted C, 2016-05-02 15:59:08]

lgtm

[qinmin, 2016-05-02 16:21:03]

The CQ bit was checked by qinmin@chromium.org

[commit-bot: I haz the power, 2016-05-02 16:21:44]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1939593002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1939593002/1

[commit-bot: I haz the power, 2016-05-02 17:07:36]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-05-02 17:09:12]

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
==========

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.

BUG=608091

Committed: https://crrev.com/f0e99eea9df4ba893cb307960f3bc7194e469bd7
Cr-Commit-Position: refs/heads/master@{#390970}
==========

[commit-bot: I haz the power, 2016-05-02 17:09:13]

Patchset 1 (id:??) landed as
https://crrev.com/f0e99eea9df4ba893cb307960f3bc7194e469bd7
Cr-Commit-Position: refs/heads/master@{#390970}