Clear Resource::m_loader in Resource::finish() and Resource::error()

chrome/test/data/extensions/extension_throttle/background.js

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 chrome.runtime.onMessage.addListener(function(message, sender, sendResponse) {
   if (message.type == "xhr") {
     var xhr = new XMLHttpRequest();
     xhr.open(message.method, message.url);
+    xhr.setRequestHeader("X-Bust-Blink-MemoryCache", Math.random());

[Nate Chapin, 2016/04/27 19:48:30]

This test depends on the memoryCache()->remove() quirk in
Resource::cancelTimerFired() that this patch made inert and removes. The test
doesn't want requests served from blink's MemoryCache, so add a bogus header to
circumvent blink's overly cautious XHR caching policy.

[hiroshige, 2016/05/02 06:39:27]

Could you add a comment that this header is added just to prevent the XHR from
being served by MemoryCache?
(to clarify the intent of this header and that the header name
'X-Bust-Blink-MemoryCache' itself has no meaning)

[Nate Chapin, 2016/05/10 18:26:17]

Yeah, sorry. I was hoping the header-name was clear enough, but overdocumenting
is better than underdocumenting for things like this :)

     xhr.send();
   } else {
     console.error("Unknown message: " + JSON.stringify(message));
   }
 });