[webkit-reviews] review granted: [Bug 25239] Add a settings entry to en/disable web font support : [Attachment 30397] my patch for this
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Fri May 15 13:13:31 PDT 2009
David Kilzer (ddkilzer) <ddkilzer at webkit.org> has granted Antti Koivisto
<koivisto at iki.fi>'s request for review:
Bug 25239: Add a settings entry to en/disable web font support
https://bugs.webkit.org/show_bug.cgi?id=25239
Attachment 30397: my patch for this
https://bugs.webkit.org/attachment.cgi?id=30397&action=review
------- Additional Comments from David Kilzer (ddkilzer) <ddkilzer at webkit.org>
>+ Settings* settings = m_document ? m_document->frame() ?
m_document->frame()->settings() : 0 : 0;
I think it's easier to read if you do:
Settings* settings = m_document && m_document->frame() ?
m_document->frame()->settings() : 0;
r=me
Dave
More information about the webkit-reviews
mailing list