Bug 66268
| Summary: | editing/deleting/regional-indicators.html fails on GTK and Qt | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Adam Barth <abarth> |
| Component: | Text | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | ap, kadam, mrobinson, ossy, schenney, steveblock, zoltan |
| Priority: | P2 | ||
| Version: | 528+ (Nightly build) | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Adam Barth
editing/deleting/regional-indicators.html was introduced in http://trac.webkit.org/changeset/93068 and fails since introduction. Alexey says that this is an minor bug in the handling of these characters in the Chromium port.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Alexey Proskuryakov
I've taken another look, and now I'm wondering about why this test doesn't fail elsewhere. The fixed code is under #if PLATFORM(MAC), and cross-platform fallback certainly returns a wrong result for that case.
There seems to be more difference between platforms than I thought.
Csaba Osztrogonác
It fails on GTK and Qt too.
Zoltan Horvath
I skipped it for Qt in http://trac.webkit.org/changeset/93088.
Steve Block
Skipped for Chromium in http://trac.webkit.org/changeset/93075
Steve Block
This test is also timing out on Chromium Linux dbg bots due to an assertion failure. See http://build.chromium.org/p/chromium.webkit/builders/Webkit%20Linux%20%28deps%29%28dbg%29%281%29/builds/9926
Added TIMEOUT expectation with http://trac.webkit.org/changeset/93099
Ádám Kallai
The test is still failing on Qt.
Martin Robinson
Expectations look good now.