No need to detect change in size when opting for PositionedMovementLayout

No need to detect change in size when opting for PositionedMovementLayout

PositionedMovementLayout determines if it needs to do a full layout so
there is no need to second-guess that when parsing the style change.

BUG=428050
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=186335

[pdr., 2014-12-01 21:03:04]

pdr@chromium.org changed reviewers:
+ pdr@chromium.org

[pdr., 2014-12-01 21:03:05]

LGTM. Took me a minute to trace this code but I agree with you that it's right.

https://codereview.chromium.org/722643002/diff/1/Source/core/rendering/style/...
File Source/core/rendering/style/RenderStyle.cpp (right):

https://codereview.chromium.org/722643002/diff/1/Source/core/rendering/style/...
Source/core/rendering/style/RenderStyle.cpp:383: // Optimize for the case where
a positioned layer is moving but not changing size.
This comment needs updating, no?

[rhogan, 2014-12-02 20:00:07]

The CQ bit was checked by robhogan@gmail.com

[commit-bot: I haz the power, 2014-12-02 20:00:23]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/722643002/20001

[commit-bot: I haz the power, 2014-12-02 21:09:20]

Committed patchset #2 (id:20001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=186335