Cross origin requests to same origin should match

third_party/WebKit/Source/core/fetch/FetchRequest.cpp

 /*
  * Copyright (C) 2012 Google, Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 55 matching lines @@
 {
     m_options.initiatorInfo = initiator;
 }
 
 FetchRequest::~FetchRequest()
 {
 }
 
 void FetchRequest::setCrossOriginAccessControl(SecurityOrigin* origin, CrossOriginAttributeValue crossOrigin)
 {
-    ASSERT(crossOrigin != CrossOriginAttributeNotSet);
     const bool useCredentials = crossOrigin == CrossOriginAttributeUseCredentials;
     const bool isSameOriginRequest = origin && origin->canRequestNoSuborigin(m_resourceRequest.url());
+    const CORSEnabled corsEnabled = (crossOrigin != CrossOriginAttributeNotSet) ? IsCORSEnabled : NotCORSEnabled;
 
-    // Currently FetchRequestMode and FetchCredentialsMode are only used when the request goes to Service Worker.
-    m_resourceRequest.setFetchRequestMode(WebURLRequest::FetchRequestModeCORS);
-    m_resourceRequest.setFetchCredentialsMode(useCredentials ? WebURLRequest::FetchCredentialsModeInclude : WebURLRequest::FetchCredentialsModeSameOrigin);
 
-    m_options.allowCredentials = (isSameOriginRequest || useCredentials) ? AllowStoredCredentials : DoNotAllowStoredCredentials;
-    m_options.corsEnabled = IsCORSEnabled;
-    m_options.securityOrigin = origin;
-    m_options.credentialsRequested = useCredentials ? ClientRequestedCredentials : ClientDidNotRequestCredentials;
-
-    updateRequestForAccessControl(m_resourceRequest, origin, m_options.allowCredentials);
+    m_options.sameOrigin = isSameOriginRequest ? IsSameOrigin : NotSameOrigin;

[Nate Chapin, 2016/03/17 22:58:18]

Instead of having this sameOrigin property, why not check
canRequestNoSuborigin() when needed?

[Yoav Weiss, 2016/03/18 07:53:14]

Because we don't necessarily have the securityOrigin nor the URL in
canReuseRequest right now. But I guess we could set the origin here and pipe the
URL in from the callers.

+    m_options.corsEnabled = corsEnabled;
+    if (corsEnabled) {
+        // Currently FetchRequestMode and FetchCredentialsMode are only used when the request goes to Service Worker.
+        m_resourceRequest.setFetchRequestMode(WebURLRequest::FetchRequestModeCORS);
+        m_resourceRequest.setFetchCredentialsMode(useCredentials ? WebURLRequest::FetchCredentialsModeInclude : WebURLRequest::FetchCredentialsModeSameOrigin);
+        m_options.allowCredentials = (isSameOriginRequest || useCredentials) ? AllowStoredCredentials : DoNotAllowStoredCredentials;
+        m_options.securityOrigin = origin;
+        m_options.credentialsRequested = useCredentials ? ClientRequestedCredentials : ClientDidNotRequestCredentials;
+        updateRequestForAccessControl(m_resourceRequest, origin, m_options.allowCredentials);
+    }
 }
 
-
 void FetchRequest::setResourceWidth(ResourceWidth resourceWidth)
 {
     if (resourceWidth.isSet) {
         m_resourceWidth.width = resourceWidth.width;
         m_resourceWidth.isSet = true;
     }
 }
 
 } // namespace blink