New issue
Advanced search Search tips

Issue 694307 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Feb 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: ----
Type: ----



Sign in to add a comment

webkit_tests failing on 6 builders

Project Member Reported by cfroussios@chromium.org, Feb 20 2017

Issue description

Comment 2 Deleted

Attached output sample
first-letter.txt.tar.gz
865 KB Download

Comment 4 by tkent@chromium.org, Feb 20 2017

Owner: cfroussios@chromium.org
Status: Fixed (was: Available)

Comment 5 by tkent@chromium.org, Feb 21 2017

Cc: tkent@chromium.org
 Issue 694320  has been merged into this issue.
Project Member

Comment 6 by bugdroid1@chromium.org, Feb 21 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/1efb7901eafb3fa00849b5f380ef8409c663b1d2

commit 1efb7901eafb3fa00849b5f380ef8409c663b1d2
Author: yosin <yosin@chromium.org>
Date: Tue Feb 21 02:27:25 2017

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}

[modify] https://crrev.com/1efb7901eafb3fa00849b5f380ef8409c663b1d2/third_party/WebKit/LayoutTests/TestExpectations
[delete] https://crrev.com/61f7ad29a14fe66a5cf77001359da80662b6ceb3/third_party/WebKit/LayoutTests/fast/css/first-letter-rtc-crash-expected.txt
[delete] https://crrev.com/61f7ad29a14fe66a5cf77001359da80662b6ceb3/third_party/WebKit/LayoutTests/fast/css/first-letter-rtc-crash.html

Sign in to add a comment