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

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

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

Status:	RESOLVED FIXED

Alias:	None

Product:	WebKit
Classification:	Unclassified
Component:	Layout and Rendering (show other bugs)
Version:	528+ (Nightly build)
Hardware:	PC All

Importance:	P1 Critical
Assignee:	Nobody

URL:
Keywords:

Depends on:
Blocks:

Reported:	2013-10-24 18:37 PDT by Roger Fong
Modified:	2013-10-25 06:47 PDT (History)
CC List:	7 users (show)

See Also:

Attachments
disable simple line layout on non-mac platforms (1.32 KB, patch) 2013-10-25 06:36 PDT, Antti Koivisto	kling: review+	Details \| Formatted Diff \| Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

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.