Space out issuing of spellcheck requests to speed up layout test.

Space out issuing of spellcheck requests to speed up layout test.

Avoid issuing all spellcheck requests (by focusing elements) in one go
as this queues up a number of tasks and timers that it will require going
back to the event loop many times to process and handle. As the actual
test also relies on timers and setTimeout() this delays the completion
of the test considerably.

Restructure the test, interleaving the element focusing (=> spellcheck
request) with asynchronously checking the spellcheck result. Test completes
earlier as a result.

R=haraken
BUG=356957
TEST=editing/spelling/spellcheck-editable-on-focus.html

Committed: https://crrev.com/7acaf8fd9fb9680ba4ba0956f289d43c4f6ef647
Cr-Commit-Position: refs/heads/master@{#380895}

[sof, 2016-03-12 17:32:31]

sigbjornf@opera.com changed reviewers:
+ haraken@chromium.org, yosin@chromium.org

[sof, 2016-03-12 17:32:32]

please take a look.

just fixing a slow-running test; mac_blink_dbg reports the test runs in 0.2secs
rather than 14.

[sof, 2016-03-12 17:52:03]

Description was changed from

==========
Space out issuing of spellcheck request to speed up layout test.

Avoid issuing all spellcheck requests (by focusing elements) in one go
as this queues up a number of tasks and timers that it will require going
back to the event loop many times to process and handle. As the actual
test also relies on timers and setTimeout() this delays the completion
of the test considerably.

Restructure the test, interleaving the element focusing (=> spellcheck
request) with asynchronously checking the spellcheck result. Test completes
earlier as a result.

R=
BUG=356957
TEST=editing/spelling/spellcheck-editable-on-focus.html
==========

to

==========
Space out issuing of spellcheck requests to speed up layout test.

Avoid issuing all spellcheck requests (by focusing elements) in one go
as this queues up a number of tasks and timers that it will require going
back to the event loop many times to process and handle. As the actual
test also relies on timers and setTimeout() this delays the completion
of the test considerably.

Restructure the test, interleaving the element focusing (=> spellcheck
request) with asynchronously checking the spellcheck result. Test completes
earlier as a result.

R=
BUG=356957
TEST=editing/spelling/spellcheck-editable-on-focus.html
==========

[haraken, 2016-03-12 23:57:13]

LGTM

[sof, 2016-03-13 07:28:46]

Description was changed from

==========
Space out issuing of spellcheck requests to speed up layout test.

Avoid issuing all spellcheck requests (by focusing elements) in one go
as this queues up a number of tasks and timers that it will require going
back to the event loop many times to process and handle. As the actual
test also relies on timers and setTimeout() this delays the completion
of the test considerably.

Restructure the test, interleaving the element focusing (=> spellcheck
request) with asynchronously checking the spellcheck result. Test completes
earlier as a result.

R=
BUG=356957
TEST=editing/spelling/spellcheck-editable-on-focus.html
==========

to

==========
Space out issuing of spellcheck requests to speed up layout test.

Avoid issuing all spellcheck requests (by focusing elements) in one go
as this queues up a number of tasks and timers that it will require going
back to the event loop many times to process and handle. As the actual
test also relies on timers and setTimeout() this delays the completion
of the test considerably.

Restructure the test, interleaving the element focusing (=> spellcheck
request) with asynchronously checking the spellcheck result. Test completes
earlier as a result.

R=haraken
BUG=356957
TEST=editing/spelling/spellcheck-editable-on-focus.html
==========

[sof, 2016-03-13 07:29:14]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2016-03-13 07:29:27]

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

[commit-bot: I haz the power, 2016-03-13 08:22:08]

Description was changed from

==========
Space out issuing of spellcheck requests to speed up layout test.

Avoid issuing all spellcheck requests (by focusing elements) in one go
as this queues up a number of tasks and timers that it will require going
back to the event loop many times to process and handle. As the actual
test also relies on timers and setTimeout() this delays the completion
of the test considerably.

Restructure the test, interleaving the element focusing (=> spellcheck
request) with asynchronously checking the spellcheck result. Test completes
earlier as a result.

R=haraken
BUG=356957
TEST=editing/spelling/spellcheck-editable-on-focus.html
==========

to

==========
Space out issuing of spellcheck requests to speed up layout test.

Avoid issuing all spellcheck requests (by focusing elements) in one go
as this queues up a number of tasks and timers that it will require going
back to the event loop many times to process and handle. As the actual
test also relies on timers and setTimeout() this delays the completion
of the test considerably.

Restructure the test, interleaving the element focusing (=> spellcheck
request) with asynchronously checking the spellcheck result. Test completes
earlier as a result.

R=haraken
BUG=356957
TEST=editing/spelling/spellcheck-editable-on-focus.html
==========

[commit-bot: I haz the power, 2016-03-13 08:22:09]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-03-13 08:23:28]

Description was changed from

==========
Space out issuing of spellcheck requests to speed up layout test.

Avoid issuing all spellcheck requests (by focusing elements) in one go
as this queues up a number of tasks and timers that it will require going
back to the event loop many times to process and handle. As the actual
test also relies on timers and setTimeout() this delays the completion
of the test considerably.

Restructure the test, interleaving the element focusing (=> spellcheck
request) with asynchronously checking the spellcheck result. Test completes
earlier as a result.

R=haraken
BUG=356957
TEST=editing/spelling/spellcheck-editable-on-focus.html
==========

to

==========
Space out issuing of spellcheck requests to speed up layout test.

Avoid issuing all spellcheck requests (by focusing elements) in one go
as this queues up a number of tasks and timers that it will require going
back to the event loop many times to process and handle. As the actual
test also relies on timers and setTimeout() this delays the completion
of the test considerably.

Restructure the test, interleaving the element focusing (=> spellcheck
request) with asynchronously checking the spellcheck result. Test completes
earlier as a result.

R=haraken
BUG=356957
TEST=editing/spelling/spellcheck-editable-on-focus.html

Committed: https://crrev.com/7acaf8fd9fb9680ba4ba0956f289d43c4f6ef647
Cr-Commit-Position: refs/heads/master@{#380895}
==========

[commit-bot: I haz the power, 2016-03-13 08:23:29]

Patchset 3 (id:??) landed as
https://crrev.com/7acaf8fd9fb9680ba4ba0956f289d43c4f6ef647
Cr-Commit-Position: refs/heads/master@{#380895}