Get rid of redundant test first-letter-rtc-crash.html

Get rid of redundant test layout test first-letter-rtc-crash.html

This patch gets or redundant layout test "first-letter-rtc-crash.html" for
improving code health and reduce maintenance cost.

The test verifies the specific edge case of |TextIterator| class. However, the
patch [1] changed first-letter handling in |TextIterator| and there are no
path verified by path.

The test is little minimized ClusterFuzz generated test script, it is hard to
read and change. Since the crash is caused by "FindString" command, the
expectation is not useful other than having it to indicate not crash.

BUG=694307, 694320
TEST=n/a; no behavior changes

Review-Url: https://codereview.chromium.org/2704273002
Cr-Commit-Position: refs/heads/master@{#451688}
Committed: https://chromium.googlesource.com/chromium/src/+/1efb7901eafb3fa00849b5f380ef8409c663b1d2

[yosin_UTC9, 2017-02-21 01:22:29]

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

[commit-bot: I haz the power, 2017-02-21 01:22:42]

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

[yosin_UTC9, 2017-02-21 01:30:21]

Description was changed from

==========
2017-02-21T10:20:57

2017-02-21T10:19:29

BUG=
==========

to

==========
Get rid of redundant test layout test first-letter-rtc-crash.html

This patch gets or redundant layout test "first-letter-rtc-crash.html" for
improving code health and reduce maintenance cost.

The test verifies the specific edge case of |TextIterator| class. However, the
patch [1] changed first-letter handling in |TextIterator| and there are no
path verified by path.

The test is little minimized ClusterFuzz generated test script, it is hard to
read and change. Since the crash is caused by "FindString" command, the
expectation is not useful other than having it to indicate not crash.

BUG=694307, 694320
TEST=n/a; no behavior changes
==========

[yosin_UTC9, 2017-02-21 01:30:54]

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

[yosin_UTC9, 2017-02-21 01:30:54]

PTAL

To remove developer pain.

[tkent, 2017-02-21 01:40:01]

lgtm

[yosin_UTC9, 2017-02-21 01:46:40]

The CQ bit was unchecked by yosin@chromium.org

[yosin_UTC9, 2017-02-21 01:46:44]

The CQ bit was checked by yosin@chromium.org

[commit-bot: I haz the power, 2017-02-21 01:46:51]

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-02-21 02:27:29]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1487641604722250, "parent_rev":
"61f7ad29a14fe66a5cf77001359da80662b6ceb3", "commit_rev":
"1efb7901eafb3fa00849b5f380ef8409c663b1d2"}

[commit-bot: I haz the power, 2017-02-21 02:28:05]

Description was changed from

==========
Get rid of redundant test layout test first-letter-rtc-crash.html

This patch gets or redundant layout test "first-letter-rtc-crash.html" for
improving code health and reduce maintenance cost.

The test verifies the specific edge case of |TextIterator| class. However, the
patch [1] changed first-letter handling in |TextIterator| and there are no
path verified by path.

The test is little minimized ClusterFuzz generated test script, it is hard to
read and change. Since the crash is caused by "FindString" command, the
expectation is not useful other than having it to indicate not crash.

BUG=694307, 694320
TEST=n/a; no behavior changes
==========

to

==========
Get rid of redundant test layout test first-letter-rtc-crash.html

This patch gets or redundant layout test "first-letter-rtc-crash.html" for
improving code health and reduce maintenance cost.

The test verifies the specific edge case of |TextIterator| class. However, the
patch [1] changed first-letter handling in |TextIterator| and there are no
path verified by path.

The test is little minimized ClusterFuzz generated test script, it is hard to
read and change. Since the crash is caused by "FindString" command, the
expectation is not useful other than having it to indicate not crash.

BUG=694307, 694320
TEST=n/a; no behavior changes

Review-Url: https://codereview.chromium.org/2704273002
Cr-Commit-Position: refs/heads/master@{#451688}
Committed:
https://chromium.googlesource.com/chromium/src/+/1efb7901eafb3fa00849b5f380ef...
==========

[commit-bot: I haz the power, 2017-02-21 02:28:07]

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