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

Side by Side Diff: third_party/WebKit/Source/core/frame/LocalFrame.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: DCHECK 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 /* 1 /*
2 * Copyright (C) 1998, 1999 Torben Weis <weis@kde.org> 2 * Copyright (C) 1998, 1999 Torben Weis <weis@kde.org>
3 * 1999-2001 Lars Knoll <knoll@kde.org> 3 * 1999-2001 Lars Knoll <knoll@kde.org>
4 * 1999-2001 Antti Koivisto <koivisto@kde.org> 4 * 1999-2001 Antti Koivisto <koivisto@kde.org>
5 * 2000-2001 Simon Hausmann <hausmann@kde.org> 5 * 2000-2001 Simon Hausmann <hausmann@kde.org>
6 * 2000-2001 Dirk Mueller <mueller@kde.org> 6 * 2000-2001 Dirk Mueller <mueller@kde.org>
7 * 2000 Stefan Schimanski <1Stein@gmx.de> 7 * 2000 Stefan Schimanski <1Stein@gmx.de>
8 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010 Apple Inc. All rights reserved. 8 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010 Apple Inc. All rights reserved.
9 * Copyright (C) 2008 Nokia Corporation and/or its subsidiary(-ies) 9 * Copyright (C) 2008 Nokia Corporation and/or its subsidiary(-ies)
10 * Copyright (C) 2008 Eric Seidel <eric@webkit.org> 10 * Copyright (C) 2008 Eric Seidel <eric@webkit.org>
(...skipping 86 matching lines...) Expand 10 before | Expand all | Expand 10 after
97 void navigate(Document& originDocument, const KURL&, bool replaceCurrentItem , UserGestureStatus) override; 97 void navigate(Document& originDocument, const KURL&, bool replaceCurrentItem , UserGestureStatus) override;
98 void navigate(const FrameLoadRequest&) override; 98 void navigate(const FrameLoadRequest&) override;
99 void reload(FrameLoadType, ClientRedirectPolicy) override; 99 void reload(FrameLoadType, ClientRedirectPolicy) override;
100 void detach(FrameDetachType) override; 100 void detach(FrameDetachType) override;
101 bool shouldClose() override; 101 bool shouldClose() override;
102 SecurityContext* securityContext() const override; 102 SecurityContext* securityContext() const override;
103 void printNavigationErrorMessage(const Frame&, const char* reason) override; 103 void printNavigationErrorMessage(const Frame&, const char* reason) override;
104 bool prepareForCommit() override; 104 bool prepareForCommit() override;
105 void didChangeVisibilityState() override; 105 void didChangeVisibilityState() override;
106 106
107 void detachChildren();
107 void willDetachFrameHost(); 108 void willDetachFrameHost();
108 109
109 LocalDOMWindow* localDOMWindow() const; 110 LocalDOMWindow* localDOMWindow() const;
110 void setDOMWindow(LocalDOMWindow*); 111 void setDOMWindow(LocalDOMWindow*);
111 FrameView* view() const; 112 FrameView* view() const;
112 Document* document() const; 113 Document* document() const;
113 void setPagePopupOwner(Element&); 114 void setPagePopupOwner(Element&);
114 Element* pagePopupOwner() const { return m_pagePopupOwner.get(); } 115 Element* pagePopupOwner() const { return m_pagePopupOwner.get(); }
115 116
116 LayoutView* contentLayoutObject() const; // Root of the layout tree for the document contained in this frame. 117 LayoutView* contentLayoutObject() const; // Root of the layout tree for the document contained in this frame.
(...skipping 217 matching lines...) Expand 10 before | Expand all | Expand 10 after
334 explicit ScopedFrameBlamer(LocalFrame*); 335 explicit ScopedFrameBlamer(LocalFrame*);
335 ~ScopedFrameBlamer(); 336 ~ScopedFrameBlamer();
336 337
337 private: 338 private:
338 Member<LocalFrame> m_frame; 339 Member<LocalFrame> m_frame;
339 }; 340 };
340 341
341 } // namespace blink 342 } // namespace blink
342 343
343 #endif // LocalFrame_h 344 #endif // LocalFrame_h
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698