[XHR] Tidy up files for testing XHR's error for mixed content check failure

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

Index: third_party/WebKit/Source/core/loader/DocumentThreadableLoader.cpp
diff --git a/third_party/WebKit/Source/core/loader/DocumentThreadableLoader.cpp b/third_party/WebKit/Source/core/loader/DocumentThreadableLoader.cpp
index b23c717a45f319e27090a277e3120d419010aff0..635b9209da8a8940ddfb7c2e90e9e68186fc574a 100644
--- a/third_party/WebKit/Source/core/loader/DocumentThreadableLoader.cpp
+++ b/third_party/WebKit/Source/core/loader/DocumentThreadableLoader.cpp
@@ -838,7 +838,7 @@ void DocumentThreadableLoader::loadRequest(const ResourceRequest& request, Resou
         if (!resource()) {
             ThreadableLoaderClient* client = m_client;
             clear();
-            client->didFail(ResourceError(errorDomainBlinkInternal, 0, requestURL.string(), "Failed to start loading."));
+            client->didFail(ResourceError(errorDomainBlinkInternal, 0, requestURL.string(), "Failed to start loading"));

[hiroshige, 2016/02/25 16:45:07]

nit: should we remove period in the messages?
Some of other messages do contain period (e.g. Line 264), while the others don't
(e.g. "Load cancelled").
Probably it's better to make other messages also consistent, or just keep
existing messages as-is (including this) if we think this is really nit and feel
no need to complicate blame information.

[tyoshino (SeeGerritForStatus), 2016/02/25 23:33:06]

Woo. Sorry for my bad. I just failed to revert it completely. I'll readd it
before committing.

             // |this| may be dead here.
             return;
         }
@@ -847,6 +847,7 @@ void DocumentThreadableLoader::loadRequest(const ResourceRequest& request, Resou
             unsigned long identifier = resource()->identifier();
             InspectorInstrumentation::documentThreadableLoaderStartedLoadingForClient(m_document, identifier, m_client);
         }
+
         return;
     }