Bug 108614

Summary: [Qt] Incorrect FontPlatformDataCacheKey impl. causes text selection wrong
Product: WebKit Reporter: Chen Zhixiang <chenzx>
Component: TextAssignee: Nobody <webkit-unassigned>
Status: RESOLVED INVALID    
Severity: Normal    
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Test case for this bug
none
Patch targeted upon Qt-4.8.0 (our internal project uses this edition), should be easily migrated to Qt-4.8.4 none

Chen Zhixiang
Reported 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.
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
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Chen Zhixiang
Comment 1 2013-02-01 04:32:10 PST
Also reported to Qt: https://bugreports.qt-project.org/browse/QTBUG-29413
Chen Zhixiang
Comment 2 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
Jocelyn Turcotte
Comment 3 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.
Note You need to log in before you can comment on or make changes to this bug.