Add <link rel=preconnect> crossorigin attribute (take 2)

Add <link rel=preconnect> crossorigin attribute (take 2)

A second attempt to land https://codereview.chromium.org/1138743002/
In order to support preconnect for both anonymous and non-anonymous connection pools we need to add support for the crossorigin attribute in <link rel=preconnect>.
This CL implements the core functionality of this change on the Blink side.

BUG=468005
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=195236

[Yoav Weiss, 2015-05-12 07:16:33]

yoav@yoav.ws changed reviewers:
+ mkwst@chromium.org

[Yoav Weiss, 2015-05-12 07:16:34]

Hey Mike,

Can you review? The only difference from
https://codereview.chromium.org/1138743002/ is that preconnect calls the older
interface rather than the new (and still empty) one.

[Mike West, 2015-05-12 07:17:46]

On 2015/05/12 at 07:16:34, yoav wrote:
> Hey Mike,
> 
> Can you review? The only difference from
https://codereview.chromium.org/1138743002/ is that preconnect calls the older
interface rather than the new (and still empty) one.

In which file?

[Yoav Weiss, 2015-05-12 07:24:51]

On 2015/05/12 07:17:46, Mike West wrote:
> On 2015/05/12 at 07:16:34, yoav wrote:
> > Hey Mike,
> > 
> > Can you review? The only difference from
> https://codereview.chromium.org/1138743002/ is that preconnect calls the older
> interface rather than the new (and still empty) one.
> 
> In which file?

Source/platform/network/NetworkHints.cpp

[Mike West, 2015-05-12 07:44:48]

LGTM.

[Yoav Weiss, 2015-05-12 07:45:28]

The CQ bit was checked by yoav@yoav.ws

[commit-bot: I haz the power, 2015-05-12 07:45:40]

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

[Yoav Weiss, 2015-05-12 08:38:12]

The CQ bit was unchecked by yoav@yoav.ws

[Yoav Weiss, 2015-05-12 08:58:24]

On 2015/05/12 07:45:40, I haz the power (commit-bot) wrote:
> CQ is trying da patch. Follow status at
>  https://chromium-cq-status.appspot.com/patch-status/1135203002/1

I've changed the call at NetworkHints.cpp to make the Chrome side testable. Can
you take a look to see that it's still good to go?

[Mike West, 2015-05-12 09:08:09]

On 2015/05/12 at 08:58:24, yoav wrote:
> On 2015/05/12 07:45:40, I haz the power (commit-bot) wrote:
> > CQ is trying da patch. Follow status at
> >  https://chromium-cq-status.appspot.com/patch-status/1135203002/1
> 
> I've changed the call at NetworkHints.cpp to make the Chrome side testable.
Can you take a look to see that it's still good to go?

Please add a TODO marking this as temporary; this isn't the state I hope we end
up in. :)

[Yoav Weiss, 2015-05-12 09:42:29]

On 2015/05/12 09:08:09, Mike West wrote:
> On 2015/05/12 at 08:58:24, yoav wrote:
> > On 2015/05/12 07:45:40, I haz the power (commit-bot) wrote:
> > > CQ is trying da patch. Follow status at
> > >  https://chromium-cq-status.appspot.com/patch-status/1135203002/1
> > 
> > I've changed the call at NetworkHints.cpp to make the Chrome side testable.
> Can you take a look to see that it's still good to go?
> 
> Please add a TODO marking this as temporary; this isn't the state I hope we
end
> up in. :)

Added! :)

[Yoav Weiss, 2015-05-12 10:43:13]

The CQ bit was checked by yoav@yoav.ws

[Yoav Weiss, 2015-05-12 10:43:14]

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

[commit-bot: I haz the power, 2015-05-12 10:43:35]

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

[commit-bot: I haz the power, 2015-05-12 10:47:52]

Committed patchset #3 (id:40001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=195236