Use internals.setMarker in layout tests in paint/spellmarkers/

Use internals.setMarker in layout tests in paint/spellmarkers/

This patch removes the dependency of layout tests in paint/spellmarkers/
on the implementation of Blink's spell checker, by making them add markers
directly with internals.setMarker().

As a result, they are no longer affected by spell checker rearchitecturing
(e.g., idle time spell checker).

BUG=698883
Review-Url: https://codereview.chromium.org/2737643002
Cr-Commit-Position: refs/heads/master@{#455073}
Committed: https://chromium.googlesource.com/chromium/src/+/e58d543442b2f3049b0bd22f2a537c05cbed6a2a

[Xiaocheng, 2017-03-06 23:14:02]

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

[commit-bot: I haz the power, 2017-03-06 23:14:54]

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

[Xiaocheng, 2017-03-07 00:02:25]

xiaochengh@chromium.org changed reviewers:
+ fmalita@chromium.org, schenney@chromium.org

[Xiaocheng, 2017-03-07 00:02:26]

PTAL.

[Xiaocheng, 2017-03-07 00:06:50]

Description was changed from

==========
Use internals.setMarker in layout tests under paint/spellmarkers/

BUG=698883
==========

to

==========
Use internals.setMarker in layout tests in paint/spellmarkers/

This patch removes the dependency of layout tests in paint/spellmarkers/
on the implementation of Blink's spell checker, by making them add markers
directly with internals.setMarker().

As a result, they are no longer affected by spell checker rearchitecturing
(e.g., idle time spell checker).

BUG=698883
==========

[commit-bot: I haz the power, 2017-03-07 00:38:48]

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

[commit-bot: I haz the power, 2017-03-07 00:38:49]

Dry run: This issue passed the CQ dry run.

[Stephen Chennney, 2017-03-07 14:46:48]

Thanks for making my tests more direct. lgtm

[Stephen Chennney, 2017-03-07 14:46:53]

The CQ bit was checked by schenney@chromium.org

[commit-bot: I haz the power, 2017-03-07 14:47:34]

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, 2017-03-07 14:54:36]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1488898013920330, "parent_rev":
"59ec56271d10fdb839339febdb8463cc1c3b2f21", "commit_rev":
"e58d543442b2f3049b0bd22f2a537c05cbed6a2a"}

[commit-bot: I haz the power, 2017-03-07 14:55:13]

Description was changed from

==========
Use internals.setMarker in layout tests in paint/spellmarkers/

This patch removes the dependency of layout tests in paint/spellmarkers/
on the implementation of Blink's spell checker, by making them add markers
directly with internals.setMarker().

As a result, they are no longer affected by spell checker rearchitecturing
(e.g., idle time spell checker).

BUG=698883
==========

to

==========
Use internals.setMarker in layout tests in paint/spellmarkers/

This patch removes the dependency of layout tests in paint/spellmarkers/
on the implementation of Blink's spell checker, by making them add markers
directly with internals.setMarker().

As a result, they are no longer affected by spell checker rearchitecturing
(e.g., idle time spell checker).

BUG=698883

Review-Url: https://codereview.chromium.org/2737643002
Cr-Commit-Position: refs/heads/master@{#455073}
Committed:
https://chromium.googlesource.com/chromium/src/+/e58d543442b2f3049b0bd22f2a53...
==========

[commit-bot: I haz the power, 2017-03-07 14:55:14]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/e58d543442b2f3049b0bd22f2a53...