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

Unified Diff: Source/core/dom/WhitespaceChildList.h

Issue 58373002: Whitespace only text nodes need to be reattached when their siblings are reattached (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Updated js-test include Created 7 years, 1 month 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 | « Source/core/dom/Node.cpp ('k') | Source/core/dom/shadow/ShadowRoot.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/dom/WhitespaceChildList.h
diff --git a/Source/core/dom/WhitespaceChildList.h b/Source/core/dom/WhitespaceChildList.h
deleted file mode 100644
index b5a4fb14f656e6ca69c20b1dbf92c26bc71b2b9b..0000000000000000000000000000000000000000
--- a/Source/core/dom/WhitespaceChildList.h
+++ /dev/null
@@ -1,71 +0,0 @@
-/*
- * Copyright (C) 2013 Google Inc. All rights reserved.
- *
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions are
- * met:
- *
- * * Redistributions of source code must retain the above copyright
- * notice, this list of conditions and the following disclaimer.
- * * Neither the name of Google Inc. nor the names of its
- * contributors may be used to endorse or promote products derived from
- * this software without specific prior written permission.
- *
- * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
- * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
- * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
- * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
- * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
- * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
- * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
- * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
- * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
- * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
- * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
- */
-
-#ifndef WhitespaceChildList_h
-#define WhitespaceChildList_h
-
-#include "core/dom/Text.h"
-#include "core/rendering/style/RenderStyleConstants.h"
-
-namespace WebCore {
-
-class Text;
-
-// Tracks a limited number of whitespace text children during style recalc
-// to postpone their style recalc as part of optimization to avoid creating
-// unnecessary whitespace text renderers. If we hit the limit, it recalcs
-// the whitespace text's style and clears the list.
-class WhitespaceChildList {
-public:
- WhitespaceChildList(StyleRecalcChange change)
- : m_change(change)
- { }
-
- void append(Text* textChild)
- {
- ASSERT(textChild->containsOnlyWhitespace());
- if (m_list.size() == maxWhitespaceChildrenToDefer) {
- recalcStyle();
- m_list.clear();
- }
- m_list.append(textChild);
- }
-
- void recalcStyle() const
- {
- for (unsigned i = 0; i < m_list.size(); ++i)
- m_list[i]->recalcTextStyle(m_change);
- }
-private:
- StyleRecalcChange m_change;
-
- static const unsigned maxWhitespaceChildrenToDefer = 10;
- Vector<Text*, maxWhitespaceChildrenToDefer> m_list;
-};
-
-} // namespace WebCore
-
-#endif // WhitespaceChildList_h
« no previous file with comments | « Source/core/dom/Node.cpp ('k') | Source/core/dom/shadow/ShadowRoot.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698