Clear mojo URLLoaderClient in ResourceDispatcher on cancellation

content/child/resource_dispatcher.cc

Index: content/child/resource_dispatcher.cc
diff --git a/content/child/resource_dispatcher.cc b/content/child/resource_dispatcher.cc
index aa413aeb97706540315d10f869a9a418683dffde..36626e7f8ab302f0a45ecfb64774e14343303caa 100644
--- a/content/child/resource_dispatcher.cc
+++ b/content/child/resource_dispatcher.cc
@@ -453,6 +453,9 @@ bool ResourceDispatcher::RemovePendingRequest(int request_id) {
 
   ReleaseResourcesInMessageQueue(&request_info->deferred_message_queue);
 
+  if (it->second->url_loader_client)
+    it->second->url_loader_client = nullptr;

[yhirano, 2016/10/05 09:03:00]

Would Calling Cancel be enough?

[tzik, 2016/10/06 04:14:02]

As we chatted offline, URLLoaderClient (and URLLoaderClientImpl) doesn't have
Cancel(), and I think it's less cleaner to implement it, since we'll have to
care a IPC message arrival after the cancellation.

[yhirano, 2016/10/06 06:53:05]

Can you run this statement unconditionally?

[tzik, 2016/10/06 12:08:13]

Done.

+
   // Always delete the pending_request asyncly so that cancelling the request
   // doesn't delete the request context info while its response is still being
   // handled.