Refactor ResourceHandler API.

content/browser/loader/null_resource_controller.h

Index: content/browser/loader/null_resource_controller.h
diff --git a/content/browser/loader/null_resource_controller.h b/content/browser/loader/null_resource_controller.h
new file mode 100644
index 0000000000000000000000000000000000000000..aebcc004eaef70b716dfa6c387d9c52f5d6d9f53
--- /dev/null
+++ b/content/browser/loader/null_resource_controller.h
@@ -0,0 +1,31 @@
+// Copyright 2016 The Chromium Authors. All rights reserved.

[Charlie Harrison, 2017/01/25 20:23:00]

2017

[mmenke, 2017/01/25 22:07:59]

Done, though not entirely sure what's right - I did write this file in 2016.

+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CONTENT_BROWSER_LOADER_NULL_RESOURCE_CONTROLLER_H_
+#define CONTENT_BROWSER_LOADER_NULL_RESOURCE_CONTROLLER_H_
+
+#include "base/macros.h"
+#include "content/browser/loader/resource_controller.h"
+
+namespace content {
+
+class NullResourceController : public ResourceController {

[Charlie Harrison, 2017/01/25 20:23:00]

A class comment would be nice.

[mmenke, 2017/01/25 22:07:59]

Done, though I'm not sure this class is worthy of a comment.  :)

+ public:
+  explicit NullResourceController(bool* was_resumed);
+  ~NullResourceController() override;
+

[Charlie Harrison, 2017/01/25 20:23:00]

// ResourceController: 
or 
// ResourceController implementation:

[mmenke, 2017/01/25 22:07:59]

Done.

+  void Cancel() override;
+  void CancelAndIgnore() override;
+  void CancelWithError(int error_code) override;
+  void Resume() override;
+
+ private:
+  bool* was_resumed_;
+
+  DISALLOW_COPY_AND_ASSIGN(NullResourceController);
+};
+
+}  // namespace content
+
+#endif  // CONTENT_BROWSER_LOADER_NULL_RESOURCE_CONTROLLER_H_