Bug 108614 - [Qt] Incorrect FontPlatformDataCacheKey impl. causes text selection wrong
Summary: [Qt] Incorrect FontPlatformDataCacheKey impl. causes text selection wrong
Status: RESOLVED INVALID
Alias: None
Product: WebKit
Classification: Unclassified
Component: Text (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-02-01 04:31 PST by Chen Zhixiang
Modified: 2014-02-03 03:24 PST (History)
0 users

See Also:

Attachments
Test case for this bug (455 bytes, text/html)
2013-02-01 04:31 PST, Chen Zhixiang 		no flags Details
Patch targeted upon Qt-4.8.0 (our internal project uses this edition), should be easily migrated to Qt-4.8.4 (3.95 KB, patch)
2013-02-01 04:36 PST, Chen Zhixiang 		no flags 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 Chen Zhixiang 2013-02-01 04:31:30 PST 
Created attachment 186007 [details]
Test case for this bug

Load the testcase.html using Qt-4.8.4 demos/browser,
And double click on text "xy", you will see "small" is wrongly selected highlighted.

Reason:
FontPlatformDataCacheKey doesn't contain m_smallCaps field, resulting in 2 RenderText elements sharing the same QFontPrivate objects.
Comment 1 Chen Zhixiang 2013-02-01 04:32:10 PST 
Also reported to Qt: https://bugreports.qt-project.org/browse/QTBUG-29413
Comment 2 Chen Zhixiang 2013-02-01 04:36:33 PST 
Created attachment 186008 [details]
Patch targeted upon Qt-4.8.0 (our internal project uses this edition), should be easily migrated to Qt-4.8.4
Comment 3 Jocelyn Turcotte 2014-02-03 03:24:38 PST 
=== Bulk closing of Qt bugs ===

If you believe that this bug report is still relevant for a non-Qt port of webkit.org, please re-open it and remove [Qt] from the summary.

If you believe that this is still an important QtWebKit bug, please fill a new report at https://bugreports.qt-project.org and add a link to this issue. See http://qt-project.org/wiki/ReportingBugsInQt for additional guidelines.