Commit c0015aa9 authored by Olivier Robin's avatar Olivier Robin Committed by Chromium LUCI CQ

Use DistilledURL when loading distilled data

When using loadData, the html has access to the cookies of the URL.
It is necessary to pass the URL of the distilled page to avoid cross
domain leak.

This CL breaks the workflow of user following a new redirection on
reloading the page.

Bug: 1167901
Change-Id: I6cb01a6580ed511de19f55b35acf91f7f3c7493d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2637584
Auto-Submit: Olivier Robin <olivierrobin@chromium.org>
Commit-Queue: Ken Buchanan <kenrb@chromium.org>
Reviewed-by: default avatarKen Buchanan <kenrb@chromium.org>
Cr-Commit-Position: refs/heads/master@{#844982}
parent f2570ac7
......@@ -296,7 +296,7 @@ void OfflinePageTabHelper::PresentOfflinePageForOnlineUrl(const GURL& url) {
base::BindOnce(&GetOfflineData, offline_root, offline_path),
base::BindOnce(&OfflinePageTabHelper::LoadData,
weak_factory_.GetWeakPtr(), last_navigation_started_,
entry_url, offline_path.Extension()));
entry->DistilledURL(), offline_path.Extension()));
}
bool OfflinePageTabHelper::HasDistilledVersionForOnlineUrl(
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment