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

Side by Side Diff: third_party/WebKit/Source/core/frame/RemoteFrame.h

Issue 2134113002: Use ChildFrameDisconnector when detaching child frames of a LocalFrame. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 4 years, 5 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 RemoteFrame_h 5 #ifndef RemoteFrame_h
6 #define RemoteFrame_h 6 #define RemoteFrame_h
7 7
8 #include "core/CoreExport.h" 8 #include "core/CoreExport.h"
9 #include "core/dom/RemoteSecurityContext.h" 9 #include "core/dom/RemoteSecurityContext.h"
10 #include "core/frame/Frame.h" 10 #include "core/frame/Frame.h"
(...skipping 53 matching lines...) Expand 10 before | Expand all | Expand 10 after
64 private: 64 private:
65 RemoteFrame(RemoteFrameClient*, FrameHost*, FrameOwner*); 65 RemoteFrame(RemoteFrameClient*, FrameHost*, FrameOwner*);
66 66
67 // Internal Frame helper overrides: 67 // Internal Frame helper overrides:
68 WindowProxyManager* getWindowProxyManager() const override { return m_window ProxyManager.get(); } 68 WindowProxyManager* getWindowProxyManager() const override { return m_window ProxyManager.get(); }
69 // Intentionally private to prevent redundant checks when the type is 69 // Intentionally private to prevent redundant checks when the type is
70 // already RemoteFrame. 70 // already RemoteFrame.
71 bool isLocalFrame() const override { return false; } 71 bool isLocalFrame() const override { return false; }
72 bool isRemoteFrame() const override { return true; } 72 bool isRemoteFrame() const override { return true; }
73 73
74 void detachChildren();
75
74 Member<RemoteFrameView> m_view; 76 Member<RemoteFrameView> m_view;
75 Member<RemoteSecurityContext> m_securityContext; 77 Member<RemoteSecurityContext> m_securityContext;
76 Member<RemoteDOMWindow> m_domWindow; 78 Member<RemoteDOMWindow> m_domWindow;
77 Member<WindowProxyManager> m_windowProxyManager; 79 Member<WindowProxyManager> m_windowProxyManager;
78 WebLayer* m_remotePlatformLayer; 80 WebLayer* m_remotePlatformLayer;
79 }; 81 };
80 82
81 inline RemoteFrameView* RemoteFrame::view() const 83 inline RemoteFrameView* RemoteFrame::view() const
82 { 84 {
83 return m_view.get(); 85 return m_view.get();
84 } 86 }
85 87
86 DEFINE_TYPE_CASTS(RemoteFrame, Frame, remoteFrame, remoteFrame->isRemoteFrame(), remoteFrame.isRemoteFrame()); 88 DEFINE_TYPE_CASTS(RemoteFrame, Frame, remoteFrame, remoteFrame->isRemoteFrame(), remoteFrame.isRemoteFrame());
87 89
88 } // namespace blink 90 } // namespace blink
89 91
90 #endif // RemoteFrame_h 92 #endif // RemoteFrame_h
OLDNEW
« no previous file with comments | « third_party/WebKit/Source/core/frame/LocalFrame.cpp ('k') | third_party/WebKit/Source/core/frame/RemoteFrame.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698