Popular sites on the NTP: store downloaded file at a fixed path

Popular sites on the NTP: store downloaded file at a fixed path

Before, we'd leak a previously-downloaded file whenever the filename changed.

BUG=516618
Committed: https://crrev.com/76e005e8ceb507becf8c2ae737a0d2ae5e274223
Cr-Commit-Position: refs/heads/master@{#350147}

[Marc Treib, 2015-09-09 16:24:00]

treib@chromium.org changed reviewers:
+ bauerb@chromium.org

[Marc Treib, 2015-09-09 16:24:01]

https://codereview.chromium.org/1306043005/diff/1/chrome/browser/android/popu...
File chrome/browser/android/popular_sites.h (left):

https://codereview.chromium.org/1306043005/diff/1/chrome/browser/android/popu...
chrome/browser/android/popular_sites.h:50: net::URLRequestContextGetter*
request_context,
Unrelated: There's no need to pass this explicitly now that we're passing the
profile.

[Bernhard Bauer, 2015-09-09 16:26:10]

lgtm

[Marc Treib, 2015-09-22 12:41:51]

The CQ bit was checked by treib@chromium.org

[Marc Treib, 2015-09-22 12:41:51]

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

[commit-bot: I haz the power, 2015-09-22 12:42:00]

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

[commit-bot: I haz the power, 2015-09-22 13:40:43]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-09-22 13:41:25]

Patchset 3 (id:??) landed as
https://crrev.com/76e005e8ceb507becf8c2ae737a0d2ae5e274223
Cr-Commit-Position: refs/heads/master@{#350147}