[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