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

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

Issue 967383002: PlzNavigate: Avoid duplicate SiteInstance creation during navigation. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 9 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 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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_RENDER_FRAME_HOST_MANAGER_H_ 5 #ifndef CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_MANAGER_H_
6 #define CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_MANAGER_H_ 6 #define CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_MANAGER_H_
7 7
8 #include <list> 8 #include <list>
9 9
10 #include "base/basictypes.h" 10 #include "base/basictypes.h"
(...skipping 470 matching lines...) Expand 10 before | Expand all | Expand 10 after
481 // This is a helper function for GetSiteInstanceForNavigation. 481 // This is a helper function for GetSiteInstanceForNavigation.
482 SiteInstance* GetSiteInstanceForURL(const GURL& dest_url, 482 SiteInstance* GetSiteInstanceForURL(const GURL& dest_url,
483 SiteInstance* source_instance, 483 SiteInstance* source_instance,
484 SiteInstance* current_instance, 484 SiteInstance* current_instance,
485 SiteInstance* dest_instance, 485 SiteInstance* dest_instance,
486 ui::PageTransition transition, 486 ui::PageTransition transition,
487 bool dest_is_restore, 487 bool dest_is_restore,
488 bool dest_is_view_source_mode, 488 bool dest_is_view_source_mode,
489 bool force_browsing_instance_swap); 489 bool force_browsing_instance_swap);
490 490
491 // Creates a new SiteInstance for the provided URL under a new browsing
492 // instance.
493 // PlzNavigate: If a SiteInstance was already previously created for this URL
494 // for the same navigation, return that one. Otherwise create a new one and
495 // return it.
496 SiteInstance* CreateSiteInstanceForURL(BrowserContext* browser_context,
497 const GURL& dest_url);
498
491 // Determines the appropriate url to use as the current url for SiteInstance 499 // Determines the appropriate url to use as the current url for SiteInstance
492 // selection. 500 // selection.
493 const GURL& GetCurrentURLForSiteInstance( 501 const GURL& GetCurrentURLForSiteInstance(
494 SiteInstance* current_instance, 502 SiteInstance* current_instance,
495 NavigationEntry* current_entry); 503 NavigationEntry* current_entry);
496 504
497 // Creates a new RenderFrameHostImpl for the |new_instance| and assign it to 505 // Creates a new RenderFrameHostImpl for the |new_instance| and assign it to
498 // |pending_render_frame_host_| while respecting the opener route if needed 506 // |pending_render_frame_host_| while respecting the opener route if needed
499 // and stores it in pending_render_frame_host_. 507 // and stores it in pending_render_frame_host_.
500 void CreatePendingRenderFrameHost(SiteInstance* old_instance, 508 void CreatePendingRenderFrameHost(SiteInstance* old_instance,
(...skipping 187 matching lines...) Expand 10 before | Expand all | Expand 10 after
688 bool should_reuse_web_ui_; 696 bool should_reuse_web_ui_;
689 697
690 base::WeakPtrFactory<RenderFrameHostManager> weak_factory_; 698 base::WeakPtrFactory<RenderFrameHostManager> weak_factory_;
691 699
692 DISALLOW_COPY_AND_ASSIGN(RenderFrameHostManager); 700 DISALLOW_COPY_AND_ASSIGN(RenderFrameHostManager);
693 }; 701 };
694 702
695 } // namespace content 703 } // namespace content
696 704
697 #endif // CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_MANAGER_H_ 705 #endif // CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_MANAGER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698