Stop using ResourceController in ResourceThrottle

content/public/browser/resource_throttle.h

 // Copyright (c) 2012 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_PUBLIC_BROWSER_RESOURCE_THROTTLE_H_
 #define CONTENT_PUBLIC_BROWSER_RESOURCE_THROTTLE_H_
 
 #include "content/common/content_export.h"
 
 namespace net {
 struct RedirectInfo;
 }
 
 namespace content {
 
 class AsyncRevalidationDriver;
 class ResourceController;
 class ThrottlingResourceHandler;
 
 // A ResourceThrottle gets notified at various points during the process of
 // loading a resource.  At each stage, it has the opportunity to defer the
 // resource load.  The ResourceController interface may be used to resume a
 // deferred resource load, or it may be used to cancel a resource load at any
 // time.
 class CONTENT_EXPORT ResourceThrottle {
  public:
+  class CONTENT_EXPORT Delegate {
+   public:
+    virtual void Cancel() = 0;

[jochen (gone - plz use gerrit), 2016/12/05 15:25:03]

please add comments explaining the delegate and every virtual method

[tyoshino (SeeGerritForStatus), 2016/12/06 11:25:31]

Done.

+    virtual void CancelAndIgnore() = 0;
+    virtual void CancelWithError(int error_code) = 0;
+    virtual void Resume() = 0;
+
+   protected:
+    virtual ~Delegate() {}
+  };
+
   virtual ~ResourceThrottle() {}
 
   // Called before the resource request is started.
   virtual void WillStartRequest(bool* defer) {}
 
   // Called when the request was redirected.  |redirect_info| contains the
   // redirect responses's HTTP status code and some information about the new
   // request that will be sent if the redirect is followed, including the new
   // URL and new method.
   virtual void WillRedirectRequest(const net::RedirectInfo& redirect_info,
                                    bool* defer) {}
 
   // Called when the response headers and meta data are available.
   virtual void WillProcessResponse(bool* defer) {}
 
   // Returns the name of the throttle, as a UTF-8 C-string, for logging
   // purposes.  nullptr is not allowed.  Caller does *not* take ownership of the
   // returned string.
   virtual const char* GetNameForLogging() const = 0;
 
   // Whether this ResourceThrottle needs to execute WillProcessResponse before
   // any part of the response body is read. Normally this is false. This should
   // be set to true if the ResourceThrottle wants to ensure that no part of the
   // response body will be cached if the request is canceled in
   // WillProcessResponse.
   virtual bool MustProcessResponseBeforeReadingBody();
 
-  void set_controller_for_testing(ResourceController* c) {
-    controller_ = c;
-  }
+  void set_delegate_for_testing(Delegate* delegate) { delegate_ = delegate; }
 
  protected:
-  ResourceThrottle() : controller_(nullptr) {}
-  ResourceController* controller() { return controller_; }
+  ResourceThrottle() : delegate_(nullptr) {}
+
+  void Cancel();

[jochen (gone - plz use gerrit), 2016/12/05 15:25:03]

please add comments here as well

[tyoshino (SeeGerritForStatus), 2016/12/06 11:25:31]

Done.

+  void CancelAndIgnore();
+  void CancelWithError(int error_code);
+  void Resume();
 
  private:
   friend class AsyncRevalidationDriver;
   friend class ThrottlingResourceHandler;
-  void set_controller(ResourceController* c) { controller_ = c; }
+  void set_delegate(Delegate* delegate) { delegate_ = delegate; }
 
-  ResourceController* controller_;
+  Delegate* delegate_;
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_RESOURCE_THROTTLE_H_