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

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: Rebase + fixed issue with layout test Created 4 years, 6 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 251 matching lines...) Expand 10 before | Expand all | Expand 10 after
262 WebUIImpl* GetNavigatingWebUI() const; 262 WebUIImpl* GetNavigatingWebUI() const;
263 263
264 // Called when we want to instruct the renderer to navigate to the given 264 // Called when we want to instruct the renderer to navigate to the given
265 // navigation entry. It may create a new RenderFrameHost or re-use an existing 265 // navigation entry. It may create a new RenderFrameHost or re-use an existing
266 // one. The RenderFrameHost to navigate will be returned. Returns NULL if one 266 // one. The RenderFrameHost to navigate will be returned. Returns NULL if one
267 // could not be created. |dest_url| takes precedence over the |frame_entry|'s 267 // could not be created. |dest_url| takes precedence over the |frame_entry|'s
268 // url (this is necessary because ReloadOriginalRequest navigates to a 268 // url (this is necessary because ReloadOriginalRequest navigates to a
269 // different URL than the last committed entry, without modifying it). 269 // different URL than the last committed entry, without modifying it).
270 RenderFrameHostImpl* Navigate(const GURL& dest_url, 270 RenderFrameHostImpl* Navigate(const GURL& dest_url,
271 const FrameNavigationEntry& frame_entry, 271 const FrameNavigationEntry& frame_entry,
272 const NavigationEntryImpl& entry); 272 const NavigationEntryImpl& entry,
273 bool is_reload);
273 274
274 // Instructs the various live views to stop. Called when the user directed the 275 // Instructs the various live views to stop. Called when the user directed the
275 // page to stop loading. 276 // page to stop loading.
276 void Stop(); 277 void Stop();
277 278
278 // Notifies the regular and pending RenderViewHosts that a load is or is not 279 // Notifies the regular and pending RenderViewHosts that a load is or is not
279 // happening. Even though the message is only for one of them, we don't know 280 // happening. Even though the message is only for one of them, we don't know
280 // which one so we tell both. 281 // which one so we tell both.
281 void SetIsLoading(bool is_loading); 282 void SetIsLoading(bool is_loading);
282 283
(...skipping 439 matching lines...) Expand 10 before | Expand all | Expand 10 after
722 std::unique_ptr<RenderFrameHostImpl> render_frame_host); 723 std::unique_ptr<RenderFrameHostImpl> render_frame_host);
723 724
724 RenderFrameHostImpl* UpdateStateForNavigate( 725 RenderFrameHostImpl* UpdateStateForNavigate(
725 const GURL& dest_url, 726 const GURL& dest_url,
726 SiteInstance* source_instance, 727 SiteInstance* source_instance,
727 SiteInstance* dest_instance, 728 SiteInstance* dest_instance,
728 ui::PageTransition transition, 729 ui::PageTransition transition,
729 bool dest_is_restore, 730 bool dest_is_restore,
730 bool dest_is_view_source_mode, 731 bool dest_is_view_source_mode,
731 const GlobalRequestID& transferred_request_id, 732 const GlobalRequestID& transferred_request_id,
732 int bindings); 733 int bindings,
734 bool is_reload);
733 735
734 // Updates the pending WebUI of the current RenderFrameHost for a same-site 736 // Updates the pending WebUI of the current RenderFrameHost for a same-site
735 // navigation. 737 // navigation.
736 void UpdatePendingWebUIOnCurrentFrameHost(const GURL& dest_url, 738 void UpdatePendingWebUIOnCurrentFrameHost(const GURL& dest_url,
737 int entry_bindings); 739 int entry_bindings);
738 740
739 // Returns true if a subframe can navigate cross-process. 741 // Returns true if a subframe can navigate cross-process.
740 bool CanSubframeSwapProcess(const GURL& dest_url, 742 bool CanSubframeSwapProcess(const GURL& dest_url,
741 SiteInstance* source_instance, 743 SiteInstance* source_instance,
742 SiteInstance* dest_instance); 744 SiteInstance* dest_instance);
(...skipping 58 matching lines...) Expand 10 before | Expand all | Expand 10 after
801 std::unique_ptr<RenderFrameHostImpl> speculative_render_frame_host_; 803 std::unique_ptr<RenderFrameHostImpl> speculative_render_frame_host_;
802 804
803 base::WeakPtrFactory<RenderFrameHostManager> weak_factory_; 805 base::WeakPtrFactory<RenderFrameHostManager> weak_factory_;
804 806
805 DISALLOW_COPY_AND_ASSIGN(RenderFrameHostManager); 807 DISALLOW_COPY_AND_ASSIGN(RenderFrameHostManager);
806 }; 808 };
807 809
808 } // namespace content 810 } // namespace content
809 811
810 #endif // CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_MANAGER_H_ 812 #endif // CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_MANAGER_H_
OLDNEW
« no previous file with comments | « content/browser/frame_host/render_frame_host_impl.cc ('k') | content/browser/frame_host/render_frame_host_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698