Switch remaining functions from SchemeIsSecure() to SchemeIsCryptographic().

net/cookies/canonical_cookie.cc

Index: net/cookies/canonical_cookie.cc
diff --git a/net/cookies/canonical_cookie.cc b/net/cookies/canonical_cookie.cc
index 5932c23a8818ef4cf6e0f60983f9694d246c8b78..df97780fe4547af6b120eaf37b7353d63f4acb70 100644
--- a/net/cookies/canonical_cookie.cc
+++ b/net/cookies/canonical_cookie.cc
@@ -187,7 +187,7 @@ std::string CanonicalCookie::GetCookieSourceFromURL(const GURL& url) {
 
   url::Replacements<char> replacements;
   replacements.ClearPort();
-  if (url.SchemeIsSecure())
+  if (url.SchemeIsCryptographic())
     replacements.SetScheme("http", url::Component(0, 4));
 
   return url.GetOrigin().ReplaceComponents(replacements).spec();
@@ -395,7 +395,7 @@ bool CanonicalCookie::IncludeForRequestURL(const GURL& url,
     return false;
   // Secure cookies should not be included in requests for URLs with an
   // insecure scheme.
-  if (IsSecure() && !url.SchemeIsSecure())
+  if (IsSecure() && !url.SchemeIsCryptographic())

[palmer, 2015/04/24 19:55:11]

This is a semantic change. It might be a change from the right semantics to the
wrong semantics (in which case we don't want to do it), or from the wrong
semantics to the right semantics (in which case this change fixes a bug
— possibly already filed, possibly not). "git blame" shows the owner of this
code is markusheintz; let's ask him.

     return false;
   // Don't include cookies for requests that don't apply to the cookie domain.
   if (!IsDomainMatch(url.host()))