Cancel asynchronously in allClientsAndObserversRemoved() except for Raw type

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 cf9ef1e62c497b4ebc069d841b93025ee0e89b3c..6c23408758d9af40db04572e91476aa76ebbb761 100644
--- a/third_party/WebKit/Source/core/fetch/Resource.cpp
+++ b/third_party/WebKit/Source/core/fetch/Resource.cpp
@@ -702,7 +702,7 @@ void Resource::allClientsAndObserversRemoved()
 {
     if (!m_loader)
         return;
-    if (m_type == MainResource || m_type == Raw || !memoryCache()->contains(this))
+    if (m_type == Raw)

[Nate Chapin, 2016/05/02 21:17:06]

I assume this case is really still needed? Ideally we'd always cancel
asynchronously.

[hiroshige, 2016/05/03 03:36:48]

I expect we can always cancel asynchronously. However, cancelling Raw type
Resource causes some more test failures and probably it can be fixed in a better
way after you remove |memoryCache()->remove(this);| in
Resource::cancelTimerFired() in https://codereview.chromium.org/1923003002/ and
I make references from Inspector to Resource weak in
https://codereview.chromium.org/1807323002/.

         cancelTimerFired(&m_cancelTimer);
     else if (!m_cancelTimer.isActive())
         m_cancelTimer.startOneShot(0, BLINK_FROM_HERE);