Refactor net predictor to use ResourceThrottles

chrome/browser/renderer_host/predictor_resource_throttle.h

Index: chrome/browser/renderer_host/predictor_resource_throttle.h
diff --git a/chrome/browser/renderer_host/predictor_resource_throttle.h b/chrome/browser/renderer_host/predictor_resource_throttle.h
new file mode 100644
index 0000000000000000000000000000000000000000..583788d47714893e61257f25f986440273bd9cbe
--- /dev/null
+++ b/chrome/browser/renderer_host/predictor_resource_throttle.h
@@ -0,0 +1,29 @@
+// Copyright 2016 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 CHROME_BROWSER_RENDERER_HOST_PREDICTOR_RESOURCE_THROTTLE_H_
+#define CHROME_BROWSER_RENDERER_HOST_PREDICTOR_RESOURCE_THROTTLE_H_
+
+#include "content/public/browser/resource_throttle.h"
+#include "chrome/browser/net/predictor.h"
+#include "net/url_request/url_request.h"

[mmenke, 2016/04/07 22:12:50]

Think these two can be forward declared. Same with RedirectInfo.

[Charlie Harrison, 2016/05/23 14:16:20]

Done.

+
+class PredictorResourceThrottle : public content::ResourceThrottle {

[mmenke, 2016/04/07 22:12:50]

Hrm...The predictor itself is in net/, wonder if this should be as well.  That
does introduce a dependency on loader/ in net, but the predictor itself does
depend on a fair number of things it probably shouldn't.  Perhaps its' the
predictor which doesn't belong in there.

[mmenke, 2016/04/07 22:12:50]

This should have some docs.

[Charlie Harrison, 2016/04/07 22:20:41]

I agree that the predictor seems like it does not truly belong in net. I can
look into moving it over here.

[Charlie Harrison, 2016/05/23 14:16:20]

I've added a TODO.

[Charlie Harrison, 2016/05/23 14:16:20]

Done.

+ public:
+  PredictorResourceThrottle(net::URLRequest* request,
+                            chrome_browser_net::Predictor* predictor);
+  ~PredictorResourceThrottle() override;
+
+  // content::ResourceThrottle:
+  void WillStartRequest(bool* defer) override;
+  void WillRedirectRequest(const net::RedirectInfo& redirect_info,
+                           bool* defer) override;
+  const char* GetNameForLogging() const override;
+
+ private:
+  net::URLRequest* request_;
+  chrome_browser_net::Predictor* predictor_;

[mmenke, 2016/04/07 22:12:50]

DISALLOW_COPY_AND_ASSIGN, and include base/macros.h.

[Charlie Harrison, 2016/05/23 14:16:20]

Done.

+};
+
+#endif  // CHROME_BROWSER_RENDERER_HOST_PREDICTOR_RESOURCE_THROTTLE_H_