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

Unified Diff: third_party/WebKit/Source/core/dom/Node.cpp

Issue 2439973005: Revert of Move Layout Tree Construction code into Element::rebuildLayoutTree() (Closed)
Patch Set: Revert "Move Layout Tree Construction code into Element::rebuildLayoutTree()" 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « third_party/WebKit/Source/core/dom/Node.h ('k') | third_party/WebKit/Source/core/dom/Text.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/Source/core/dom/Node.cpp
diff --git a/third_party/WebKit/Source/core/dom/Node.cpp b/third_party/WebKit/Source/core/dom/Node.cpp
index b8dbf92e9be8c48d703215204c61353db4b26207..8059397fe39faf8fa56d3bc2bd6e957b4626d4e9 100644
--- a/third_party/WebKit/Source/core/dom/Node.cpp
+++ b/third_party/WebKit/Source/core/dom/Node.cpp
@@ -390,18 +390,6 @@ Node* Node::getRootNode(const GetRootNodeOptions& options) const {
: &treeRoot();
}
-Text* Node::nextTextSibling() const {
- for (Node* sibling = nextSibling();
- sibling &&
- (!sibling->isElementNode() || !toElement(sibling)->layoutObject());
- sibling = sibling->nextSibling()) {
- if (sibling->isTextNode()) {
- return toText(sibling);
- }
- }
- return nullptr;
-}
-
Node* Node::insertBefore(Node* newChild,
Node* refChild,
ExceptionState& exceptionState) {
@@ -709,17 +697,6 @@ void Node::markAncestorsWithChildNeedsStyleRecalc() {
document().scheduleLayoutTreeUpdateIfNeeded();
}
-void Node::markAncestorsWithChildNeedsReattachLayoutTree() {
- for (ContainerNode* p = parentOrShadowHostNode();
- p && !p->childNeedsReattachLayoutTree(); p = p->parentOrShadowHostNode())
- p->setChildNeedsReattachLayoutTree();
-}
-
-void Node::setNeedsReattachLayoutTree() {
- setFlag(NeedsReattachLayoutTree);
- markAncestorsWithChildNeedsReattachLayoutTree();
-}
-
void Node::setNeedsStyleRecalc(StyleChangeType changeType,
const StyleChangeReasonForTracing& reason) {
DCHECK(changeType != NoStyleChange);
@@ -911,7 +888,6 @@ void Node::attachLayoutTree(const AttachContext&) {
(layoutObject()->parent() || layoutObject()->isLayoutView())));
clearNeedsStyleRecalc();
- clearNeedsReattachLayoutTree();
if (AXObjectCache* cache = document().axObjectCache())
cache->updateCacheAfterNodeIsAttached(this);
« no previous file with comments | « third_party/WebKit/Source/core/dom/Node.h ('k') | third_party/WebKit/Source/core/dom/Text.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698