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

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

Issue 2753093002: Ensure that with PlzNavigate, reloads after process crash display the RenderView. (Closed)
Patch Set: nit Created 3 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 <stdint.h> 8 #include <stdint.h>
9 9
10 #include <list> 10 #include <list>
(...skipping 732 matching lines...) Expand 10 before | Expand all | Expand 10 after
743 // navigation. 743 // navigation.
744 void UpdatePendingWebUIOnCurrentFrameHost(const GURL& dest_url, 744 void UpdatePendingWebUIOnCurrentFrameHost(const GURL& dest_url,
745 int entry_bindings); 745 int entry_bindings);
746 746
747 // Returns true if a subframe can navigate cross-process. 747 // Returns true if a subframe can navigate cross-process.
748 bool CanSubframeSwapProcess(const GURL& dest_url, 748 bool CanSubframeSwapProcess(const GURL& dest_url,
749 SiteInstance* source_instance, 749 SiteInstance* source_instance,
750 SiteInstance* dest_instance, 750 SiteInstance* dest_instance,
751 bool was_server_redirect); 751 bool was_server_redirect);
752 752
753 // After a renderer process crash we'd have marked the host as invisible, so
754 // we need to set the visibility of the new View to the correct value here
755 // after reload.
756 void EnsureRenderFrameHostVisibilityConsistent();
757
753 // For use in creating RenderFrameHosts. 758 // For use in creating RenderFrameHosts.
754 FrameTreeNode* frame_tree_node_; 759 FrameTreeNode* frame_tree_node_;
755 760
756 // Our delegate, not owned by us. Guaranteed non-NULL. 761 // Our delegate, not owned by us. Guaranteed non-NULL.
757 Delegate* delegate_; 762 Delegate* delegate_;
758 763
759 // Implemented by the owner of this class. These delegates are installed into 764 // Implemented by the owner of this class. These delegates are installed into
760 // all the RenderFrameHosts that we create. 765 // all the RenderFrameHosts that we create.
761 RenderFrameHostDelegate* render_frame_delegate_; 766 RenderFrameHostDelegate* render_frame_delegate_;
762 RenderWidgetHostDelegate* render_widget_delegate_; 767 RenderWidgetHostDelegate* render_widget_delegate_;
(...skipping 41 matching lines...) Expand 10 before | Expand all | Expand 10 after
804 std::unique_ptr<RenderFrameHostImpl> speculative_render_frame_host_; 809 std::unique_ptr<RenderFrameHostImpl> speculative_render_frame_host_;
805 810
806 base::WeakPtrFactory<RenderFrameHostManager> weak_factory_; 811 base::WeakPtrFactory<RenderFrameHostManager> weak_factory_;
807 812
808 DISALLOW_COPY_AND_ASSIGN(RenderFrameHostManager); 813 DISALLOW_COPY_AND_ASSIGN(RenderFrameHostManager);
809 }; 814 };
810 815
811 } // namespace content 816 } // namespace content
812 817
813 #endif // CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_MANAGER_H_ 818 #endif // CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_MANAGER_H_
OLDNEW
« no previous file with comments | « chrome/browser/crash_recovery_browsertest.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