Add cross origin to Blink-driven preconnect

components/network_hints/renderer/renderer_preconnect.cc

Index: components/network_hints/renderer/renderer_preconnect.cc
diff --git a/components/network_hints/renderer/renderer_preconnect.cc b/components/network_hints/renderer/renderer_preconnect.cc
index 41d8033478e7c58ad2954ff81951f05e6ae9b9a3..da770caf91df112c4c46da7bdcf69d1bd9d4ebd4 100644
--- a/components/network_hints/renderer/renderer_preconnect.cc
+++ b/components/network_hints/renderer/renderer_preconnect.cc
@@ -20,12 +20,14 @@ RendererPreconnect::RendererPreconnect() {
 RendererPreconnect::~RendererPreconnect() {
 }
 
-void RendererPreconnect::Preconnect(const GURL &url) {
+void RendererPreconnect::Preconnect(const GURL &url,
+    blink::CrossOriginAttributeValue crossOrigin) {
   if (!url.is_valid())
     return;
 
+  bool isAnonymous = (crossOrigin == blink::CrossOriginAttributeAnonymous);

[Ryan Sleevi, 2015/06/10 19:30:42]

This isn't really correct, is it? You should be gating it on whether or not the
CORS credentials flag is set, which happens to be not set in anonymous mode.

[Ryan Sleevi, 2015/06/11 18:30:09]

Yoav wrote:
No, I mean this code should not at all be dealing with CrossOriginAttribute.
That's not the pivot point for the API, as reflected in Fetch.

https://fetch.spec.whatwg.org/#http-network-fetch

"HTTP requests should be grouped based on the credentials flag. HTTP requests
with the credentials flag set should not share anything about the connection or
TLS particulars (such as session identifiers) with HTTP requests that have the
credentials flag unset. In other words, ambient authority should not leak across
this grouping."

The CrossOriginAttribute is just Blink's way of tracking that Fetch attribute,
but the algorithm in 4.3 establishes

"Let credentials flag be set if either request's credentials mode is "include",
or request's credentials mode is "same-origin" and the CORS flag is unset, and
unset otherwise."

I'm saying that the distinction about "isAnonymous" is wrong. For preconnect (or
for prefetch, for that matter), the "which pool to go to" is keyed on the
credentials flag.

   RenderThread::Get()->Send(
-      new NetworkHintsMsg_Preconnect(url, 1));
+      new NetworkHintsMsg_Preconnect(url, isAnonymous, 1));
 }
 
 }  // namespace network_hints