Revert "Do not suppress referrers for '<a ... rel="noopener">'." on M52 branch

Revert "Do not suppress referrers for '<a ... rel="noopener">'." on M52 branch

The current implementation of 'noopener' correctly split the referrer
and opener states in Blink, but missed a spot higher up the stack where
the two were still conflated. This patch fixes that issue, adds tests
to ensure we don't regress, and refactors the Blink-side code a bit so
that we're no longer passing the data around twice.

BUG=618532, 619856
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation

Committed: https://chromium.googlesource.com/chromium/src/+/f7939c2fff08f8f0d2700eea46ef94c57a985166

[dmazzoni, 2016-06-14 20:12:24]

Description was changed from

==========
Revert "Do not suppress referrers for '<a ... rel="noopener">'." on M52 branch

The current implementation of 'noopener' correctly split the referrer
and opener states in Blink, but missed a spot higher up the stack where
the two were still conflated. This patch fixes that issue, adds tests
to ensure we don't regress, and refactors the Blink-side code a bit so
that we're no longer passing the data around twice.

BUG=618532
==========

to

==========
Revert "Do not suppress referrers for '<a ... rel="noopener">'." on M52 branch

The current implementation of 'noopener' correctly split the referrer
and opener states in Blink, but missed a spot higher up the stack where
the two were still conflated. This patch fixes that issue, adds tests
to ensure we don't regress, and refactors the Blink-side code a bit so
that we're no longer passing the data around twice.

BUG=618532
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
==========

[dmazzoni, 2016-06-14 20:12:53]

Description was changed from

==========
Revert "Do not suppress referrers for '<a ... rel="noopener">'." on M52 branch

The current implementation of 'noopener' correctly split the referrer
and opener states in Blink, but missed a spot higher up the stack where
the two were still conflated. This patch fixes that issue, adds tests
to ensure we don't regress, and refactors the Blink-side code a bit so
that we're no longer passing the data around twice.

BUG=618532
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
==========

to

==========
Revert "Do not suppress referrers for '<a ... rel="noopener">'." on M52 branch

The current implementation of 'noopener' correctly split the referrer
and opener states in Blink, but missed a spot higher up the stack where
the two were still conflated. This patch fixes that issue, adds tests
to ensure we don't regress, and refactors the Blink-side code a bit so
that we're no longer passing the data around twice.

BUG=618532,619856
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
==========

[dmazzoni, 2016-06-14 20:12:53]

dmazzoni@chromium.org changed reviewers:
+ hcarmona@chromium.org, mkwst@chromium.org

[dmazzoni, 2016-06-14 20:17:18]

Description was changed from

==========
Revert "Do not suppress referrers for '<a ... rel="noopener">'." on M52 branch

The current implementation of 'noopener' correctly split the referrer
and opener states in Blink, but missed a spot higher up the stack where
the two were still conflated. This patch fixes that issue, adds tests
to ensure we don't regress, and refactors the Blink-side code a bit so
that we're no longer passing the data around twice.

BUG=618532,619856
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
==========

to

==========
Revert "Do not suppress referrers for '<a ... rel="noopener">'." on M52 branch

The current implementation of 'noopener' correctly split the referrer
and opener states in Blink, but missed a spot higher up the stack where
the two were still conflated. This patch fixes that issue, adds tests
to ensure we don't regress, and refactors the Blink-side code a bit so
that we're no longer passing the data around twice.

BUG=618532,619856
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation

Committed:
https://chromium.googlesource.com/chromium/src/+/f7939c2fff08f8f0d2700eea46ef...
==========

[dmazzoni, 2016-06-14 20:17:20]

Committed patchset #1 (id:1) manually as
f7939c2fff08f8f0d2700eea46ef94c57a985166.