For Offliner, pass in the URL from WebContents to SavePage request instead of the request URL since…

For Offliner, pass in the URL from WebContents to SavePage request instead of the request URL since redirect link will fail a URL check in OfflinePageModelImpl.
This is workaround until better understand how to deal with redirected URLs for async loading.

BUG=618716
Committed: https://crrev.com/e759fe98d6b70f487d5f5474b00703d1ba492d53
Cr-Commit-Position: refs/heads/master@{#399001}

[dougarnett, 2016-06-09 15:46:18]

Description was changed from

==========
For Offliner, pass in the URL from WebContents to SavePage request instead of
the request URL since redirect link will fail a URL check in
OfflinePageModelImpl.
This is workaround until better understand how to deal with redirected URLs for
async loading.

webcontentsurl

BUG=618716
==========

to

==========
For Offliner, pass in the URL from WebContents to SavePage request instead of
the request URL since redirect link will fail a URL check in
OfflinePageModelImpl.
This is workaround until better understand how to deal with redirected URLs for
async loading.


BUG=618716
==========

[dougarnett, 2016-06-09 15:46:18]

dougarnett@chromium.org changed reviewers:
+ fgorski@chromium.org, petewil@chromium.org

[dougarnett, 2016-06-09 15:47:41]

Pete, here is another CL to patch to help more links works (although you could
carefully choose specific link to demo).

[Pete Williamson, 2016-06-09 16:52:56]

lgtm

[fgorski, 2016-06-09 17:13:55]

lgtm

[dougarnett, 2016-06-09 20:29:33]

The CQ bit was checked by dougarnett@chromium.org

[commit-bot: I haz the power, 2016-06-09 20:30:05]

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

[commit-bot: I haz the power, 2016-06-09 20:45:28]

Description was changed from

==========
For Offliner, pass in the URL from WebContents to SavePage request instead of
the request URL since redirect link will fail a URL check in
OfflinePageModelImpl.
This is workaround until better understand how to deal with redirected URLs for
async loading.


BUG=618716
==========

to

==========
For Offliner, pass in the URL from WebContents to SavePage request instead of
the request URL since redirect link will fail a URL check in
OfflinePageModelImpl.
This is workaround until better understand how to deal with redirected URLs for
async loading.


BUG=618716
==========

[commit-bot: I haz the power, 2016-06-09 20:45:29]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-09 20:45:35]

CQ bit was unchecked

[commit-bot: I haz the power, 2016-06-09 20:47:08]

Description was changed from

==========
For Offliner, pass in the URL from WebContents to SavePage request instead of
the request URL since redirect link will fail a URL check in
OfflinePageModelImpl.
This is workaround until better understand how to deal with redirected URLs for
async loading.


BUG=618716
==========

to

==========
For Offliner, pass in the URL from WebContents to SavePage request instead of
the request URL since redirect link will fail a URL check in
OfflinePageModelImpl.
This is workaround until better understand how to deal with redirected URLs for
async loading.

BUG=618716

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

[commit-bot: I haz the power, 2016-06-09 20:47:09]

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