[Webkit-unassigned] [Bug 18965] Allow platform specific adjustments to the default style sheet

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sun Jun 8 14:02:17 PDT 2008


http://bugs.webkit.org/show_bug.cgi?id=18965


darin at apple.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #21387|review?                     |review+
               Flag|                            |




------- Comment #7 from darin at apple.com  2008-06-08 14:02 PDT -------
(From update of attachment 21387)
+#include "CSSStyleSheet.h"

This include is not needed in RenderTheme.cpp.

+void RenderTheme::adjustDefaultStyleSheet(CSSStyleSheet* style) {}

For people compiling with unused argument warnings it would be better to omit
the word "style".

I don't see any major benefit to using the "all in one line" format here. This
could just be laid out in the normal way.

Seems OK, r=me


-- 
Configure bugmail: http://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.



More information about the webkit-unassigned mailing list