Fix an issue that referrer field is no longer exposed to pdfviewer

Fix an issue that referrer field is no longer exposed to pdfviewer

New pdfviewer starts to handle http scheme.
As a result, clicking a pdf link will no longer trigger a download.
Instead, the navigation will be intercepted and passed to pdfviewer.
We previously added the referrer extra for analytics purpose for pdfviewer.
But that only happens in the download code.
This CL fixes the referrer field issue for intercepted navigations.

BUG=603730
Committed: https://crrev.com/30be6d55e09751882232f06ee0db9541d7a201bd
Cr-Commit-Position: refs/heads/master@{#388535}

[qinmin, 2016-04-14 22:09:19]

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

[qinmin, 2016-04-14 22:09:20]

PTAL

[Yaron, 2016-04-14 22:26:37]

Do we need both? Users with old pdf viewer won't get the interception?

[qinmin, 2016-04-14 22:36:46]

On 2016/04/14 22:26:37, Yaron wrote:
> Do we need both? Users with old pdf viewer won't get the interception?

use the download code path will still call into
ExternalNavigationDelegateImpl#resolveIntent(), so this CL should apply to both
situations

[Yaron, 2016-04-15 14:23:55]

On 2016/04/14 22:36:46, qinmin wrote:
> On 2016/04/14 22:26:37, Yaron wrote:
> > Do we need both? Users with old pdf viewer won't get the interception?
> 
> use the download code path will still call into
> ExternalNavigationDelegateImpl#resolveIntent(), so this CL should apply to
both
> situations

lgtm

[qinmin, 2016-04-20 16:49:07]

The CQ bit was checked by qinmin@chromium.org

[commit-bot: I haz the power, 2016-04-20 16:49:29]

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

[commit-bot: I haz the power, 2016-04-20 18:22:59]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-22 19:24:53]

Description was changed from

==========
Fix an issue that referrer field is no longer exposed to pdfviewer

New pdfviewer starts to handle http scheme.
As a result, clicking a pdf link will no longer trigger a download.
Instead, the navigation will be intercepted and passed to pdfviewer.
We previously added the referrer extra for analytics purpose for pdfviewer.
But that only happens in the download code.
This CL fixes the referrer field issue for intercepted navigations.

BUG=603730
==========

to

==========
Fix an issue that referrer field is no longer exposed to pdfviewer

New pdfviewer starts to handle http scheme.
As a result, clicking a pdf link will no longer trigger a download.
Instead, the navigation will be intercepted and passed to pdfviewer.
We previously added the referrer extra for analytics purpose for pdfviewer.
But that only happens in the download code.
This CL fixes the referrer field issue for intercepted navigations.

BUG=603730

Committed: https://crrev.com/30be6d55e09751882232f06ee0db9541d7a201bd
Cr-Commit-Position: refs/heads/master@{#388535}
==========

[commit-bot: I haz the power, 2016-04-22 19:24:53]

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