| Index: third_party/WebKit/Source/core/dom/Text.cpp
|
| diff --git a/third_party/WebKit/Source/core/dom/Text.cpp b/third_party/WebKit/Source/core/dom/Text.cpp
|
| index 8c6d94d84da28da44e1804c14189c4dd5b893beb..55e0947ec28d962a57ae1b8b69760c2c3906b56d 100644
|
| --- a/third_party/WebKit/Source/core/dom/Text.cpp
|
| +++ b/third_party/WebKit/Source/core/dom/Text.cpp
|
| @@ -395,16 +395,30 @@ void Text::recalcTextStyle(StyleRecalcChange change, Text* nextTextSibling) {
|
| layoutItem.setStyle(document().ensureStyleResolver().styleForText(this));
|
| if (needsStyleRecalc())
|
| layoutItem.setText(dataImpl());
|
| - clearNeedsStyleRecalc();
|
| + // In case we don't perform rebuildTextLayoutTree we will never clear the
|
| + // NeedsReattachLayoutTree flag which is set on the creation of each
|
| + // Node. Clear that here.
|
| + clearNeedsReattachLayoutTree();
|
| } else if (needsStyleRecalc() || needsWhitespaceLayoutObject()) {
|
| - rebuildTextLayoutTree(nextTextSibling);
|
| + StyleReattachData styleReattachData;
|
| + styleReattachData.nextTextSibling = nextTextSibling;
|
| + document().addStyleReattachData(*this, styleReattachData);
|
| + setNeedsReattachLayoutTree();
|
| }
|
| + clearNeedsStyleRecalc();
|
| }
|
|
|
| -void Text::rebuildTextLayoutTree(Text* nextTextSibling) {
|
| +void Text::rebuildTextLayoutTree() {
|
| + DCHECK(!needsStyleRecalc());
|
| + DCHECK(!childNeedsStyleRecalc());
|
| + DCHECK(needsReattachLayoutTree());
|
| + DCHECK(parentNode());
|
| +
|
| reattachLayoutTree();
|
| - if (layoutObject())
|
| - reattachWhitespaceSiblingsIfNeeded(nextTextSibling);
|
| + if (layoutObject()) {
|
| + reattachWhitespaceSiblingsIfNeeded(
|
| + document().getStyleReattachData(*this).nextTextSibling);
|
| + }
|
| clearNeedsReattachLayoutTree();
|
| }
|
|
|
|
|