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

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

Issue 1890493002: PlzNavigate: properly execute BeforeUnload on renderer initiated navigations (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addressed comments Created 4 years, 7 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 <stdint.h> 8 #include <stdint.h>
9 9
10 #include <list> 10 #include <list>
(...skipping 250 matching lines...) Expand 10 before | Expand all | Expand 10 after
261 WebUIImpl* GetNavigatingWebUI() const; 261 WebUIImpl* GetNavigatingWebUI() const;
262 262
263 // Called when we want to instruct the renderer to navigate to the given 263 // Called when we want to instruct the renderer to navigate to the given
264 // navigation entry. It may create a new RenderFrameHost or re-use an existing 264 // navigation entry. It may create a new RenderFrameHost or re-use an existing
265 // one. The RenderFrameHost to navigate will be returned. Returns NULL if one 265 // one. The RenderFrameHost to navigate will be returned. Returns NULL if one
266 // could not be created. |dest_url| takes precedence over the |frame_entry|'s 266 // could not be created. |dest_url| takes precedence over the |frame_entry|'s
267 // url (this is necessary because ReloadOriginalRequest navigates to a 267 // url (this is necessary because ReloadOriginalRequest navigates to a
268 // different URL than the last committed entry, without modifying it). 268 // different URL than the last committed entry, without modifying it).
269 RenderFrameHostImpl* Navigate(const GURL& dest_url, 269 RenderFrameHostImpl* Navigate(const GURL& dest_url,
270 const FrameNavigationEntry& frame_entry, 270 const FrameNavigationEntry& frame_entry,
271 const NavigationEntryImpl& entry); 271 const NavigationEntryImpl& entry,
272 bool is_reload);
272 273
273 // Instructs the various live views to stop. Called when the user directed the 274 // Instructs the various live views to stop. Called when the user directed the
274 // page to stop loading. 275 // page to stop loading.
275 void Stop(); 276 void Stop();
276 277
277 // Notifies the regular and pending RenderViewHosts that a load is or is not 278 // Notifies the regular and pending RenderViewHosts that a load is or is not
278 // happening. Even though the message is only for one of them, we don't know 279 // happening. Even though the message is only for one of them, we don't know
279 // which one so we tell both. 280 // which one so we tell both.
280 void SetIsLoading(bool is_loading); 281 void SetIsLoading(bool is_loading);
281 282
(...skipping 431 matching lines...) Expand 10 before | Expand all | Expand 10 after
713 std::unique_ptr<RenderFrameHostImpl> render_frame_host); 714 std::unique_ptr<RenderFrameHostImpl> render_frame_host);
714 715
715 RenderFrameHostImpl* UpdateStateForNavigate( 716 RenderFrameHostImpl* UpdateStateForNavigate(
716 const GURL& dest_url, 717 const GURL& dest_url,
717 SiteInstance* source_instance, 718 SiteInstance* source_instance,
718 SiteInstance* dest_instance, 719 SiteInstance* dest_instance,
719 ui::PageTransition transition, 720 ui::PageTransition transition,
720 bool dest_is_restore, 721 bool dest_is_restore,
721 bool dest_is_view_source_mode, 722 bool dest_is_view_source_mode,
722 const GlobalRequestID& transferred_request_id, 723 const GlobalRequestID& transferred_request_id,
723 int bindings); 724 int bindings,
725 bool is_reload);
724 726
725 // Updates the pending WebUI of the current RenderFrameHost for a same-site 727 // Updates the pending WebUI of the current RenderFrameHost for a same-site
726 // navigation. 728 // navigation.
727 void UpdatePendingWebUIOnCurrentFrameHost(const GURL& dest_url, 729 void UpdatePendingWebUIOnCurrentFrameHost(const GURL& dest_url,
728 int entry_bindings); 730 int entry_bindings);
729 731
730 // Returns true if a subframe can navigate cross-process. 732 // Returns true if a subframe can navigate cross-process.
731 bool CanSubframeSwapProcess(const GURL& dest_url, 733 bool CanSubframeSwapProcess(const GURL& dest_url,
732 SiteInstance* source_instance, 734 SiteInstance* source_instance,
733 SiteInstance* dest_instance); 735 SiteInstance* dest_instance);
(...skipping 58 matching lines...) Expand 10 before | Expand all | Expand 10 after
792 std::unique_ptr<RenderFrameHostImpl> speculative_render_frame_host_; 794 std::unique_ptr<RenderFrameHostImpl> speculative_render_frame_host_;
793 795
794 base::WeakPtrFactory<RenderFrameHostManager> weak_factory_; 796 base::WeakPtrFactory<RenderFrameHostManager> weak_factory_;
795 797
796 DISALLOW_COPY_AND_ASSIGN(RenderFrameHostManager); 798 DISALLOW_COPY_AND_ASSIGN(RenderFrameHostManager);
797 }; 799 };
798 800
799 } // namespace content 801 } // namespace content
800 802
801 #endif // CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_MANAGER_H_ 803 #endif // CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_MANAGER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698