Use WeakPtr and do not use pointer equality in WebURLLoaderMockFactory

content/test/weburl_loader_mock.h

Index: content/test/weburl_loader_mock.h
diff --git a/content/test/weburl_loader_mock.h b/content/test/weburl_loader_mock.h
index 81a8e187bd9aff0cc648f366b23ecf3b96080716..465d6282725c9b037c47f5635df9b025f6947f67 100644
--- a/content/test/weburl_loader_mock.h
+++ b/content/test/weburl_loader_mock.h
@@ -54,9 +54,12 @@ class WebURLLoaderMock : public blink::WebURLLoader {
   void setDefersLoading(bool defer) override;
 
   bool isDeferred() { return is_deferred_; }
+  bool isCancelled() { return !client_; }

[jam, 2016/02/29 16:41:09]

nit: these two should be is_deferred() and is_cancelled() per google style guide

[hiroshige, 2016/02/29 21:21:55]

Done.

 
   void setLoadingTaskRunner(blink::WebTaskRunner*) override;
 
+  base::WeakPtr<WebURLLoaderMock> GetWeakPtr();
+
  private:
   WebURLLoaderMockFactory* factory_;
   blink::WebURLLoaderClient* client_;