Add cross origin to Blink-driven preconnect

chrome/browser/net/predictor.h

Index: chrome/browser/net/predictor.h
diff --git a/chrome/browser/net/predictor.h b/chrome/browser/net/predictor.h
index d1983e36f486e844e31edfd0ec5453cd35796fae..24fd70cbbc38ad21688f729d1d7ee14a5d65f027 100644
--- a/chrome/browser/net/predictor.h
+++ b/chrome/browser/net/predictor.h
@@ -243,12 +243,14 @@ class Predictor {
   // May be called from either the IO or UI thread and will PostTask
   // to the IO thread if necessary.

[Ryan Sleevi, 2015/06/11 22:20:15]

Should update documentation for these methods to reflect parameters.

   void PreconnectUrl(const GURL& url, const GURL& first_party_for_cookies,
-                     UrlInfo::ResolutionMotivation motivation, int count);
+                     UrlInfo::ResolutionMotivation motivation, int count,
+                     bool is_credentials_flag_set = true);

[Ryan Sleevi, 2015/06/11 22:20:15]

STYLE: Still with the defaults here; can't do this in Chrome code (
http://google-styleguide.googlecode.com/svn/trunk/cppguide.html#Default_Argum...
)

[Ryan Sleevi, 2015/06/11 22:20:15]

NAMING: Same concerns re: is_

 
   void PreconnectUrlOnIOThread(const GURL& url,
                                const GURL& first_party_for_cookies,
                                UrlInfo::ResolutionMotivation motivation,
-                               int count);
+                               int count,
+                               bool is_credentials_flag_set = true);
 
   // ------------- End IO thread methods.