Reviewed by Geoff.
authordarin <darin@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 11 Jul 2006 06:04:43 +0000 (06:04 +0000)
committerdarin <darin@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 11 Jul 2006 06:04:43 +0000 (06:04 +0000)
        - fix http://bugzilla.opendarwin.org/show_bug.cgi?id=9811
          rdar://problem/4619572
          OPTION text contained in OPTGROUP appears at the wrong size

        * dom/Node.h:
        * dom/Node.cpp: (WebCore::Node::renderStyle): Made a const member function.
        The derived classes were overriding with const member functions, and I could
        change either this or the derived -- decided to do this.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@15330 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebCore/ChangeLog
WebCore/dom/Node.cpp
WebCore/dom/Node.h

index 8d22c304307ba755c767230a8b2814e5c25fbea3..b16285d71eb9887bfdc0010b8086283de988f9f8 100644 (file)
@@ -2,7 +2,21 @@
 
         Reviewed by Geoff.
 
 
         Reviewed by Geoff.
 
-        - fix rdar://problem/4619534 http://bugzilla.opendarwin.org/show_bug.cgi?id=9810
+        - fix http://bugzilla.opendarwin.org/show_bug.cgi?id=9811
+          rdar://problem/4619572
+          OPTION text contained in OPTGROUP appears at the wrong size
+
+        * dom/Node.h:
+        * dom/Node.cpp: (WebCore::Node::renderStyle): Made a const member function.
+        The derived classes were overriding with const member functions, and I could
+        change either this or the derived -- decided to do this.
+
+2006-07-10  Darin Adler  <darin@apple.com>
+
+        Reviewed by Geoff.
+
+        - fix http://bugzilla.opendarwin.org/show_bug.cgi?id=9810
+          rdar://problem/4619534
           OPTGROUP label text is using the default font instead of Lucida Grande
 
         * rendering/RenderThemeMac.mm: (WebCore::RenderThemeMac::setFontFromControlSize):
           OPTGROUP label text is using the default font instead of Lucida Grande
 
         * rendering/RenderThemeMac.mm: (WebCore::RenderThemeMac::setFontFromControlSize):
index f586d9a62ddd9e663b3f3f4f6a37464c13a423e1..9cd770d3cacab952e6744106e5567d90aca445a0 100644 (file)
@@ -940,12 +940,13 @@ RenderObject *Node::createRenderer(RenderArena *arena, RenderStyle *style)
     return 0;
 }
 
     return 0;
 }
 
-RenderStyle* Node::renderStyle()
+RenderStyle* Node::renderStyle() const
 { 
     return m_renderer ? m_renderer->style() : 0; 
 }
 
 { 
     return m_renderer ? m_renderer->style() : 0; 
 }
 
-void Node::setRenderStyle(RenderStyle* s) {
+void Node::setRenderStyle(RenderStyle* s)
+{
     if (m_renderer)
         m_renderer->setStyle(s); 
 }
     if (m_renderer)
         m_renderer->setStyle(s); 
 }
index ebf7df4f30d6a600d7e3e43403170bc528255aad..039e1be360026ee397b7e692ea1603248f65bf69 100644 (file)
@@ -366,8 +366,8 @@ public:
     virtual RenderObject* createRenderer(RenderArena*, RenderStyle*);
     
     // Wrapper for nodes that don't have a renderer, but still cache the style (like HTMLOptionElement).
     virtual RenderObject* createRenderer(RenderArena*, RenderStyle*);
     
     // Wrapper for nodes that don't have a renderer, but still cache the style (like HTMLOptionElement).
-    virtual RenderStyle* renderStyle();
-    virtual void setRenderStyle(RenderStyle* s);
+    virtual RenderStyle* renderStyle() const;
+    virtual void setRenderStyle(RenderStyle*);
 
     // -----------------------------------------------------------------------------
     // Notification of document structure changes
 
     // -----------------------------------------------------------------------------
     // Notification of document structure changes