Revert of Make ResourceFetcher return Resources with LoadError instead of nullptrs.

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

Index: third_party/WebKit/Source/core/fetch/Resource.cpp
diff --git a/third_party/WebKit/Source/core/fetch/Resource.cpp b/third_party/WebKit/Source/core/fetch/Resource.cpp
index 79a9db08045f768b9b21e680e7c13e3ce01007cb..932a38e5481f5917c7368b44d0b1b5394b12628c 100644
--- a/third_party/WebKit/Source/core/fetch/Resource.cpp
+++ b/third_party/WebKit/Source/core/fetch/Resource.cpp
@@ -645,7 +645,7 @@
   }
 }
 
-static bool typeNeedsSynchronousCacheHit(Resource::Type type) {
+static bool shouldSendCachedDataSynchronouslyForType(Resource::Type type) {
   // Some resources types default to return data synchronously. For most of
   // these, it's because there are layout tests that expect data to return
   // synchronously in case of cache hit. In the case of fonts, there was a
@@ -695,10 +695,11 @@
     return;
   }
 
-  // If an error has occurred or we have existing data to send to the new client
-  // and the resource type supprts it, send it asynchronously.
-  if ((errorOccurred() || !m_response.isNull()) &&
-      !typeNeedsSynchronousCacheHit(getType()) && !m_needsSynchronousCacheHit) {
+  // If we have existing data to send to the new client and the resource type
+  // supprts it, send it asynchronously.
+  if (!m_response.isNull() &&
+      !shouldSendCachedDataSynchronouslyForType(getType()) &&
+      !m_needsSynchronousCacheHit) {
     m_clientsAwaitingCallback.add(client);
     ResourceCallback::callbackHandler().schedule(this);
     return;