Bug 56046 - [Qt] Text layout is broken on Symbian
Summary: [Qt] Text layout is broken on Symbian
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: S60 Hardware S60 3rd edition
: P2 Major
Assignee: Viatcheslav Ostapenko
URL:
Keywords: Qt
Depends on:
Blocks:
 
Reported: 2011-03-09 12:43 PST by Viatcheslav Ostapenko
Modified: 2011-03-30 09:38 PDT (History)
10 users (show)

See Also:


Attachments
Change bitfield integers to be signed by default on RVCT compiler. (1.33 KB, patch)
2011-03-09 12:53 PST, Viatcheslav Ostapenko
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Viatcheslav Ostapenko 2011-03-09 12:43:11 PST
Load any page that has text on Symbian device and text is not rendered.

Appeared to be compiler implementation difference. RVCT compiler treats bitfields as unsigned by default and because of this in RenderBlock:lineHeight comparison "m_lineHeight == -1" is always false .

There are number of other places in webkit where integers in bitfields are assumed signed (most compilers have this default).

By quote from C++(2003) standard section 9.6/3 :
“It is implementation-defined whether a plain (neither explicitly signed nor unsigned) char, short, int or long bit-field is signed or unsigned”.
Comment 1 Viatcheslav Ostapenko 2011-03-09 12:53:14 PST
Created attachment 85218 [details]
Change bitfield integers to be signed by default on RVCT compiler.
Comment 2 Laszlo Gombos 2011-03-09 14:47:48 PST
Comment on attachment 85218 [details]
Change bitfield integers to be signed by default on RVCT compiler.

r=me. Great.
Comment 3 Laszlo Gombos 2011-03-09 15:21:10 PST
CCing Nick, Daniel and Kwang as the "RVCT interest group" to make sure that they evaluate if a similar change is needed for their ports.
Comment 4 WebKit Commit Bot 2011-03-10 17:14:15 PST
Comment on attachment 85218 [details]
Change bitfield integers to be signed by default on RVCT compiler.

Clearing flags on attachment: 85218

Committed r80796: <http://trac.webkit.org/changeset/80796>
Comment 5 WebKit Commit Bot 2011-03-10 17:14:21 PST
All reviewed patches have been landed.  Closing bug.
Comment 6 WebKit Review Bot 2011-03-10 19:29:55 PST
http://trac.webkit.org/changeset/80796 might have broken SnowLeopard Intel Release (Tests)
The following tests are not passing:
fast/text/large-text-composed-char-dos.html
Comment 7 Luiz Agostini 2011-03-30 09:38:02 PDT
Revision r80796 cherry-picked into qtwebkit-2.1.x with commit 3ecec08c9c7638a214d0e7c272190a94c27fd5cc <https://gitorious.org/+qtwebkit-developers/webkit/qtwebkit/commit/3ecec08c9c7638a214d0e7c272190a94c27fd5cc>