Add layout test for the crash on infinite loop in SVG textPath.

Add a layout test for the crash on infinite loop in SVG textPath.

The reported crash can't be reproduced anymore.

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

[rwlbuis, 2014-02-28 18:42:38]

Looks good but can be imporoved a bit.

For the description I'd say:

"
Add a layout test for the crash on infinite loop in SVG textPath.

The reported crash can't be reproduced anymore.
BUG=
"

https://codereview.chromium.org/180213005/diff/1/LayoutTests/svg/text/svg-tex...
File LayoutTests/svg/text/svg-textpath-infinite-loop-crash.html (right):

https://codereview.chromium.org/180213005/diff/1/LayoutTests/svg/text/svg-tex...
LayoutTests/svg/text/svg-textpath-infinite-loop-crash.html:4: <div
id="goal-line1">Test for infinite loop in textPath.</div>
You can remove id's if they are not useful. Basically these tests have to be
minimal but still reproduce the problem.

[ostap, 2014-02-28 19:25:43]

https://codereview.chromium.org/180213005/diff/1/LayoutTests/svg/text/svg-tex...
File LayoutTests/svg/text/svg-textpath-infinite-loop-crash.html (right):

https://codereview.chromium.org/180213005/diff/1/LayoutTests/svg/text/svg-tex...
LayoutTests/svg/text/svg-textpath-infinite-loop-crash.html:4: <div
id="goal-line1">Test for infinite loop in textPath.</div>
On 2014/02/28 18:42:38, rwlbuis wrote:
> You can remove id's if they are not useful. Basically these tests have to be
> minimal but still reproduce the problem.

Done.

[rwlbuis, 2014-02-28 19:56:17]

lgtm

[commit-bot: I haz the power, 2014-02-28 19:56:32]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/sl.ostapenko@samsung.com/180213005/20001

[commit-bot: I haz the power, 2014-02-28 21:23:06]

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

[commit-bot: I haz the power, 2014-02-28 21:23:06]

Retried try job too often on blink_android_compile_rel for step(s) webkit_lint
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=blink_andr...

[ostap, 2014-02-28 21:30:09]

The CQ bit was checked by sl.ostapenko@samsung.com

[commit-bot: I haz the power, 2014-02-28 21:30:26]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/sl.ostapenko@samsung.com/180213005/20001

[commit-bot: I haz the power, 2014-02-28 22:39:17]

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

[commit-bot: I haz the power, 2014-02-28 22:39:18]

Retried try job too often on mac_blink_rel for step(s) webkit_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_blink_...

[ostap, 2014-02-28 22:41:57]

The CQ bit was checked by sl.ostapenko@samsung.com

[commit-bot: I haz the power, 2014-02-28 22:42:10]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/sl.ostapenko@samsung.com/180213005/20001

[commit-bot: I haz the power, 2014-03-01 00:02:22]

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

[commit-bot: I haz the power, 2014-03-01 00:02:22]

Retried try job too often on blink_android_compile_dbg for step(s) webkit_lint
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=blink_andr...

[ostap, 2014-03-01 04:10:10]

The CQ bit was checked by sl.ostapenko@samsung.com

[commit-bot: I haz the power, 2014-03-01 04:10:25]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/sl.ostapenko@samsung.com/180213005/20001

[commit-bot: I haz the power, 2014-03-01 05:22:43]

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

[commit-bot: I haz the power, 2014-03-01 05:22:44]

Retried try job too often on mac_blink_rel for step(s) webkit_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_blink_...

[ostap, 2014-03-01 06:10:47]

The CQ bit was checked by sl.ostapenko@samsung.com

[commit-bot: I haz the power, 2014-03-01 06:11:00]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/sl.ostapenko@samsung.com/180213005/20001

[commit-bot: I haz the power, 2014-03-01 07:48:37]

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

[commit-bot: I haz the power, 2014-03-01 07:48:38]

Retried try job too often on mac_blink_rel for step(s) webkit_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_blink_...

[rwlbuis, 2014-03-01 12:40:27]

On 2014/02/28 17:05:28, ostap wrote:

Oops, I forgot something when reviewing this. You should either make the test
dumpAsText or add a reftest. The way it is now will likely generate pixel tests,
which is maybe why it is failing.

[ostap, 2014-03-02 05:44:40]

On 2014/03/01 12:40:27, rwlbuis wrote:
> On 2014/02/28 17:05:28, ostap wrote:
> 
> Oops, I forgot something when reviewing this. You should either make the test
> dumpAsText or add a reftest. The way it is now will likely generate pixel
tests,
> which is maybe why it is failing.

Hm! Then why it works fine on my build and doesn't generate pixel test?
In any case, I'll do ref test.

[rwlbuis, 2014-03-03 12:47:03]

lgtm

[commit-bot: I haz the power, 2014-03-03 12:47:16]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/sl.ostapenko@samsung.com/180213005/40001

[commit-bot: I haz the power, 2014-03-03 21:09:10]

Change committed as 168330