Pull up LayoutStateDisabler in updateFirstLetterStyle

Pull up LayoutStateDisabler in updateFirstLetterStyle

Avoid improper repainting/asserting when falling into the
case in updateFirstLetterStyle where the style change doesn't
trigger a reattach. In that case, we'd still potentially
issue a repaint, but without the LayoutStateDisabler,
and with LayoutState set up improperly for the task.

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

[leviw_travelin_and_unemployed, 2014-06-02 18:47:37]

PTAL

[dsinclair, 2014-06-02 19:12:53]

lgtm

https://codereview.chromium.org/311633002/diff/1/Source/core/rendering/Render...
File Source/core/rendering/RenderBlock.cpp (right):

https://codereview.chromium.org/311633002/diff/1/Source/core/rendering/Render...
Source/core/rendering/RenderBlock.cpp:3989: LayoutStateDisabler disabler(*this);
nit: s/disabler/layoutStateDisabler

I've run into this before when adding second disabler to a method, it's nice to
have unique names.

[leviw_travelin_and_unemployed, 2014-06-02 19:32:03]

On 2014/06/02 19:12:53, dsinclair wrote:
> lgtm
> 
>
https://codereview.chromium.org/311633002/diff/1/Source/core/rendering/Render...
> File Source/core/rendering/RenderBlock.cpp (right):
> 
>
https://codereview.chromium.org/311633002/diff/1/Source/core/rendering/Render...
> Source/core/rendering/RenderBlock.cpp:3989: LayoutStateDisabler
disabler(*this);
> nit: s/disabler/layoutStateDisabler
> 
> I've run into this before when adding second disabler to a method, it's nice
to
> have unique names.

Done. TY!

[leviw_travelin_and_unemployed, 2014-06-02 19:32:16]

The CQ bit was checked by leviw@chromium.org

[commit-bot: I haz the power, 2014-06-02 19:33:05]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/leviw@chromium.org/311633002/20001

[commit-bot: I haz the power, 2014-06-02 21:42:04]

Change committed as 175319