RESOLVED FIXED 55303
[Qt] Use WTF ref counting for FontPlatformDataPrivate
https://bugs.webkit.org/show_bug.cgi?id=55303
Summary [Qt] Use WTF ref counting for FontPlatformDataPrivate
Andreas Kling
Reported 2011-02-26 12:28:35 PST
FontPlatformDataPrivate currently has hand-rolled ref counting for some reason. We should use RefCounted/RefPtr instead.
Attachments
Proposed patch (5.48 KB, patch)
2011-02-26 12:39 PST, Andreas Kling
no flags
Proposed patch (11.37 KB, patch)
2011-02-26 13:35 PST, Andreas Kling
kenneth: review+
Andreas Kling
Comment 1 2011-02-26 12:39:07 PST
Created attachment 83946 [details] Proposed patch
Andreas Kling
Comment 2 2011-02-26 13:35:14 PST
Created attachment 83951 [details] Proposed patch
Kenneth Rohde Christiansen
Comment 3 2011-02-27 04:46:50 PST
Comment on attachment 83951 [details] Proposed patch View in context: https://bugs.webkit.org/attachment.cgi?id=83951&action=review Nice! > Source/WebCore/platform/graphics/qt/FontPlatformData.h:101 > + return 0; Nice, fixing style as well :-) > Source/WebCore/platform/graphics/qt/FontPlatformData.h:137 > + // WebKit allows font size zero but QFont does not. WebCore allows a font size of zero, but QFont does not.
Andreas Kling
Comment 4 2011-02-27 07:56:17 PST
Note You need to log in before you can comment on or make changes to this bug.