Set the request mode and the credentials mode even if the request will not go to ServiceWorker.

third_party/WebKit/Source/web/AssociatedURLLoaderTest.cpp

 /*
  * Copyright (C) 2011 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:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 303 matching lines @@
     WebURLRequest request;
     request.initialize();
     request.setURL(url);
     CheckFails(request);
 }
 
 // Test a successful cross-origin load.
 TEST_F(AssociatedURLLoaderTest, CrossOriginSuccess)
 {
     // This is cross-origin since the frame was loaded from www.test.com.
-    KURL url = toKURL("http://www.other.com/CrossOriginSuccess.html");
+    KURL url = toKURL("http://www.other.com/CrossOriginSuccess");

[tyoshino (SeeGerritForStatus), 2016/05/16 07:12:42]

What's this change for?

[horo, 2016/05/16 07:23:15]

I think it is weird to send the Script context request to ".html" URL.

     WebURLRequest request;
     request.initialize();
     request.setURL(url);
+    // No-CORS requests (CrossOriginRequestPolicyAllow) aren't allowed for the
+    // default context requests. So we set the context as Script here.

[tyoshino (SeeGerritForStatus), 2016/05/16 07:12:42]

"request" is duplicated.

No-CORS is not allowed for the default context requests

or

No-CORS requests aren't allowed for the default context

?

[horo, 2016/05/16 07:23:15]

Done.

+    request.setRequestContext(WebURLRequest::RequestContextScript);
 
     m_expectedResponse = WebURLResponse();
     m_expectedResponse.initialize();
     m_expectedResponse.setMIMEType("text/html");
     m_expectedResponse.setHTTPStatusCode(200);
     Platform::current()->getURLLoaderMockFactory()->registerURL(url, m_expectedResponse, m_frameFilePath);
 
     WebURLLoaderOptions options;
     options.crossOriginRequestPolicy = WebURLLoaderOptions::CrossOriginRequestPolicyAllow;
     m_expectedLoader = createAssociatedURLLoader(options);
@@ skipping 371 matching lines @@
     m_expectedLoader->loadAsynchronously(request, this);
     serveRequests();
     EXPECT_TRUE(m_didReceiveResponse);
     EXPECT_TRUE(m_didReceiveData);
     EXPECT_TRUE(m_didFinishLoading);
 
     EXPECT_FALSE(m_actualResponse.httpHeaderField(headerNameString).isEmpty());
 }
 
 } // namespace blink