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

Side by Side Diff: content/public/renderer/render_frame.h

Issue 1360123006: Introduce RenderFrame::IsMainFrame and clean up a few things. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixed content_browsertests failure. Created 5 years, 2 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_PUBLIC_RENDERER_RENDER_FRAME_H_ 5 #ifndef CONTENT_PUBLIC_RENDERER_RENDER_FRAME_H_
6 #define CONTENT_PUBLIC_RENDERER_RENDER_FRAME_H_ 6 #define CONTENT_PUBLIC_RENDERER_RENDER_FRAME_H_
7 7
8 #include "base/callback_forward.h" 8 #include "base/callback_forward.h"
9 #include "base/memory/scoped_ptr.h" 9 #include "base/memory/scoped_ptr.h"
10 #include "base/strings/string16.h" 10 #include "base/strings/string16.h"
(...skipping 97 matching lines...) Expand 10 before | Expand all | Expand 10 after
108 scoped_ptr<PluginInstanceThrottler> throttler) = 0; 108 scoped_ptr<PluginInstanceThrottler> throttler) = 0;
109 109
110 // The client should handle the navigation externally. 110 // The client should handle the navigation externally.
111 virtual void LoadURLExternally(blink::WebLocalFrame* frame, 111 virtual void LoadURLExternally(blink::WebLocalFrame* frame,
112 const blink::WebURLRequest& request, 112 const blink::WebURLRequest& request,
113 blink::WebNavigationPolicy policy) = 0; 113 blink::WebNavigationPolicy policy) = 0;
114 114
115 // Execute a string of JavaScript in this frame's context. 115 // Execute a string of JavaScript in this frame's context.
116 virtual void ExecuteJavaScript(const base::string16& javascript) = 0; 116 virtual void ExecuteJavaScript(const base::string16& javascript) = 0;
117 117
118 // Returns true if this is the main frame
119 // (That's the frame at the top of the frame tree).
Charlie Reis 2015/09/26 00:08:04 nit: No need for the clarification, I think. At m
dglazkov 2015/09/28 04:03:35 Done.
120 virtual bool IsMainFrame() = 0;
121
118 // Return true if this frame is hidden. 122 // Return true if this frame is hidden.
119 virtual bool IsHidden() = 0; 123 virtual bool IsHidden() = 0;
120 124
121 // Returns the ServiceRegistry for this frame. 125 // Returns the ServiceRegistry for this frame.
122 virtual ServiceRegistry* GetServiceRegistry() = 0; 126 virtual ServiceRegistry* GetServiceRegistry() = 0;
123 127
124 #if defined(ENABLE_PLUGINS) 128 #if defined(ENABLE_PLUGINS)
125 // Registers a plugin that has been marked peripheral. If the origin 129 // Registers a plugin that has been marked peripheral. If the origin
126 // whitelist is later updated and includes |content_origin|, then 130 // whitelist is later updated and includes |content_origin|, then
127 // |unthrottle_callback| will be called. 131 // |unthrottle_callback| will be called.
(...skipping 32 matching lines...) Expand 10 before | Expand all | Expand 10 after
160 164
161 private: 165 private:
162 // This interface should only be implemented inside content. 166 // This interface should only be implemented inside content.
163 friend class RenderFrameImpl; 167 friend class RenderFrameImpl;
164 RenderFrame() {} 168 RenderFrame() {}
165 }; 169 };
166 170
167 } // namespace content 171 } // namespace content
168 172
169 #endif // CONTENT_PUBLIC_RENDERER_RENDER_FRAME_H_ 173 #endif // CONTENT_PUBLIC_RENDERER_RENDER_FRAME_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698