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

Side by Side Diff: content/browser/frame_host/navigation_handle_impl.h

Issue 1294243004: PlzNavigate: Make ServiceWorker work with PlzNavigate. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Review comments Created 5 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
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CONTENT_BROWSER_FRAME_HOST_NAVIGATION_HANDLE_IMPL_H_ 5 #ifndef CONTENT_BROWSER_FRAME_HOST_NAVIGATION_HANDLE_IMPL_H_
6 #define CONTENT_BROWSER_FRAME_HOST_NAVIGATION_HANDLE_IMPL_H_ 6 #define CONTENT_BROWSER_FRAME_HOST_NAVIGATION_HANDLE_IMPL_H_
7 7
8 #include "content/public/browser/navigation_handle.h" 8 #include "content/public/browser/navigation_handle.h"
9 9
10 #include "base/macros.h" 10 #include "base/macros.h"
(...skipping 84 matching lines...) Expand 10 before | Expand all | Expand 10 after
95 95
96 // Returns whether the navigation is currently being transferred from one 96 // Returns whether the navigation is currently being transferred from one
97 // RenderFrameHost to another. In particular, a DidStartProvisionalLoad IPC 97 // RenderFrameHost to another. In particular, a DidStartProvisionalLoad IPC
98 // for the navigation URL, received in the new RenderFrameHost, should not 98 // for the navigation URL, received in the new RenderFrameHost, should not
99 // indicate the start of a new navigation in that case. 99 // indicate the start of a new navigation in that case.
100 bool is_transferring() const { return is_transferring_; } 100 bool is_transferring() const { return is_transferring_; }
101 void set_is_transferring(bool is_transferring) { 101 void set_is_transferring(bool is_transferring) {
102 is_transferring_ = is_transferring; 102 is_transferring_ = is_transferring;
103 } 103 }
104 104
105 // PlzNavigate
106 void set_service_worker_provider_id(int service_worker_provider_id) {
107 service_worker_provider_id_ = service_worker_provider_id;
108 }
109
105 // Called when the URLRequest will start in the network stack. 110 // Called when the URLRequest will start in the network stack.
106 NavigationThrottle::ThrottleCheckResult WillStartRequest( 111 NavigationThrottle::ThrottleCheckResult WillStartRequest(
107 bool is_post, 112 bool is_post,
108 const Referrer& sanitized_referrer, 113 const Referrer& sanitized_referrer,
109 bool has_user_gesture, 114 bool has_user_gesture,
110 ui::PageTransition transition, 115 ui::PageTransition transition,
111 bool is_external_protocol); 116 bool is_external_protocol);
112 117
113 // Called when the URLRequest will be redirected in the network stack. 118 // Called when the URLRequest will be redirected in the network stack.
114 NavigationThrottle::ThrottleCheckResult WillRedirectRequest( 119 NavigationThrottle::ThrottleCheckResult WillRedirectRequest(
(...skipping 46 matching lines...) Expand 10 before | Expand all | Expand 10 after
161 // Whether the navigation is in the middle of a transfer. Set to false when 166 // Whether the navigation is in the middle of a transfer. Set to false when
162 // the DidStartProvisionalLoad is received from the new renderer. 167 // the DidStartProvisionalLoad is received from the new renderer.
163 bool is_transferring_; 168 bool is_transferring_;
164 169
165 // The FrameTreeNode this navigation is happening in. 170 // The FrameTreeNode this navigation is happening in.
166 FrameTreeNode* frame_tree_node_; 171 FrameTreeNode* frame_tree_node_;
167 172
168 // A list of Throttles registered for this navigation. 173 // A list of Throttles registered for this navigation.
169 ScopedVector<NavigationThrottle> throttles_; 174 ScopedVector<NavigationThrottle> throttles_;
170 175
176 // PlzNavigate
177 // The ServiceWorkerProviderHost ID used for navigations.
178 // Set to kInvalidServiceWorkerProviderId for sandboxed frames and sync loads.
179 // This parameter is not used in the current navigation architecture, where it
180 // will always be kInvalidServiceWorkerProviderId.
181 int service_worker_provider_id_;
182
171 DISALLOW_COPY_AND_ASSIGN(NavigationHandleImpl); 183 DISALLOW_COPY_AND_ASSIGN(NavigationHandleImpl);
172 }; 184 };
173 185
174 } // namespace content 186 } // namespace content
175 187
176 #endif // CONTENT_BROWSER_FRAME_HOST_NAVIGATION_HANDLE_IMPL_H_ 188 #endif // CONTENT_BROWSER_FRAME_HOST_NAVIGATION_HANDLE_IMPL_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698