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

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

Index: chrome/test/data/extensions/extension_throttle/background.js
diff --git a/chrome/test/data/extensions/extension_throttle/background.js b/chrome/test/data/extensions/extension_throttle/background.js
index 52c8e340e0c6c8317bbe9d32311d549dee751ca0..3fe16cd7df404b84f15b943039ec2853ec55e8b5 100644
--- a/chrome/test/data/extensions/extension_throttle/background.js
+++ b/chrome/test/data/extensions/extension_throttle/background.js
@@ -6,6 +6,7 @@ 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));