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

Side by Side Diff: content/renderer/render_frame_impl.h

Issue 1023783004: [DevTools] Move DevToolsAgent to RenderFrameImpl. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: is_main_or_local_root_ -> is_local_root_ 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
« no previous file with comments | « content/renderer/devtools/devtools_agent.cc ('k') | content/renderer/render_frame_impl.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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_RENDERER_RENDER_FRAME_IMPL_H_ 5 #ifndef CONTENT_RENDERER_RENDER_FRAME_IMPL_H_
6 #define CONTENT_RENDERER_RENDER_FRAME_IMPL_H_ 6 #define CONTENT_RENDERER_RENDER_FRAME_IMPL_H_
7 7
8 #include <vector> 8 #include <vector>
9 9
10 #include "base/basictypes.h" 10 #include "base/basictypes.h"
(...skipping 56 matching lines...) Expand 10 before | Expand all | Expand 10 after
67 67
68 namespace media { 68 namespace media {
69 class MediaPermission; 69 class MediaPermission;
70 class WebEncryptedMediaClientImpl; 70 class WebEncryptedMediaClientImpl;
71 } 71 }
72 72
73 namespace content { 73 namespace content {
74 74
75 class ChildFrameCompositingHelper; 75 class ChildFrameCompositingHelper;
76 class CompositorDependencies; 76 class CompositorDependencies;
77 class DevToolsAgent;
77 class DocumentState; 78 class DocumentState;
78 class ExternalPopupMenu; 79 class ExternalPopupMenu;
79 class GeolocationDispatcher; 80 class GeolocationDispatcher;
80 class ManifestManager; 81 class ManifestManager;
81 class MediaStreamDispatcher; 82 class MediaStreamDispatcher;
82 class MediaStreamRendererFactory; 83 class MediaStreamRendererFactory;
83 class MediaPermissionDispatcher; 84 class MediaPermissionDispatcher;
84 class MidiDispatcher; 85 class MidiDispatcher;
85 class NavigationState; 86 class NavigationState;
86 class NotificationPermissionDispatcher; 87 class NotificationPermissionDispatcher;
(...skipping 661 matching lines...) Expand 10 before | Expand all | Expand 10 after
748 #if defined(ENABLE_BROWSER_CDMS) 749 #if defined(ENABLE_BROWSER_CDMS)
749 RendererCdmManager* GetCdmManager(); 750 RendererCdmManager* GetCdmManager();
750 #endif 751 #endif
751 752
752 // Stores the WebLocalFrame we are associated with. This is null from the 753 // Stores the WebLocalFrame we are associated with. This is null from the
753 // constructor until SetWebFrame is called, and it is null after 754 // constructor until SetWebFrame is called, and it is null after
754 // frameDetached is called until destruction (which is asynchronous in the 755 // frameDetached is called until destruction (which is asynchronous in the
755 // case of the main frame, but not subframes). 756 // case of the main frame, but not subframes).
756 blink::WebLocalFrame* frame_; 757 blink::WebLocalFrame* frame_;
757 758
759 // Frame is a local root if it is rendered in a process different than parent
760 // or it is a main frame.
761 bool is_local_root_;
762
758 base::WeakPtr<RenderViewImpl> render_view_; 763 base::WeakPtr<RenderViewImpl> render_view_;
759 int routing_id_; 764 int routing_id_;
760 bool is_swapped_out_; 765 bool is_swapped_out_;
761 // RenderFrameProxy exists only when is_swapped_out_ is true. 766 // RenderFrameProxy exists only when is_swapped_out_ is true.
762 // TODO(nasko): This can be removed once we don't have a swapped out state on 767 // TODO(nasko): This can be removed once we don't have a swapped out state on
763 // RenderFrame. See https://crbug.com/357747. 768 // RenderFrame. See https://crbug.com/357747.
764 RenderFrameProxy* render_frame_proxy_; 769 RenderFrameProxy* render_frame_proxy_;
765 bool is_detaching_; 770 bool is_detaching_;
766 771
767 // If this frame was created to replace a proxy, this will store the routing 772 // If this frame was created to replace a proxy, this will store the routing
(...skipping 92 matching lines...) Expand 10 before | Expand all | Expand 10 after
860 // RenderFrameObserver. 865 // RenderFrameObserver.
861 RendererCdmManager* cdm_manager_; 866 RendererCdmManager* cdm_manager_;
862 #endif 867 #endif
863 868
864 #if defined(VIDEO_HOLE) 869 #if defined(VIDEO_HOLE)
865 // Whether or not this RenderFrameImpl contains a media player. Used to 870 // Whether or not this RenderFrameImpl contains a media player. Used to
866 // register as an observer for video-hole-specific events. 871 // register as an observer for video-hole-specific events.
867 bool contains_media_player_; 872 bool contains_media_player_;
868 #endif 873 #endif
869 874
875 // The devtools agent for this frame; only created for main frame and
876 // local roots.
877 DevToolsAgent* devtools_agent_;
878
870 // The geolocation dispatcher attached to this frame, lazily initialized. 879 // The geolocation dispatcher attached to this frame, lazily initialized.
871 GeolocationDispatcher* geolocation_dispatcher_; 880 GeolocationDispatcher* geolocation_dispatcher_;
872 881
873 // The push messaging dispatcher attached to this frame, lazily initialized. 882 // The push messaging dispatcher attached to this frame, lazily initialized.
874 PushMessagingDispatcher* push_messaging_dispatcher_; 883 PushMessagingDispatcher* push_messaging_dispatcher_;
875 884
876 // The presentation dispatcher implementation attached to this frame, lazily 885 // The presentation dispatcher implementation attached to this frame, lazily
877 // initialized. 886 // initialized.
878 PresentationDispatcher* presentation_dispatcher_; 887 PresentationDispatcher* presentation_dispatcher_;
879 888
(...skipping 22 matching lines...) Expand all
902 #endif 911 #endif
903 912
904 base::WeakPtrFactory<RenderFrameImpl> weak_factory_; 913 base::WeakPtrFactory<RenderFrameImpl> weak_factory_;
905 914
906 DISALLOW_COPY_AND_ASSIGN(RenderFrameImpl); 915 DISALLOW_COPY_AND_ASSIGN(RenderFrameImpl);
907 }; 916 };
908 917
909 } // namespace content 918 } // namespace content
910 919
911 #endif // CONTENT_RENDERER_RENDER_FRAME_IMPL_H_ 920 #endif // CONTENT_RENDERER_RENDER_FRAME_IMPL_H_
OLDNEW
« no previous file with comments | « content/renderer/devtools/devtools_agent.cc ('k') | content/renderer/render_frame_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698