[DocumentThreadableLoader] Rename notifyResponseReceived to reportResponseReceived

"notifySomething" style method naming is often used to notify its
client about completion. This is just for inspector and console logging.
It's surprising that we see a function looks like notifying completion
called inside access control failure path. Use reportSomething that
is often used to methods that report to inspector or writing some log.

R=mkwst
BUG=none

Review URL: https://codereview.chromium.org/700413002

git-svn-id: svn://svn.chromium.org/blink/trunk@184905 bbb929c8-8fbe-4397-9dbb-9b2b20218538

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

@@ -380,7 +380,7 @@ void DocumentThreadableLoader::handlePreflightResponse(const ResourceResponse& r
     CrossOriginPreflightResultCache::shared().appendEntry(securityOrigin()->toString(), m_actualRequest->url(), preflightResult.release());
 }
 
-void DocumentThreadableLoader::notifyResponseReceived(unsigned long identifier, const ResourceResponse& response)
+void DocumentThreadableLoader::reportResponseReceived(unsigned long identifier, const ResourceResponse& response)
 {
     DocumentLoader* loader = m_document.frame()->loader().documentLoader();
     TRACE_EVENT_INSTANT1(TRACE_DISABLED_BY_DEFAULT("devtools.timeline"), "ResourceReceiveResponse", "data", InspectorReceiveResponseEvent::data(identifier, m_document.frame(), response));
@@ -395,7 +395,7 @@ void DocumentThreadableLoader::handleResponse(unsigned long identifier, const Re
     ASSERT(m_client);
 
     if (m_actualRequest) {
-        notifyResponseReceived(identifier, response);
+        reportResponseReceived(identifier, response);
         handlePreflightResponse(response);
         return;
     }
@@ -416,7 +416,7 @@ void DocumentThreadableLoader::handleResponse(unsigned long identifier, const Re
     if (!m_sameOriginRequest && m_options.crossOriginRequestPolicy == UseAccessControl) {
         String accessControlErrorDescription;
         if (!passesAccessControlCheck(response, effectiveAllowCredentials(), securityOrigin(), accessControlErrorDescription)) {
-            notifyResponseReceived(identifier, response);
+            reportResponseReceived(identifier, response);
             m_client->didFailAccessControlCheck(ResourceError(errorDomainBlinkInternal, 0, response.url().string(), accessControlErrorDescription));
             return;
         }

third_party/WebKit/Source/core/loader/DocumentThreadableLoader.h

@@ -81,8 +81,9 @@ class DocumentThreadableLoader final : public ThreadableLoader, private Resource
 
         void cancelWithError(const ResourceError&);
 
-        // Notify Inspector about resource response. Use this method if response is not going to be finished normally.
-        void notifyResponseReceived(unsigned long identifier, const ResourceResponse&);
+        // Notify Inspector and log to console about resource response. Use
+        // this method if response is not going to be finished normally.
+        void reportResponseReceived(unsigned long identifier, const ResourceResponse&);
 
         // Methods containing code to handle resource fetch results which is
         // common to both sync and async mode.