Use WeakPtr and do not use pointer equality in WebURLLoaderMockFactory

content/test/weburl_loader_mock.h

 // Copyright 2013 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.
 
 #ifndef CONTENT_TEST_WEBURL_LOADER_MOCK_H_
 #define CONTENT_TEST_WEBURL_LOADER_MOCK_H_
 
 #include "base/macros.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/weak_ptr.h"
@@ skipping 36 matching lines @@
   void loadSynchronously(const blink::WebURLRequest& request,
                          blink::WebURLResponse& response,
                          blink::WebURLError& error,
                          blink::WebData& data) override;
   void loadAsynchronously(const blink::WebURLRequest& request,
                           blink::WebURLLoaderClient* client) override;
   void cancel() override;
   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_;
   scoped_ptr<blink::WebURLLoader> default_loader_;
   bool using_default_loader_;
   bool is_deferred_;
 
   base::WeakPtrFactory<WebURLLoaderMock> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(WebURLLoaderMock);
 };
 
 #endif  // CONTENT_TEST_WEBURL_LOADER_MOCK_H_