Bug 123309

Summary:

REGRESSION(r157950): It made many tests assert on Windows, EFL, GTK

Product:

WebKit

Reporter:

Roger Fong <roger_fong>

Component:

Layout and Rendering

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Critical

CC:

commit-queue, darin, esprehn+autocc, glenn, koivisto, kondapallykalyan, ossy

Priority:

Version:

528+ (Nightly build)

Hardware:

OS:

All

Attachments:

Description	Flags
disable simple line layout on non-mac platforms	kling: review+

Description Roger Fong 2013-10-24 18:37:18 PDT

Stepping in the issue, there is an assertion failure: 
ASSERT(run.charactersLength() >= run.length());

The line before is run.setCharactersLength(text.textLength() - from);

However, stepping into this line results in stepping into DocumentMarker::setStartOffset instead.

Comment 1 Roger Fong 2013-10-24 18:38:07 PDT

Started from https://bugs.webkit.org/show_bug.cgi?id=122458

Comment 2 Csaba Osztrogonác 2013-10-25 03:38:06 PDT

and GTK and EFL too ...
- http://build.webkit.org/builders/GTK%20Linux%2064-bit%20Debug%20WK1/builds/4983
- http://build.webkit.org/builders/EFL%20Linux%2064-bit%20Release%20WK1/builds/2321

Comment 3 Antti Koivisto 2013-10-25 06:36:50 PDT

Created attachment 215172 [details]
disable simple line layout on non-mac platforms

Comment 4 Andreas Kling 2013-10-25 06:38:31 PDT

Comment on attachment 215172 [details]
disable simple line layout on non-mac platforms

r=me :|

Comment 5 Antti Koivisto 2013-10-25 06:45:51 PDT

https://trac.webkit.org/r158007

Comment 6 Antti Koivisto 2013-10-25 06:46:40 PDT

Filed bug 123338 for re-enabling.