Audit the use of updateStyleAndLayoutIgnorePendingStylesheets in toNormalizedEphemeralRange

Audit the use of updateStyleAndLayoutIgnorePendingStylesheets in toNormalizedEphemeralRange

BUG=590369
Committed: https://crrev.com/fa41120b705ec7ae5fc8c8f280bd76a05bfc97ef
Cr-Commit-Position: refs/heads/master@{#425279}

[Xiaocheng, 2016-10-14 04:12:05]

The CQ bit was checked by xiaochengh@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-14 04:12:13]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Xiaocheng, 2016-10-14 04:14:40]

The CQ bit was checked by xiaochengh@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-14 04:14:44]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Xiaocheng, 2016-10-14 04:15:46]

The CQ bit was checked by xiaochengh@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-14 04:16:01]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-14 05:33:20]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-14 05:33:21]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[Xiaocheng, 2016-10-14 07:05:34]

The CQ bit was checked by xiaochengh@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-14 07:05:43]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Xiaocheng, 2016-10-14 08:06:59]

xiaochengh@chromium.org changed reviewers:
+ tkent@chromium.org, yosin@chromium.org

[Xiaocheng, 2016-10-14 08:06:59]

PTAL.

[yosin_UTC9, 2016-10-14 08:13:00]

lgtm w/ small nit

https://codereview.chromium.org/2414263002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/editing/spellcheck/SpellChecker.cpp (right):

https://codereview.chromium.org/2414263002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/editing/spellcheck/SpellChecker.cpp:998: const
EphemeralRange range = selection.toNormalizedEphemeralRange();
s/const EphemeralRange/const EphemeralRange&/

[tkent, 2016-10-14 08:24:23]

lgtm

[Xiaocheng, 2016-10-14 08:35:00]

The CQ bit was checked by xiaochengh@chromium.org

[Xiaocheng, 2016-10-14 08:35:00]

The patchset sent to the CQ was uploaded after l-g-t-m from tkent@chromium.org,
yosin@chromium.org
Link to the patchset: https://codereview.chromium.org/2414263002/#ps60001
(title: "fix nit")

[Xiaocheng, 2016-10-14 08:35:14]

Thanks!

Committing with nit fixed.

https://codereview.chromium.org/2414263002/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/editing/spellcheck/SpellChecker.cpp (right):

https://codereview.chromium.org/2414263002/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/editing/spellcheck/SpellChecker.cpp:998: const
EphemeralRange range = selection.toNormalizedEphemeralRange();
On 2016/10/14 at 08:13:00, Yosi_UTC9 wrote:
> s/const EphemeralRange/const EphemeralRange&/

Done.

[commit-bot: I haz the power, 2016-10-14 08:35:17]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-14 09:34:40]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-10-14 09:36:10]

Description was changed from

==========
Audit the use of updateStyleAndLayoutIgnorePendingStylesheets in
toNormalizedEphemeralRange

BUG=590369
==========

to

==========
Audit the use of updateStyleAndLayoutIgnorePendingStylesheets in
toNormalizedEphemeralRange

BUG=590369

Committed: https://crrev.com/fa41120b705ec7ae5fc8c8f280bd76a05bfc97ef
Cr-Commit-Position: refs/heads/master@{#425279}
==========

[commit-bot: I haz the power, 2016-10-14 09:36:12]

Patchset 4 (id:??) landed as
https://crrev.com/fa41120b705ec7ae5fc8c8f280bd76a05bfc97ef
Cr-Commit-Position: refs/heads/master@{#425279}