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/layout/LayoutObject.h

Issue 2171173002: Renamed attach to attachLayoutTree in comment missed in 2167063002. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2000 Lars Knoll (knoll@kde.org) 2 * Copyright (C) 2000 Lars Knoll (knoll@kde.org)
3 * (C) 2000 Antti Koivisto (koivisto@kde.org) 3 * (C) 2000 Antti Koivisto (koivisto@kde.org)
4 * (C) 2000 Dirk Mueller (mueller@kde.org) 4 * (C) 2000 Dirk Mueller (mueller@kde.org)
5 * (C) 2004 Allan Sandfeld Jensen (kde@carewolf.com) 5 * (C) 2004 Allan Sandfeld Jensen (kde@carewolf.com)
6 * Copyright (C) 2003, 2004, 2005, 2006, 2007, 2008, 2009, 2012 Apple Inc. All r ights reserved. 6 * Copyright (C) 2003, 2004, 2005, 2006, 2007, 2008, 2009, 2012 Apple Inc. All r ights reserved.
7 * Copyright (C) 2009 Google Inc. All rights reserved. 7 * Copyright (C) 2009 Google Inc. All rights reserved.
8 * 8 *
9 * This library is free software; you can redistribute it and/or 9 * This library is free software; you can redistribute it and/or
10 * modify it under the terms of the GNU Library General Public 10 * modify it under the terms of the GNU Library General Public
(...skipping 161 matching lines...) Expand 10 before | Expand all | Expand 10 after
172 // 172 //
173 // 173 //
174 // ***** LIFETIME ***** 174 // ***** LIFETIME *****
175 // 175 //
176 // LayoutObjects are fully owned by their associated DOM node. In other words, 176 // LayoutObjects are fully owned by their associated DOM node. In other words,
177 // it's the DOM node's responsibility to free its LayoutObject, this is why 177 // it's the DOM node's responsibility to free its LayoutObject, this is why
178 // LayoutObjects are not and SHOULD NOT be RefCounted. 178 // LayoutObjects are not and SHOULD NOT be RefCounted.
179 // 179 //
180 // LayoutObjects are created during the DOM attachment. This phase computes 180 // LayoutObjects are created during the DOM attachment. This phase computes
181 // the style and create the LayoutObject associated with the Node (see 181 // the style and create the LayoutObject associated with the Node (see
182 // Node::attach). LayoutObjects are destructed during detachment (see 182 // Node::attachLayoutTree). LayoutObjects are destructed during detachment (see
183 // Node::detach), which can happen when the DOM node is removed from the 183 // Node::detach), which can happen when the DOM node is removed from the
184 // DOM tree, during page tear down or when the style is changed to contain 184 // DOM tree, during page tear down or when the style is changed to contain
185 // 'display: none'. 185 // 'display: none'.
186 // 186 //
187 // Anonymous LayoutObjects are owned by their enclosing DOM node. This means 187 // Anonymous LayoutObjects are owned by their enclosing DOM node. This means
188 // that if the DOM node is detached, it has to destroy any anonymous 188 // that if the DOM node is detached, it has to destroy any anonymous
189 // descendants. This is done in LayoutObject::destroy(). 189 // descendants. This is done in LayoutObject::destroy().
190 // 190 //
191 // Note that for correctness, destroy() is expected to clean any anonymous 191 // Note that for correctness, destroy() is expected to clean any anonymous
192 // wrappers as sequences of insertion / removal could make them visible to 192 // wrappers as sequences of insertion / removal could make them visible to
(...skipping 1984 matching lines...) Expand 10 before | Expand all | Expand 10 after
2177 void showTree(const blink::LayoutObject*); 2177 void showTree(const blink::LayoutObject*);
2178 void showLineTree(const blink::LayoutObject*); 2178 void showLineTree(const blink::LayoutObject*);
2179 void showLayoutTree(const blink::LayoutObject* object1); 2179 void showLayoutTree(const blink::LayoutObject* object1);
2180 // We don't make object2 an optional parameter so that showLayoutTree 2180 // We don't make object2 an optional parameter so that showLayoutTree
2181 // can be called from gdb easily. 2181 // can be called from gdb easily.
2182 void showLayoutTree(const blink::LayoutObject* object1, const blink::LayoutObjec t* object2); 2182 void showLayoutTree(const blink::LayoutObject* object1, const blink::LayoutObjec t* object2);
2183 2183
2184 #endif 2184 #endif
2185 2185
2186 #endif // LayoutObject_h 2186 #endif // LayoutObject_h
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698