Update list markers in notify change.

Source/core/layout/TextAutosizer.cpp

Index: Source/core/layout/TextAutosizer.cpp
diff --git a/Source/core/layout/TextAutosizer.cpp b/Source/core/layout/TextAutosizer.cpp
index 568da42a402b5e6fa9d1d1c72f169184c082d132..2ceb0ba84f6c17e9ff8418c5be4aa499af6a986b 100644
--- a/Source/core/layout/TextAutosizer.cpp
+++ b/Source/core/layout/TextAutosizer.cpp
@@ -475,6 +475,15 @@ float TextAutosizer::inflate(LayoutObject* parent, InflateBehavior behavior, flo
         // multiplier of 1 (except for list items which are handled in inflateListItem).
         applyMultiplier(parent, 1);
     }
+
+    if (parent->isListItem()) {

[ojan, 2015/04/30 02:56:45]

I think it'd be better to keep this code in LayoutListItem::layout instead of in
the generic TextAutosizer code. It avoids a needless branch for non list-items
that way.

pdr, mind taking a look over this since you understand the auto-sizing code? I
believe the code inside this if-statement can just be wholesale moved back into
LayoutListItem::layout.

[dsinclair, 2015/04/30 17:52:30]

From reading the comment in the old LayoutListItem::layout it sounds like that
was also the wrong place for it:

        // The marker must be autosized before calling
        // updateMarkerLocationAndInvalidateWidth. It cannot be done in the
        // parent's beginLayout because it is not yet in the render tree.

We should now have the marker in the tree before ::layout is kicked off. It
looks like this is called through the beginLayout part of the autosizer?

[ojan, 2015/05/01 01:09:55]

Yeah. I don't really understand this comment. pdr or skobes should take a look
to see what the right thing to do here is.

[skobes, 2015/05/01 17:58:16]

TextAutosizer::inflateListItem was introduced because of the strange way the
list item marker was added to the tree during layout.  We had to inflate the
marker before the call to LayoutListItem::updateMarkerLocation, so we added a
new hook, since by the time we call inflate() on the list item it is too late to
autosize.

The change that added it is https://codereview.chromium.org/145003009/.

If we no longer add the marker during layout, perhaps inflateListItem can go
away.  In any case it doesn't make much sense to call inflateListItem from
wihtin inflate.

You can run the tests in fast/text-autosizing to verify that you are not
regressing the behavior.

+        LayoutListItem* item = toLayoutListItem(parent);
+        if (item->marker()) {
+            inflateListItem(item, item->marker());
+            item->marker()->setPreferredLogicalWidthsDirty(MarkOnlyThis);
+        }
+    }
+
     return multiplier;
 }