Fix the inability to delete just-downloaded pages from Download Home.

Fix the inability to delete just-downloaded pages from Download Home.

This happens because Offline Page is initially created w/o file path, which becomes known when the OfflinePageDownloadItem is updated in the process of download. We need to update mFilePathsToItemsMap map which is used during deletion to find all the items using the same file path.

BUG=696830
Review-Url: https://codereview.chromium.org/2721803002
Cr-Commit-Position: refs/heads/master@{#453719}
Committed: https://chromium.googlesource.com/chromium/src/+/335b9f4cc30ccbeeb8c40e3ad6b7435a73af9a6a

[Dmitry Titov, 2017-02-28 01:29:38]

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

[commit-bot: I haz the power, 2017-02-28 01:30:01]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Dmitry Titov, 2017-02-28 01:31:53]

Description was changed from

==========
Fix the inability to delete just-downloaded pages from Download Home.

BUG=696830
==========

to

==========
Fix the inability to delete just-downloaded pages from Download Home.

This happens because Offline Page is initially created w/o file path, which
becomes known when the OfflinePageDownloadItem is updated in the process of
download. We need to update mFilePathsToItemsMap map which is used during
deletion to find all the items using the same file path.

BUG=696830
==========

[Dmitry Titov, 2017-02-28 02:05:13]

dimich@chromium.org changed reviewers:
+ twellington@chromium.org

[Dmitry Titov, 2017-02-28 02:05:14]

Hi Theresa, this is another small fix for Offline Pages in Download Home.

No test for this since I didn't find an easy way to add a DownloadManagerUi
unittest. Tested manually...

[commit-bot: I haz the power, 2017-02-28 02:09:43]

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

[commit-bot: I haz the power, 2017-02-28 02:09:44]

Dry run: This issue passed the CQ dry run.

[Theresa, 2017-02-28 21:18:04]

lgtm

[Dmitry Titov, 2017-02-28 21:58:52]

The CQ bit was checked by dimich@chromium.org

[commit-bot: I haz the power, 2017-02-28 21:59:46]

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-02-28 22:08:39]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1488319132650050, "parent_rev":
"f244f7a2adbcbab7a29213da18772270208e987d", "commit_rev":
"335b9f4cc30ccbeeb8c40e3ad6b7435a73af9a6a"}

[commit-bot: I haz the power, 2017-02-28 22:09:19]

Description was changed from

==========
Fix the inability to delete just-downloaded pages from Download Home.

This happens because Offline Page is initially created w/o file path, which
becomes known when the OfflinePageDownloadItem is updated in the process of
download. We need to update mFilePathsToItemsMap map which is used during
deletion to find all the items using the same file path.

BUG=696830
==========

to

==========
Fix the inability to delete just-downloaded pages from Download Home.

This happens because Offline Page is initially created w/o file path, which
becomes known when the OfflinePageDownloadItem is updated in the process of
download. We need to update mFilePathsToItemsMap map which is used during
deletion to find all the items using the same file path.

BUG=696830

Review-Url: https://codereview.chromium.org/2721803002
Cr-Commit-Position: refs/heads/master@{#453719}
Committed:
https://chromium.googlesource.com/chromium/src/+/335b9f4cc30ccbeeb8c40e3ad6b7...
==========

[commit-bot: I haz the power, 2017-02-28 22:09:20]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/335b9f4cc30ccbeeb8c40e3ad6b7...