RESOLVED FIXED 59080
Add FontItalics and FontSmallCaps enums to be used instead of boolean values. 
https://bugs.webkit.org/show_bug.cgi?id=59080
Summary Add FontItalics and FontSmallCaps enums to be used instead of boolean values.
Luke Macpherson
Reported 2011-04-21 00:25:31 PDT
Add FontItalics and FontSmallCaps enums to be used instead of boolean values.
Attachments
Patch (8.63 KB, patch)
2011-04-21 00:28 PDT, Luke Macpherson 		no flags
DetailsFormatted DiffDiff
Patch (8.74 KB, patch)
2011-04-21 17:39 PDT, Luke Macpherson 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Luke Macpherson
Comment 1 2011-04-21 00:28:49 PDT
Created attachment 90501 [details] Patch
WebKit Review Bot
Comment 2 2011-04-21 00:53:42 PDT
Attachment 90501 [details] did not build on chromium: Build output: http://queues.webkit.org/results/8494028
Eric Seidel (no email)
Comment 3 2011-04-21 01:59:54 PDT
Comment on attachment 90501 [details] Patch I'm surprised there are no callers of italic() or smallCaps()?
Luke Macpherson
Comment 4 2011-04-21 16:48:13 PDT
(In reply to comment #3) > (From update of attachment 90501 [details]) > I'm surprised there are no callers of italic() or smallCaps()? There are two cases: 1) the form if(x->italic()), which has the same behavior since the *Off enum values are defined to be zero. 2) the form x->setItalic(y->italic), which is also covered by this CL.
Luke Macpherson
Comment 5 2011-04-21 17:39:47 PDT
Created attachment 90642 [details] Patch
WebKit Commit Bot
Comment 6 2011-04-22 00:40:48 PDT
Comment on attachment 90642 [details] Patch Clearing flags on attachment: 90642 Committed r84609: <http://trac.webkit.org/changeset/84609>
WebKit Commit Bot
Comment 7 2011-04-22 00:40:53 PDT
All reviewed patches have been landed. Closing bug.
WebKit Review Bot
Comment 8 2011-04-22 01:45:54 PDT
http://trac.webkit.org/changeset/84609 might have broken WinCairo Debug (Build)
Note You need to log in before you can comment on or make changes to this bug.