PlzNavigate: Add first version of NavigationURLLoader.

content/browser/loader/navigation_url_loader_impl.h

+// Copyright 2014 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_BROWSER_LOADER_NAVIGATION_URL_LOADER_IMPL_H_
+#define CONTENT_BROWSER_LOADER_NAVIGATION_URL_LOADER_IMPL_H_
+
+#include "base/macros.h"
+#include "base/memory/scoped_ptr.h"
+#include "base/memory/weak_ptr.h"
+#include "content/browser/loader/navigation_url_loader.h"
+
+namespace net {
+struct RedirectInfo;
+}
+
+namespace content {
+
+class NavigationURLLoaderImplCore;
+class StreamHandle;
+struct ResourceResponse;
+
+class NavigationURLLoaderImpl : public NavigationURLLoader {
+ public:
+  // The caller is responsible for ensuring that |delegate| outlives the loader.
+  NavigationURLLoaderImpl(BrowserContext* browser_context,
+                          int64 frame_tree_node_id,
+                          const CommonNavigationParams& common_params,
+                          scoped_ptr<NavigationRequestInfo> request_info,
+                          ResourceRequestBody* request_body,
+                          NavigationURLLoaderDelegate* delegate);
+  ~NavigationURLLoaderImpl() override;
+
+  // Called in response to OnRequestRedirected to continue processing the
+  // request.
+  void FollowRedirect() override;
+
+ private:
+  friend class NavigationURLLoaderImplCore;
+
+  // Notifies the delegate of a redirect.
+  void NotifyRequestRedirected(const net::RedirectInfo& redirect_info,
+                               ResourceResponse* response);
+
+  // Notifies the delegate that the response has started.
+  void NotifyResponseStarted(ResourceResponse* response,
+                             scoped_ptr<StreamHandle> body);
+
+  // Notifies the delegate the request failed to return a response.
+  void NotifyRequestFailed(int net_error);
+
+  NavigationURLLoaderDelegate *delegate_;

[nasko, 2014/10/23 22:36:17]

* goes with the type, not the variable.

[davidben, 2014/10/27 20:46:04]

Done.

+
+  // |core_| is deleted on the IO thread in a subsequent task when the
+  // |NavigationURLLoaderImpl goes out of scope.

[nasko, 2014/10/23 22:36:16]

Drop the | before NavigationURLLoaderImpl, as it is not a parameter name, but
rather a type.

[davidben, 2014/10/27 20:46:04]

Done. I think emacs M-q tried to be clever...

+  NavigationURLLoaderImplCore* core_;
+
+  base::WeakPtrFactory<NavigationURLLoaderImpl> weak_factory_;
+
+  DISALLOW_COPY_AND_ASSIGN(NavigationURLLoaderImpl);
+};
+
+}  // namespace content
+
+#endif  // CONTENT_BROWSER_LOADER_NAVIGATION_URL_LOADER_IMPL_H_