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

Side by Side Diff: third_party/WebKit/Source/web/WebRemoteFrameImpl.h

Issue 2021513002: Move willEnterFullscreen to WebRemoteFrame. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Add comment about user gestures 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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 WebRemoteFrameImpl_h 5 #ifndef WebRemoteFrameImpl_h
6 #define WebRemoteFrameImpl_h 6 #define WebRemoteFrameImpl_h
7 7
8 #include "core/frame/RemoteFrame.h" 8 #include "core/frame/RemoteFrame.h"
9 #include "platform/heap/SelfKeepAlive.h" 9 #include "platform/heap/SelfKeepAlive.h"
10 #include "public/web/WebRemoteFrame.h" 10 #include "public/web/WebRemoteFrame.h"
(...skipping 149 matching lines...) Expand 10 before | Expand all | Expand 10 after
160 void resetReplicatedContentSecurityPolicy() const override; 160 void resetReplicatedContentSecurityPolicy() const override;
161 void setReplicatedShouldEnforceStrictMixedContentChecking(bool) const overri de; 161 void setReplicatedShouldEnforceStrictMixedContentChecking(bool) const overri de;
162 void setReplicatedPotentiallyTrustworthyUniqueOrigin(bool) const override; 162 void setReplicatedPotentiallyTrustworthyUniqueOrigin(bool) const override;
163 void DispatchLoadEventForFrameOwner() const override; 163 void DispatchLoadEventForFrameOwner() const override;
164 164
165 void didStartLoading() override; 165 void didStartLoading() override;
166 void didStopLoading() override; 166 void didStopLoading() override;
167 167
168 bool isIgnoredForHitTest() const override; 168 bool isIgnoredForHitTest() const override;
169 169
170 void willEnterFullScreen() override;
171
170 DECLARE_TRACE(); 172 DECLARE_TRACE();
171 173
172 private: 174 private:
173 WebRemoteFrameImpl(WebTreeScopeType, WebRemoteFrameClient*); 175 WebRemoteFrameImpl(WebTreeScopeType, WebRemoteFrameClient*);
174 176
175 // Inherited from WebFrame, but intentionally hidden: it never makes sense 177 // Inherited from WebFrame, but intentionally hidden: it never makes sense
176 // to call these on a WebRemoteFrameImpl. 178 // to call these on a WebRemoteFrameImpl.
177 bool isWebLocalFrame() const override; 179 bool isWebLocalFrame() const override;
178 WebLocalFrame* toWebLocalFrame() override; 180 WebLocalFrame* toWebLocalFrame() override;
179 bool isWebRemoteFrame() const override; 181 bool isWebRemoteFrame() const override;
180 WebRemoteFrame* toWebRemoteFrame() override; 182 WebRemoteFrame* toWebRemoteFrame() override;
181 183
182 Member<RemoteFrameClientImpl> m_frameClient; 184 Member<RemoteFrameClientImpl> m_frameClient;
183 Member<RemoteFrame> m_frame; 185 Member<RemoteFrame> m_frame;
184 WebRemoteFrameClient* m_client; 186 WebRemoteFrameClient* m_client;
185 187
186 // Oilpan: WebRemoteFrameImpl must remain alive until close() is called. 188 // Oilpan: WebRemoteFrameImpl must remain alive until close() is called.
187 // Accomplish that by keeping a self-referential Persistent<>. It is 189 // Accomplish that by keeping a self-referential Persistent<>. It is
188 // cleared upon close(). 190 // cleared upon close().
189 SelfKeepAlive<WebRemoteFrameImpl> m_selfKeepAlive; 191 SelfKeepAlive<WebRemoteFrameImpl> m_selfKeepAlive;
190 }; 192 };
191 193
192 DEFINE_TYPE_CASTS(WebRemoteFrameImpl, WebFrame, frame, frame->isWebRemoteFrame() , frame.isWebRemoteFrame()); 194 DEFINE_TYPE_CASTS(WebRemoteFrameImpl, WebFrame, frame, frame->isWebRemoteFrame() , frame.isWebRemoteFrame());
193 195
194 } // namespace blink 196 } // namespace blink
195 197
196 #endif // WebRemoteFrameImpl_h 198 #endif // WebRemoteFrameImpl_h
OLDNEW
« no previous file with comments | « third_party/WebKit/Source/web/FullscreenController.cpp ('k') | third_party/WebKit/Source/web/WebRemoteFrameImpl.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698