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

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

Issue 2954623003: PlzNavigate: implement REUSE_COMMITTED_OR_PENDING_SITE for redirects (Closed)
Patch Set: Addressed Charlie's comments Created 3 years, 5 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 2014 The Chromium Authors. All rights reserved. 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 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_REQUEST_H_ 5 #ifndef CONTENT_BROWSER_FRAME_HOST_NAVIGATION_REQUEST_H_
6 #define CONTENT_BROWSER_FRAME_HOST_NAVIGATION_REQUEST_H_ 6 #define CONTENT_BROWSER_FRAME_HOST_NAVIGATION_REQUEST_H_
7 7
8 #include <memory> 8 #include <memory>
9 9
10 #include "base/callback_forward.h" 10 #include "base/callback_forward.h"
(...skipping 251 matching lines...) Expand 10 before | Expand all | Expand 10 after
262 int bindings_; 262 int bindings_;
263 int nav_entry_id_ = 0; 263 int nav_entry_id_ = 0;
264 264
265 // Whether the navigation should be sent to a renderer a process. This is 265 // Whether the navigation should be sent to a renderer a process. This is
266 // true, except for 204/205 responses and downloads. 266 // true, except for 204/205 responses and downloads.
267 bool response_should_be_rendered_; 267 bool response_should_be_rendered_;
268 268
269 // The type of SiteInstance associated with this navigation. 269 // The type of SiteInstance associated with this navigation.
270 AssociatedSiteInstanceType associated_site_instance_type_; 270 AssociatedSiteInstanceType associated_site_instance_type_;
271 271
272 // Stores the SiteInstance created on redirects to check if there is an
273 // existing RenderProcessHost that can commit the navigation so that the
274 // renderer process is not deleted while the navigation is ongoing. If the
275 // SiteInstance was a brand new SiteInstance, it is not stored.
276 scoped_refptr<SiteInstance> speculative_site_instance_;
277
272 // Whether the request may be transferred to a different process upon commit. 278 // Whether the request may be transferred to a different process upon commit.
273 // True for browser-initiated navigations and renderer-inititated navigations 279 // True for browser-initiated navigations and renderer-inititated navigations
274 // started via the OpenURL path. 280 // started via the OpenURL path.
275 // Note: the RenderFrameHostManager may still decide to have the navigation 281 // Note: the RenderFrameHostManager may still decide to have the navigation
276 // commit in a different renderer process if it detects that a renderer 282 // commit in a different renderer process if it detects that a renderer
277 // transfer is needed. This is the case in particular when --site-per-process 283 // transfer is needed. This is the case in particular when --site-per-process
278 // is enabled. 284 // is enabled.
279 bool may_transfer_; 285 bool may_transfer_;
280 286
281 std::unique_ptr<NavigationHandleImpl> navigation_handle_; 287 std::unique_ptr<NavigationHandleImpl> navigation_handle_;
(...skipping 12 matching lines...) Expand all
294 mojom::URLLoaderFactoryPtrInfo subresource_loader_factory_info_; 300 mojom::URLLoaderFactoryPtrInfo subresource_loader_factory_info_;
295 301
296 base::WeakPtrFactory<NavigationRequest> weak_factory_; 302 base::WeakPtrFactory<NavigationRequest> weak_factory_;
297 303
298 DISALLOW_COPY_AND_ASSIGN(NavigationRequest); 304 DISALLOW_COPY_AND_ASSIGN(NavigationRequest);
299 }; 305 };
300 306
301 } // namespace content 307 } // namespace content
302 308
303 #endif // CONTENT_BROWSER_FRAME_HOST_NAVIGATION_REQUEST_H_ 309 #endif // CONTENT_BROWSER_FRAME_HOST_NAVIGATION_REQUEST_H_
OLDNEW
« no previous file with comments | « content/browser/frame_host/navigation_handle_impl_unittest.cc ('k') | content/browser/frame_host/navigation_request.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698