Make ResourceFetcher return Resources with LoadError instead of nullptrs.

third_party/WebKit/Source/core/loader/DocumentLoader.cpp

 /*
  * Copyright (C) 2006, 2007, 2008 Apple Inc. All rights reserved.
  * 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:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
@@ skipping 181 matching lines @@
     case Resource::ImportResource:
         resource = RawResource::fetchImport(request, fetcher());
         break;
     case Resource::Raw:
         resource = RawResource::fetch(request, fetcher());
         break;
     default:
         NOTREACHED();
     }
 
-    if (resource)
+    if (resource && !resource->resourceError().isAccessCheck())

[Nate Chapin, 2016/09/22 18:30:44]

Here and in XMLHttpRequest.cpp, we should probably comment why this only applies
to isAccessCheck() errors.

[engedy, 2016/09/30 16:03:15]

Done. Please let me know if the information I put in the comments matches what
you wanted to convey.

         fetcher()->preloadStarted(resource);
     return resource;
 }
 
 void DocumentLoader::didRedirect(const KURL& oldURL, const KURL& newURL)
 {
     timing().addRedirect(oldURL, newURL);
 
     // If a redirection happens during a back/forward navigation, don't restore
     // any state from the old HistoryItem.
@@ skipping 507 matching lines @@
 {
     m_writer = createWriterFor(init, mimeType(), m_writer ? m_writer->encoding() : emptyAtom, true, ForceSynchronousParsing);
     if (!source.isNull())
         m_writer->appendReplacingData(source);
     endWriting(m_writer.get());
 }
 
 DEFINE_WEAK_IDENTIFIER_MAP(DocumentLoader);
 
 } // namespace blink