Revert of TextIteratorAlgorithm should not force layout update in constructor.

Revert of TextIteratorAlgorithm should not force layout update in constructor. (patchset #8 id:130001 of https://codereview.chromium.org/1740173003/ )

Reason for revert:
Suspected of breaking RenderViewImplTest.OnHandleKeyboardEvent on https://build.chromium.org/p/chromium.win/builders/Win%20x64%20GN%20(dbg)

First failing build:
https://build.chromium.org/p/chromium.win/builders/Win%20x64%20GN%20%28dbg%29/builds/19191

Original issue's description:
> TextIteratorAlgorithm should not force layout update in constructor.
>
> Asking to update layout should not be done at this level -- updating layout is part of the document/view lifecycle and should be coordinated at a higher level than an iterator constructor.
>
> Thus, move calling Document::updateLayoutIgnorePendingStylesheets up to the callsites of the TextIterator constructor. These callsites should be audited and moved up even farther up the call stack.
>
> BUG=590295
>
> Committed: https://crrev.com/4ed5e3b54598950731022926b09fdc8742abea74
> Cr-Commit-Position: refs/heads/master@{#379205}

TBR=esprehn@chromium.org,yosin@chromium.org,dglazkov@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=590295
Committed: https://crrev.com/2d8c5743dc6cef1332eb671d9be6c54de00bd611
Cr-Commit-Position: refs/heads/master@{#379265}

[Marc Treib, 2016-03-04 10:49:04]

Created Revert of TextIteratorAlgorithm should not force layout update in
constructor.

[commit-bot: I haz the power, 2016-03-04 10:49:15]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1762993002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1762993002/1

[commit-bot: I haz the power, 2016-03-04 10:49:48]

Description was changed from

==========
Revert of TextIteratorAlgorithm should not force layout update in constructor.
(patchset #8 id:130001 of https://codereview.chromium.org/1740173003/ )

Reason for revert:
Suspected of breaking RenderViewImplTest.OnHandleKeyboardEvent on
https://build.chromium.org/p/chromium.win/builders/Win%20x64%20GN%20(dbg)

First failing build:
https://build.chromium.org/p/chromium.win/builders/Win%20x64%20GN%20%28dbg%29...

Original issue's description:
> TextIteratorAlgorithm should not force layout update in constructor.
> 
> Asking to update layout should not be done at this level -- updating layout is
part of the document/view lifecycle and should be coordinated at a higher level
than an iterator constructor.
> 
> Thus, move calling Document::updateLayoutIgnorePendingStylesheets up to the
callsites of the TextIterator constructor. These callsites should be audited and
moved up even farther up the call stack.
> 
> BUG=590295
> 
> Committed: https://crrev.com/4ed5e3b54598950731022926b09fdc8742abea74
> Cr-Commit-Position: refs/heads/master@{#379205}

TBR=esprehn@chromium.org,yosin@chromium.org,dglazkov@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=590295
==========

to

==========
Revert of TextIteratorAlgorithm should not force layout update in constructor.
(patchset #8 id:130001 of https://codereview.chromium.org/1740173003/ )

Reason for revert:
Suspected of breaking RenderViewImplTest.OnHandleKeyboardEvent on
https://build.chromium.org/p/chromium.win/builders/Win%20x64%20GN%20(dbg)

First failing build:
https://build.chromium.org/p/chromium.win/builders/Win%20x64%20GN%20%28dbg%29...

Original issue's description:
> TextIteratorAlgorithm should not force layout update in constructor.
> 
> Asking to update layout should not be done at this level -- updating layout is
part of the document/view lifecycle and should be coordinated at a higher level
than an iterator constructor.
> 
> Thus, move calling Document::updateLayoutIgnorePendingStylesheets up to the
callsites of the TextIterator constructor. These callsites should be audited and
moved up even farther up the call stack.
> 
> BUG=590295
> 
> Committed: https://crrev.com/4ed5e3b54598950731022926b09fdc8742abea74
> Cr-Commit-Position: refs/heads/master@{#379205}

TBR=esprehn@chromium.org,yosin@chromium.org,dglazkov@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=590295
==========

[commit-bot: I haz the power, 2016-03-04 10:49:48]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-04 10:50:36]

Description was changed from

==========
Revert of TextIteratorAlgorithm should not force layout update in constructor.
(patchset #8 id:130001 of https://codereview.chromium.org/1740173003/ )

Reason for revert:
Suspected of breaking RenderViewImplTest.OnHandleKeyboardEvent on
https://build.chromium.org/p/chromium.win/builders/Win%20x64%20GN%20(dbg)

First failing build:
https://build.chromium.org/p/chromium.win/builders/Win%20x64%20GN%20%28dbg%29...

Original issue's description:
> TextIteratorAlgorithm should not force layout update in constructor.
> 
> Asking to update layout should not be done at this level -- updating layout is
part of the document/view lifecycle and should be coordinated at a higher level
than an iterator constructor.
> 
> Thus, move calling Document::updateLayoutIgnorePendingStylesheets up to the
callsites of the TextIterator constructor. These callsites should be audited and
moved up even farther up the call stack.
> 
> BUG=590295
> 
> Committed: https://crrev.com/4ed5e3b54598950731022926b09fdc8742abea74
> Cr-Commit-Position: refs/heads/master@{#379205}

TBR=esprehn@chromium.org,yosin@chromium.org,dglazkov@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=590295
==========

to

==========
Revert of TextIteratorAlgorithm should not force layout update in constructor.
(patchset #8 id:130001 of https://codereview.chromium.org/1740173003/ )

Reason for revert:
Suspected of breaking RenderViewImplTest.OnHandleKeyboardEvent on
https://build.chromium.org/p/chromium.win/builders/Win%20x64%20GN%20(dbg)

First failing build:
https://build.chromium.org/p/chromium.win/builders/Win%20x64%20GN%20%28dbg%29...

Original issue's description:
> TextIteratorAlgorithm should not force layout update in constructor.
>
> Asking to update layout should not be done at this level -- updating layout is
part of the document/view lifecycle and should be coordinated at a higher level
than an iterator constructor.
>
> Thus, move calling Document::updateLayoutIgnorePendingStylesheets up to the
callsites of the TextIterator constructor. These callsites should be audited and
moved up even farther up the call stack.
>
> BUG=590295
>
> Committed: https://crrev.com/4ed5e3b54598950731022926b09fdc8742abea74
> Cr-Commit-Position: refs/heads/master@{#379205}

TBR=esprehn@chromium.org,yosin@chromium.org,dglazkov@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=590295

Committed: https://crrev.com/2d8c5743dc6cef1332eb671d9be6c54de00bd611
Cr-Commit-Position: refs/heads/master@{#379265}
==========

[commit-bot: I haz the power, 2016-03-04 10:50:37]

Patchset 1 (id:??) landed as
https://crrev.com/2d8c5743dc6cef1332eb671d9be6c54de00bd611
Cr-Commit-Position: refs/heads/master@{#379265}