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

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

Issue 2393313002: reflow comments in core/frame (Closed)
Patch Set: tweak Created 4 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 /* 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
9 * reserved.
9 * Copyright (C) 2008 Nokia Corporation and/or its subsidiary(-ies) 10 * Copyright (C) 2008 Nokia Corporation and/or its subsidiary(-ies)
10 * Copyright (C) 2008 Eric Seidel <eric@webkit.org> 11 * Copyright (C) 2008 Eric Seidel <eric@webkit.org>
11 * 12 *
12 * This library is free software; you can redistribute it and/or 13 * This library is free software; you can redistribute it and/or
13 * modify it under the terms of the GNU Library General Public 14 * modify it under the terms of the GNU Library General Public
14 * License as published by the Free Software Foundation; either 15 * License as published by the Free Software Foundation; either
15 * version 2 of the License, or (at your option) any later version. 16 * version 2 of the License, or (at your option) any later version.
16 * 17 *
17 * This library is distributed in the hope that it will be useful, 18 * This library is distributed in the hope that it will be useful,
18 * but WITHOUT ANY WARRANTY; without even the implied warranty of 19 * but WITHOUT ANY WARRANTY; without even the implied warranty of
(...skipping 95 matching lines...) Expand 10 before | Expand all | Expand 10 after
114 // This prepares the Frame for the next commit. It will detach children, 115 // This prepares the Frame for the next commit. It will detach children,
115 // dispatch unload events, abort XHR requests and detach the document. 116 // dispatch unload events, abort XHR requests and detach the document.
116 // Returns true if the frame is ready to receive the next commit, or false 117 // Returns true if the frame is ready to receive the next commit, or false
117 // otherwise. 118 // otherwise.
118 virtual bool prepareForCommit() = 0; 119 virtual bool prepareForCommit() = 0;
119 120
120 bool canNavigate(const Frame&); 121 bool canNavigate(const Frame&);
121 virtual void printNavigationErrorMessage(const Frame&, 122 virtual void printNavigationErrorMessage(const Frame&,
122 const char* reason) = 0; 123 const char* reason) = 0;
123 124
124 // TODO(pilgrim) replace all instances of ownerLayoutObject() with ownerLayout Item() 125 // TODO(pilgrim): Replace all instances of ownerLayoutObject() with
125 // https://crbug.com/499321 126 // ownerLayoutItem(), https://crbug.com/499321
126 LayoutPart* ownerLayoutObject() 127 LayoutPart* ownerLayoutObject()
127 const; // LayoutObject for the element that contains this frame. 128 const; // LayoutObject for the element that contains this frame.
128 LayoutPartItem ownerLayoutItem() const; 129 LayoutPartItem ownerLayoutItem() const;
129 130
130 Settings* settings() const; // can be null 131 Settings* settings() const; // can be null
131 132
132 // isLoading() is true when the embedder should think a load is in progress. 133 // isLoading() is true when the embedder should think a load is in progress.
133 // In the case of LocalFrames, it means that the frame has sent a didStartLoad ing() 134 // In the case of LocalFrames, it means that the frame has sent a
134 // callback, but not the matching didStopLoading(). Inside blink, you probably 135 // didStartLoading() callback, but not the matching didStopLoading(). Inside
135 // want Document::loadEventFinished() instead. 136 // blink, you probably want Document::loadEventFinished() instead.
136 void setIsLoading(bool isLoading) { m_isLoading = isLoading; } 137 void setIsLoading(bool isLoading) { m_isLoading = isLoading; }
137 bool isLoading() const { return m_isLoading; } 138 bool isLoading() const { return m_isLoading; }
138 139
139 virtual WindowProxyManager* getWindowProxyManager() const = 0; 140 virtual WindowProxyManager* getWindowProxyManager() const = 0;
140 141
141 virtual void didChangeVisibilityState(); 142 virtual void didChangeVisibilityState();
142 143
143 protected: 144 protected:
144 Frame(FrameClient*, FrameHost*, FrameOwner*); 145 Frame(FrameClient*, FrameHost*, FrameOwner*);
145 146
(...skipping 14 matching lines...) Expand all
160 } 161 }
161 162
162 inline FrameOwner* Frame::owner() const { 163 inline FrameOwner* Frame::owner() const {
163 return m_owner; 164 return m_owner;
164 } 165 }
165 166
166 inline FrameTree& Frame::tree() const { 167 inline FrameTree& Frame::tree() const {
167 return m_treeNode; 168 return m_treeNode;
168 } 169 }
169 170
170 // Allow equality comparisons of Frames by reference or pointer, interchangeably . 171 // Allow equality comparisons of Frames by reference or pointer,
172 // interchangeably.
171 DEFINE_COMPARISON_OPERATORS_WITH_REFERENCES(Frame) 173 DEFINE_COMPARISON_OPERATORS_WITH_REFERENCES(Frame)
172 174
173 } // namespace blink 175 } // namespace blink
174 176
175 #endif // Frame_h 177 #endif // Frame_h
OLDNEW
« no previous file with comments | « third_party/WebKit/Source/core/frame/EventHandlerRegistry.cpp ('k') | third_party/WebKit/Source/core/frame/Frame.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698