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->errorOccurred() && resource->resourceError().isAccessCheck()))

[Nate Chapin, 2016/09/21 23:23:41]

I don't know if this is too cute, but you don't strictly need to check
errorOccurred(). isAccessCheck() will return false if resourceError().isNull(),
so errorOccurred() should be a strictly superset of isAccessCheck().

[engedy, 2016/09/22 13:30:03]

Done.

         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