Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(74)

Side by Side Diff: content/browser/loader/navigation_url_loader.h

Issue 648813002: PlzNavigate: Add first version of NavigationURLLoader. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@plz-navigate-prepare
Patch Set: mmenke comments Created 6 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
(Empty)
1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #ifndef CONTENT_BROWSER_LOADER_NAVIGATION_URL_LOADER_H_
6 #define CONTENT_BROWSER_LOADER_NAVIGATION_URL_LOADER_H_
7
8 #include "base/basictypes.h"
9 #include "base/macros.h"
10 #include "base/memory/scoped_ptr.h"
11 #include "content/common/content_export.h"
12
13 namespace content {
14
15 class BrowserContext;
16 class NavigationURLLoaderDelegate;
17 class NavigationURLLoaderFactory;
18 class ResourceRequestBody;
19 struct CommonNavigationParams;
20 struct NavigationRequestInfo;
21
22 // PlzNavigate: The navigation logic's UI thread entry point into the resource
23 // loading stack. It exposes an interface to control the request prior to
24 // receiving the response. If the NavigationURLLoader is destroyed before
25 // OnResponseStarted is called, the request is aborted.
26 class CONTENT_EXPORT NavigationURLLoader {
27 public:
28 // Creates a NavigationURLLoader. The caller is responsible for ensuring that
29 // |delegate| outlives the loader. |request_body| must not be accessed on the
30 // UI thread after this point.
31 //
32 // TODO(davidben): When navigation is disentangled from the loader, the
33 // request parameters should not come in as a navigation-specific
34 // structure. Information like has_user_gesture and
35 // should_replace_current_entry shouldn't be needed at this layer.
36 static scoped_ptr<NavigationURLLoader> Create(
37 BrowserContext* browser_context,
38 int64 frame_tree_node_id,
39 const CommonNavigationParams& common_params,
40 scoped_ptr<NavigationRequestInfo> request_info,
41 ResourceRequestBody* request_body,
42 NavigationURLLoaderDelegate* delegate);
43
44 // For testing purposes; sets the factory for use in testing.
45 static void SetFactoryForTesting(NavigationURLLoaderFactory* factory);
46
47 virtual ~NavigationURLLoader() {}
48
49 // Called in response to OnRequestRedirected to continue processing the
50 // request.
51 virtual void FollowRedirect() = 0;
52
53 protected:
mmenke 2014/10/23 20:43:40 Doesn't look like you fixed the indent here.
davidben 2014/10/27 20:46:04 Mrrf. Done.
54 NavigationURLLoader() {}
55
56 private:
57 DISALLOW_COPY_AND_ASSIGN(NavigationURLLoader);
58 };
59
60 } // namespace content
61
62 #endif // CONTENT_BROWSER_LOADER_NAVIGATION_URL_LOADER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698