Reviewed by Mitz.
authorweinig <weinig@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 26 Oct 2006 20:23:55 +0000 (20:23 +0000)
committerweinig <weinig@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 26 Oct 2006 20:23:55 +0000 (20:23 +0000)
        Fix for http://bugs.webkit.org/show_bug.cgi?id=11424
        Cleanup rendering code

        - Assorted code cleanup.

        * rendering/AutoTableLayout.cpp:
        * rendering/AutoTableLayout.h:
        (WebCore::AutoTableLayout::totalPercent):
        * rendering/CounterListItem.h:
        * rendering/CounterNode.cpp:
        * rendering/CounterNode.h:
        (WebCore::CounterNode::~CounterNode):
        (WebCore::CounterNode::isReset):
        * rendering/CounterResetNode.cpp:
        * rendering/CounterResetNode.h:
        * rendering/DataRef.h:
        (WebCore::DataRef::DataRef):
        * rendering/DeprecatedRenderSelect.h:
        * rendering/DeprecatedSlider.cpp:
        * rendering/DeprecatedSlider.h:
        * rendering/EllipsisBox.cpp:
        * rendering/EllipsisBox.h:
        (WebCore::EllipsisBox::EllipsisBox):
        * rendering/FixedTableLayout.cpp:
        * rendering/FixedTableLayout.h:
        * rendering/GapRects.h:
        * rendering/InlineBox.cpp:
        * rendering/InlineBox.h:
        (WebCore::InlineBox::InlineBox):
        (WebCore::InlineBox::setConstructed):
        (WebCore::InlineBox::clearTruncation):
        (WebCore::InlineBox::markDirty):
        * rendering/InlineFlowBox.cpp:
        * rendering/InlineFlowBox.h:
        (WebCore::InlineFlowBox::InlineFlowBox):
        (WebCore::InlineFlowBox::setConstructed):
        (WebCore::InlineFlowBox::setEdges):
        (WebCore::InlineFlowBox::setVerticalOverflowPositions):
        (WebCore::InlineFlowBox::setVerticalSelectionPositions):
        * rendering/InlineRunBox.h:
        (WebCore::InlineRunBox::paintBackgroundAndBorder):
        (WebCore::InlineRunBox::paintDecorations):
        * rendering/InlineTextBox.cpp:
        * rendering/InlineTextBox.h:
        (WebCore::InlineTextBox::InlineTextBox):
        (WebCore::InlineTextBox::end):
        (WebCore::InlineTextBox::textObject):
        * rendering/Length.h:
        (WebCore::Length::Length):
        * rendering/ListMarkerBox.cpp:
        * rendering/ListMarkerBox.h:
        * rendering/RenderPartObject.h:
        * rendering/RootInlineBox.cpp:
        * rendering/RootInlineBox.h:
        * rendering/SVGInlineFlowBox.cpp:
        * rendering/SVGInlineFlowBox.h:
        (WebCore::SVGInlineFlowBox::SVGInlineFlowBox):
        * rendering/SVGRootInlineBox.cpp:
        * rendering/SVGRootInlineBox.h:
        (WebCore::SVGRootInlineBox::SVGRootInlineBox):
        * rendering/TableLayout.h:
        (WebCore::TableLayout::TableLayout):
        (WebCore::TableLayout::~TableLayout):
        * rendering/bidi.h:
        (WebCore::BidiStatus::BidiStatus):
        (WebCore::BidiRun::BidiRun):
        * rendering/break_lines.h:

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

37 files changed:
WebCore/ChangeLog
WebCore/rendering/AutoTableLayout.cpp
WebCore/rendering/AutoTableLayout.h
WebCore/rendering/CounterListItem.h
WebCore/rendering/CounterNode.cpp
WebCore/rendering/CounterNode.h
WebCore/rendering/CounterResetNode.cpp
WebCore/rendering/CounterResetNode.h
WebCore/rendering/DataRef.h
WebCore/rendering/DeprecatedRenderSelect.h
WebCore/rendering/DeprecatedSlider.cpp
WebCore/rendering/DeprecatedSlider.h
WebCore/rendering/EllipsisBox.cpp
WebCore/rendering/EllipsisBox.h
WebCore/rendering/FixedTableLayout.cpp
WebCore/rendering/FixedTableLayout.h
WebCore/rendering/GapRects.h
WebCore/rendering/InlineBox.cpp
WebCore/rendering/InlineBox.h
WebCore/rendering/InlineFlowBox.cpp
WebCore/rendering/InlineFlowBox.h
WebCore/rendering/InlineRunBox.h
WebCore/rendering/InlineTextBox.cpp
WebCore/rendering/InlineTextBox.h
WebCore/rendering/Length.h
WebCore/rendering/ListMarkerBox.cpp
WebCore/rendering/ListMarkerBox.h
WebCore/rendering/RenderPartObject.h
WebCore/rendering/RootInlineBox.cpp
WebCore/rendering/RootInlineBox.h
WebCore/rendering/SVGInlineFlowBox.cpp
WebCore/rendering/SVGInlineFlowBox.h
WebCore/rendering/SVGRootInlineBox.cpp
WebCore/rendering/SVGRootInlineBox.h
WebCore/rendering/TableLayout.h
WebCore/rendering/bidi.h
WebCore/rendering/break_lines.h

index 3690474..f40e85d 100644 (file)
@@ -1,3 +1,75 @@
+2006-10-26  Sam Weinig  <sam.weinig@gmail.com>
+
+        Reviewed by Mitz.
+
+        Fix for http://bugs.webkit.org/show_bug.cgi?id=11424
+        Cleanup rendering code
+
+        - Assorted code cleanup.
+
+        * rendering/AutoTableLayout.cpp:
+        * rendering/AutoTableLayout.h:
+        (WebCore::AutoTableLayout::totalPercent):
+        * rendering/CounterListItem.h:
+        * rendering/CounterNode.cpp:
+        * rendering/CounterNode.h:
+        (WebCore::CounterNode::~CounterNode):
+        (WebCore::CounterNode::isReset):
+        * rendering/CounterResetNode.cpp:
+        * rendering/CounterResetNode.h:
+        * rendering/DataRef.h:
+        (WebCore::DataRef::DataRef):
+        * rendering/DeprecatedRenderSelect.h:
+        * rendering/DeprecatedSlider.cpp:
+        * rendering/DeprecatedSlider.h:
+        * rendering/EllipsisBox.cpp:
+        * rendering/EllipsisBox.h:
+        (WebCore::EllipsisBox::EllipsisBox):
+        * rendering/FixedTableLayout.cpp:
+        * rendering/FixedTableLayout.h:
+        * rendering/GapRects.h:
+        * rendering/InlineBox.cpp:
+        * rendering/InlineBox.h:
+        (WebCore::InlineBox::InlineBox):
+        (WebCore::InlineBox::setConstructed):
+        (WebCore::InlineBox::clearTruncation):
+        (WebCore::InlineBox::markDirty):
+        * rendering/InlineFlowBox.cpp:
+        * rendering/InlineFlowBox.h:
+        (WebCore::InlineFlowBox::InlineFlowBox):
+        (WebCore::InlineFlowBox::setConstructed):
+        (WebCore::InlineFlowBox::setEdges):
+        (WebCore::InlineFlowBox::setVerticalOverflowPositions):
+        (WebCore::InlineFlowBox::setVerticalSelectionPositions):
+        * rendering/InlineRunBox.h:
+        (WebCore::InlineRunBox::paintBackgroundAndBorder):
+        (WebCore::InlineRunBox::paintDecorations):
+        * rendering/InlineTextBox.cpp:
+        * rendering/InlineTextBox.h:
+        (WebCore::InlineTextBox::InlineTextBox):
+        (WebCore::InlineTextBox::end):
+        (WebCore::InlineTextBox::textObject):
+        * rendering/Length.h:
+        (WebCore::Length::Length):
+        * rendering/ListMarkerBox.cpp:
+        * rendering/ListMarkerBox.h:
+        * rendering/RenderPartObject.h:
+        * rendering/RootInlineBox.cpp:
+        * rendering/RootInlineBox.h:
+        * rendering/SVGInlineFlowBox.cpp:
+        * rendering/SVGInlineFlowBox.h:
+        (WebCore::SVGInlineFlowBox::SVGInlineFlowBox):
+        * rendering/SVGRootInlineBox.cpp:
+        * rendering/SVGRootInlineBox.h:
+        (WebCore::SVGRootInlineBox::SVGRootInlineBox):
+        * rendering/TableLayout.h:
+        (WebCore::TableLayout::TableLayout):
+        (WebCore::TableLayout::~TableLayout):
+        * rendering/bidi.h:
+        (WebCore::BidiStatus::BidiStatus):
+        (WebCore::BidiRun::BidiRun):
+        * rendering/break_lines.h:
+
 2006-10-26  Justin Garcia  <justin.garcia@apple.com>
 
         Reviewed by john
index 8e070d1..3aa6c8c 100644 (file)
@@ -20,6 +20,7 @@
  * the Free Software Foundation, Inc., 59 Temple Place - Suite 330,
  * Boston, MA 02111-1307, USA.
  */
+
 #include "config.h"
 #include "AutoTableLayout.h"
 
index 04489a3..de65f46 100644 (file)
@@ -32,8 +32,7 @@ namespace WebCore {
 class RenderTable;
 class RenderTableCell;
 
-class AutoTableLayout : public TableLayout
-{
+class AutoTableLayout : public TableLayout {
 public:
     AutoTableLayout(RenderTable*);
     ~AutoTableLayout();
@@ -44,13 +43,17 @@ public:
 protected:
     void fullRecalc();
     void recalcColumn(int effCol);
-    int totalPercent() const {
+
+    void calcPercentages() const;
+    int totalPercent() const
+    {
         if (m_percentagesDirty)
             calcPercentages();
         return m_totalPercent;
     }
-    void calcPercentages() const;
+
     int calcEffectiveWidth();
+
     void insertSpanCell(RenderTableCell*);
 
     struct Layout {
@@ -77,6 +80,6 @@ protected:
     mutable unsigned short m_totalPercent;
 };
 
-}
+} // namespace WebCore
 
-#endif
+#endif // AutoTableLayout_H
index d6a7b6b..3bf16b2 100644 (file)
 
 namespace WebCore {
 
-class CounterListItem : public CounterNode
-{
+class CounterListItem : public CounterNode {
 public:
     int count() const;
 
     virtual void recount(bool setDirty = true);
     virtual void setSelfDirty();
-
 };
 
-} //namespace
+} // namespace WebCore
 
-#endif
+#endif // CounterListItem_H
index c967250..4128f1a 100644 (file)
@@ -108,4 +108,4 @@ void CounterNode::setParentDirty()
         m_renderer->setNeedsLayoutAndMinMaxRecalc();
 }
 
-} // namespace
+} // namespace WebCore
index 9d318cc..53c15a8 100644 (file)
@@ -20,8 +20,8 @@
  * Boston, MA 02111-1307, USA.
  *
 */
-#ifndef Counter_Node_h_
-#define Counter_Node_h_
+#ifndef CounterNode_h
+#define CounterNode_h
 
 #include "RenderObject.h"
 
@@ -37,7 +37,7 @@ class CounterResetNode;
 class CounterNode {
 public:
     CounterNode(RenderObject*);
-    virtual ~CounterNode() {}
+    virtual ~CounterNode() { }
 
     CounterResetNode* parent() const { return m_parent; }
     CounterNode* previousSibling() const { return m_previous; }
@@ -45,7 +45,7 @@ public:
     virtual CounterNode* firstChild() const { return 0; }
     virtual CounterNode* lastChild() const { return 0; }
     virtual void insertAfter(CounterNode* newChild, CounterNode* refChild);
-    virtual void removeChild (CounterNode* oldChild);
+    virtual void removeChild(CounterNode*);
     void remove();
 
     int value() const { return m_value; }
@@ -54,7 +54,7 @@ public:
     void setCount(int c) { m_count = c; }
     void setHasSeparator() { m_hasSeparator = true; }
 
-    virtual bool isReset() const{ return false; }
+    virtual bool isReset() const { return false; }
     virtual CounterNode* recountAndGetNext(bool setDirty = true);
     virtual void recountTree(bool setDirty = true);
     virtual void setSelfDirty();
index 0da567c..22b38aa 100644 (file)
@@ -153,4 +153,4 @@ void CounterResetNode::updateTotal(int value)
         m_total = value;
 }
 
-} // namespace
+} // namespace WebCore
index f761b86..20f813b 100644 (file)
@@ -20,8 +20,8 @@
  * Boston, MA 02111-1307, USA.
  *
 */
-#ifndef Counter_Reset_Node_h_
-#define Counter_Reset_Node_h_
+#ifndef CounterResetNode_h
+#define CounterResetNode_h
 
 #include "CounterNode.h"
 
@@ -34,14 +34,13 @@ namespace WebCore {
 
 // Implementation of counter-reset and root
 class CounterResetNode : public CounterNode {
-
 public:
     CounterResetNode(RenderObject*);
 
     virtual CounterNode* firstChild() const { return m_first; };
     virtual CounterNode* lastChild() const { return m_last; };
     virtual void insertAfter(CounterNode* newChild, CounterNode* refChild);
-    virtual void removeChild (CounterNode*);
+    virtual void removeChild(CounterNode*);
 
     virtual bool isReset() const { return true; };
     virtual CounterNode* recountAndGetNext(bool setDirty = true);
index 1fc1c44..b8a47ad 100644 (file)
 
 namespace WebCore {
 
-template <typename T> class DataRef
-{
+template <typename T> class DataRef {
 public:
-    DataRef() : m_data(0) { }
+    DataRef()
+        : m_data(0)
+    {
+    }
 
     DataRef(const DataRef<T>& d)
     {
@@ -98,6 +100,6 @@ private:
     T* m_data;
 };
 
-}
+} // namespace WebCore
 
-#endif
+#endif // DataRef_h
index 29a1d79..a4ba94c 100644 (file)
@@ -29,7 +29,7 @@
 #include "RenderFormElement.h"
 
 namespace WebCore {
-    
+
     class HTMLSelectElement;
     class ListBox;
 
index cc40468..de35732 100644 (file)
@@ -27,8 +27,8 @@
 #include "DeprecatedSlider.h"
 
 #include "EventNames.h"
-#include "HTMLNames.h"
 #include "HTMLInputElement.h"
+#include "HTMLNames.h"
 #include "Slider.h"
 
 using std::min;
index 72142e9..83a829f 100644 (file)
 namespace WebCore {
 
     class HTMLInputElement;
-    
+
     class DeprecatedSlider : public RenderFormElement {
     public:
         DeprecatedSlider(HTMLInputElement*);
-        
+
         virtual const char* renderName() const { return "DeprecatedSlider"; }
 
         virtual void calcMinMaxWidth();
@@ -45,7 +45,6 @@ namespace WebCore {
         virtual void valueChanged(Widget*);
     };
 
-
 } // namespace WebCore
 
 #endif // DeprecatedSlider_h
index 932696a..3e51455 100644 (file)
@@ -18,7 +18,6 @@
  * the Free Software Foundation, Inc., 59 Temple Place - Suite 330,
  * Boston, MA 02111-1307, USA.
  */
-// -------------------------------------------------------------------------
 
 #include "config.h"
 #include "EllipsisBox.h"
@@ -84,4 +83,4 @@ bool EllipsisBox::nodeAtPoint(RenderObject::NodeInfo& info, int x, int y, int tx
     return false;
 }
 
-}
+} // namespace WebCore
index 945806e..656e865 100644 (file)
@@ -28,23 +28,16 @@ namespace WebCore {
 
 class EllipsisBox : public InlineBox {
 public:
-    EllipsisBox(RenderObject* obj, const AtomicString& ellipsisStr, InlineFlowBox* p,
-                int w, int y, int h, int b, bool firstLine, InlineBox* markupBox)
-        : InlineBox(obj)
+    EllipsisBox(RenderObject* obj, const AtomicString& ellipsisStr, InlineFlowBox* parent,
+                int width, int y, int height, int baseline, bool firstLine, InlineBox* markupBox)
+        : InlineBox(obj, 0, y, width, height, baseline, firstLine, true, false, false, 0, 0, parent)
         , m_str(ellipsisStr)
+        , m_markupBox(markupBox)
     {
-        m_parent = p;
-        m_width = w;
-        m_y = y;
-        m_height = h;
-        m_baseline = b;
-        m_firstLine = firstLine;
-        m_constructed = true;
-        m_markupBox = markupBox;
     }
-    
-    virtual void paint(RenderObject::PaintInfo& i, int _tx, int _ty);
-    virtual bool nodeAtPoint(RenderObject::NodeInfo& info, int _x, int _y, int _tx, int _ty);
+
+    virtual void paint(RenderObject::PaintInfo&, int tx, int ty);
+    virtual bool nodeAtPoint(RenderObject::NodeInfo&, int x, int y, int tx, int ty);
 
 private:
     AtomicString m_str;
index d696544..7d4ba68 100644 (file)
@@ -336,4 +336,4 @@ void FixedTableLayout::layout()
 
 #undef DEBUG_LAYOUT
 
-}
+} // namespace WebCore
index 6a59beb..168af7e 100644 (file)
@@ -31,8 +31,7 @@ namespace WebCore {
 
 class RenderTable;
 
-class FixedTableLayout : public TableLayout
-{
+class FixedTableLayout : public TableLayout {
 public:
     FixedTableLayout(RenderTable*);
     ~FixedTableLayout();
@@ -46,6 +45,6 @@ protected:
     Vector<Length> m_width;
 };
 
-}
+} // namespace WebCore
 
-#endif
+#endif // FixedTableLayout_H
index fb109ff..901632f 100644 (file)
@@ -59,6 +59,6 @@ namespace WebCore {
         IntRect m_right;
     };
 
-}
+} // namespace WebCore
 
-#endif
+#endif // GapRects_h
index 5dcbe9b..776f05e 100644 (file)
  * the Free Software Foundation, Inc., 59 Temple Place - Suite 330,
  * Boston, MA 02111-1307, USA.
  */
+
 #include "config.h"
 #include "InlineBox.h"
 
 #include "InlineFlowBox.h"
-#include "RootInlineBox.h"
 #include "RenderArena.h"
+#include "RootInlineBox.h"
 
 using namespace std;
 
@@ -238,7 +239,7 @@ int InlineBox::placeEllipsisBox(bool ltr, int blockEdge, int ellipsisWidth, bool
     return -1;
 }
 
-}
+} // namespace WebCore
 
 #ifndef NDEBUG
 
index c1a58d5..d28c433 100644 (file)
@@ -31,8 +31,7 @@ class RootInlineBox;
 
 // InlineBox represents a rectangle that occurs on a line.  It corresponds to
 // some RenderObject (i.e., it represents a portion of that RenderObject).
-class InlineBox
-{
+class InlineBox {
 public:
     InlineBox(RenderObject* obj)
         : m_object(obj)
@@ -51,6 +50,24 @@ public:
     {
     }
 
+    InlineBox(RenderObject* obj, int x, int y, int width, int height, int baseline, bool firstLine, bool constructed,
+              bool dirty, bool extracted, InlineBox* next, InlineBox* prev, InlineFlowBox* parent)
+        : m_object(obj)
+        , m_x(x)
+        , m_y(y)
+        , m_width(width)
+        , m_height(height)
+        , m_baseline(baseline)
+        , m_firstLine(firstLine)
+        , m_constructed(constructed)
+        , m_dirty(dirty)
+        , m_extracted(extracted)
+        , m_next(next)
+        , m_prev(prev)
+        , m_parent(parent)
+    {
+    }
+
     virtual ~InlineBox() { }
 
     virtual void destroy(RenderArena*);
@@ -63,9 +80,9 @@ public:
 
     virtual void adjustPosition(int dx, int dy);
 
-    virtual void paint(RenderObject::PaintInfo&, int _tx, int _ty);
+    virtual void paint(RenderObject::PaintInfo&, int tx, int ty);
     virtual bool nodeAtPoint(RenderObject::NodeInfo&, int x, int y, int tx, int ty);
-    
+
     // Overloaded new operator.
     void* operator new(size_t, RenderArena*) throw();
 
@@ -75,7 +92,7 @@ public:
 private:
     // The normal operator new is disallowed.
     void* operator new(size_t) throw();
-    
+
 public:
 #ifndef NDEBUG
     void showTreeForThis() const;
@@ -89,11 +106,13 @@ public:
     virtual bool isText() const { return false; }
 
     bool isConstructed() { return m_constructed; }
-    virtual void setConstructed() {
+    virtual void setConstructed()
+    {
         m_constructed = true;
         if (m_next)
             m_next->setConstructed();
     }
+
     void setExtracted(bool b = true) { m_extracted = b; }
     
     void setFirstLineStyleBit(bool f) { m_firstLine = f; }
@@ -148,10 +167,10 @@ public:
     virtual int caretMaxOffset() const;
     virtual unsigned caretMaxRenderedOffset() const;
     
-    virtual void clearTruncation() {};
+    virtual void clearTruncation() { }
 
     bool isDirty() const { return m_dirty; }
-    void markDirty(bool dirty=true) { m_dirty = dirty; }
+    void markDirty(bool dirty = true) { m_dirty = dirty; }
 
     void dirtyLineBoxes();
     
@@ -169,7 +188,7 @@ public: // FIXME: Would like to make this protected, but methods are accessing t
     int m_width;
     int m_height;
     int m_baseline;
-    
+
     bool m_firstLine : 1;
     bool m_constructed : 1;
     bool m_dirty : 1;
@@ -181,11 +200,11 @@ public: // FIXME: Would like to make this protected, but methods are accessing t
     InlineFlowBox* m_parent; // The box that contains us.
 };
 
-} //namespace
+} // namespace WebCore
 
 #ifndef NDEBUG
 // Outside the WebCore namespace for ease of invocation from gdb.
 void showTree(const WebCore::InlineBox*);
 #endif
 
-#endif
+#endif // InlineBox_H
index 32ef279..50e18fa 100644 (file)
@@ -18,6 +18,7 @@
  * the Free Software Foundation, Inc., 59 Temple Place - Suite 330,
  * Boston, MA 02111-1307, USA.
  */
+
 #include "config.h"
 #include "InlineFlowBox.h"
 
 #include "EllipsisBox.h"
 #include "GraphicsContext.h"
 #include "InlineTextBox.h"
-#include "RootInlineBox.h"
 #include "RenderBlock.h"
 #include "RenderFlow.h"
 #include "RenderListMarker.h"
 #include "RenderTableCell.h"
+#include "RootInlineBox.h"
 
 using namespace std;
 
@@ -849,4 +850,4 @@ void InlineFlowBox::clearTruncation()
         box->clearTruncation();
 }
 
-} // namespace
+} // namespace WebCore
index 33e73f3..9443ebb 100644 (file)
 
 namespace WebCore {
 
-class InlineFlowBox : public InlineRunBox
-{
+class InlineFlowBox : public InlineRunBox {
 public:
     InlineFlowBox(RenderObject* obj)
-    :InlineRunBox(obj)
+        : InlineRunBox(obj)
+        , m_firstChild(0)
+        , m_lastChild(0)
+        , m_maxHorizontalShadow(0)
+        , m_includeLeftEdge(false)
+        , m_includeRightEdge(false)
+        , m_hasTextChildren(false)
     {
-        m_firstChild = 0;
-        m_lastChild = 0;
-        m_includeLeftEdge = m_includeRightEdge = false;
-        m_hasTextChildren = false;
-        m_maxHorizontalShadow = 0;
     }
 
     RenderFlow* flowObject();
@@ -46,37 +46,38 @@ public:
 
     InlineFlowBox* prevFlowBox() const { return static_cast<InlineFlowBox*>(m_prevLine); }
     InlineFlowBox* nextFlowBox() const { return static_cast<InlineFlowBox*>(m_nextLine); }
-    
+
     InlineBox* firstChild() { return m_firstChild; }
     InlineBox* lastChild() { return m_lastChild; }
 
     virtual InlineBox* firstLeafChild();
     virtual InlineBox* lastLeafChild();
-    InlineBox* firstLeafChildAfterBox(InlineBox* start=0);
-    InlineBox* lastLeafChildBeforeBox(InlineBox* start=0);
-        
-    virtual void setConstructed() {
+    InlineBox* firstLeafChildAfterBox(InlineBox* start = 0);
+    InlineBox* lastLeafChildBeforeBox(InlineBox* start = 0);
+
+    virtual void setConstructed()
+    {
         InlineBox::setConstructed();
         if (m_firstChild)
             m_firstChild->setConstructed();
     }
-    void addToLine(InlineBox* child);
 
-    virtual void deleteLine(RenderArena* arena);
+    void addToLine(InlineBox* child);
+    virtual void deleteLine(RenderArena*);
     virtual void extractLine();
     virtual void attachLine();
     virtual void adjustPosition(int dx, int dy);
 
     virtual void clearTruncation();
-    
-    virtual void paintBackgroundAndBorder(RenderObject::PaintInfo& i, int _tx, int _ty);
-    void paintBackgrounds(GraphicsContext* p, const Color& c, const BackgroundLayer* bgLayer,
-                          int my, int mh, int _tx, int _ty, int w, int h);
-    void paintBackground(GraphicsContext* p, const Color& c, const BackgroundLayer* bgLayer,
-                         int my, int mh, int _tx, int _ty, int w, int h);
-    virtual void paintDecorations(RenderObject::PaintInfo& i, int _tx, int _ty, bool paintedChildren = false);
-    virtual void paint(RenderObject::PaintInfo& i, int _tx, int _ty);
-    virtual bool nodeAtPoint(RenderObject::NodeInfo& i, int x, int y, int tx, int ty);
+
+    virtual void paintBackgroundAndBorder(RenderObject::PaintInfo&, int tx, int ty);
+    void paintBackgrounds(GraphicsContext*, const Color&, const BackgroundLayer*,
+                          int my, int mh, int tx, int ty, int w, int h);
+    void paintBackground(GraphicsContext*, const Color&, const BackgroundLayer*,
+                         int my, int mh, int tx, int ty, int w, int h);
+    virtual void paintDecorations(RenderObject::PaintInfo&, int tx, int ty, bool paintedChildren = false);
+    virtual void paint(RenderObject::PaintInfo&, int tx, int ty);
+    virtual bool nodeAtPoint(RenderObject::NodeInfo&, int x, int y, int tx, int ty);
 
     int marginBorderPaddingLeft();
     int marginBorderPaddingRight();
@@ -86,13 +87,15 @@ public:
     int borderRight() { if (includeRightEdge()) return object()->borderRight(); return 0; }
     int paddingLeft() { if (includeLeftEdge()) return object()->paddingLeft(); return 0; }
     int paddingRight() { if (includeRightEdge()) return object()->paddingRight(); return 0; }
-    
+
     bool includeLeftEdge() { return m_includeLeftEdge; }
     bool includeRightEdge() { return m_includeRightEdge; }
-    void setEdges(bool includeLeft, bool includeRight) {
+    void setEdges(bool includeLeft, bool includeRight)
+    {
         m_includeLeftEdge = includeLeft;
         m_includeRightEdge = includeRight;
     }
+
     virtual bool hasTextChildren() { return m_hasTextChildren; }
 
     // Helper functions used during line construction and placement.
@@ -109,12 +112,12 @@ public:
                               int& topPosition, int& bottomPosition, int& selectionTop, int& selectionBottom);
     void shrinkBoxesWithNoTextChildren(int topPosition, int bottomPosition);
     
-    virtual void setVerticalOverflowPositions(int top, int bottom) {}
-    virtual void setVerticalSelectionPositions(int top, int bottom) {}
+    virtual void setVerticalOverflowPositions(int top, int bottom) { }
+    virtual void setVerticalSelectionPositions(int top, int bottom) { }
     int maxHorizontalShadow() const { return m_maxHorizontalShadow; }
 
     void removeChild(InlineBox* child);
-    
+
     virtual RenderObject::SelectionState selectionState();
 
     virtual bool canAccommodateEllipsis(bool ltr, int blockEdge, int ellipsisWidth);
@@ -129,11 +132,11 @@ protected:
     bool m_hasTextChildren : 1;
 };
 
-} //namespace
+} // namespace WebCore
 
 #ifndef NDEBUG
 // Outside the WebCore namespace for ease of invocation from gdb.
 void showTree(const WebCore::InlineBox*);
 #endif
 
-#endif
+#endif // InlineFlowBox_H
index ea75859..6256269 100644 (file)
@@ -27,8 +27,7 @@
 
 namespace WebCore {
 
-class InlineRunBox : public InlineBox
-{
+class InlineRunBox : public InlineBox {
 public:
     InlineRunBox(RenderObject* obj)
         : InlineBox(obj)
@@ -42,14 +41,14 @@ public:
     void setNextLineBox(InlineRunBox* n) { m_nextLine = n; }
     void setPreviousLineBox(InlineRunBox* p) { m_prevLine = p; }
 
-    virtual void paintBackgroundAndBorder(RenderObject::PaintInfo&, int _tx, int _ty) {};
-    virtual void paintDecorations(RenderObject::PaintInfo&, int _tx, int _ty, bool paintedChildren = false) {};
+    virtual void paintBackgroundAndBorder(RenderObject::PaintInfo&, int tx, int ty) { }
+    virtual void paintDecorations(RenderObject::PaintInfo&, int tx, int ty, bool paintedChildren = false) { }
     
 protected:
     InlineRunBox* m_prevLine;  // The previous box that also uses our RenderObject
     InlineRunBox* m_nextLine;  // The next box that also uses our RenderObject
 };
 
-} //namespace
+} // namespace WebCore
 
-#endif
+#endif // InlineRunBox_H
index 033e265..dff2a80 100644 (file)
@@ -764,4 +764,4 @@ bool InlineTextBox::containsCaretOffset(int offset) const
     return true;
 }
 
-}
+} // namespace WebCore
index ef25f6c..520b3f8 100644 (file)
  *
  */
 
-#ifndef KHTML_InlineTextBox_H
-#define KHTML_InlineTextBox_H
+#ifndef InlineTextBox_H
+#define InlineTextBox_H
 
 #include "DocumentMarker.h"
-#include "RenderText.h"
 #include "InlineRunBox.h"
+#include "RenderText.h"
 
 namespace WebCore {
 
@@ -39,44 +39,43 @@ class StringImpl;
 class MarkedTextUnderline;
 class Position;
 
-class InlineTextBox : public InlineRunBox
-{
+class InlineTextBox : public InlineRunBox {
 public:
     InlineTextBox(RenderObject* obj)
-    :InlineRunBox(obj)
+        : InlineRunBox(obj)
+        , m_start(0)
+        , m_len(0)
+        , m_truncation(cNoTruncation)
+        , m_reversed(false)
+        , m_treatAsText(true)
+        , m_toAdd(0)
     {
-        m_start = 0;
-        m_len = 0;
-        m_reversed = false;
-        m_treatAsText = true;
-        m_toAdd = 0;
-        m_truncation = cNoTruncation;
     }
-    
+
     InlineTextBox* nextTextBox() const { return static_cast<InlineTextBox*>(nextLineBox()); }
     InlineTextBox* prevTextBox() const { return static_cast<InlineTextBox*>(prevLineBox()); }
-    
+
     unsigned start() const { return m_start; }
-    unsigned end() const { return m_len ? m_start+m_len-1 : m_start; }
+    unsigned end() const { return m_len ? m_start + m_len - 1 : m_start; }
     unsigned len() const { return m_len; }
-  
+
     void setStart(unsigned start) { m_start = start; }
     void setLen(unsigned len) { m_len = len; }
 
     void offsetRun(int d) { m_start += d; }
 
-    void destroy(RenderArena* arena);
-    
+    void destroy(RenderArena*);
+
     IntRect selectionRect(int absx, int absy, int startPos, int endPos);
     bool isSelected(int startPos, int endPos) const;
     void selectionStartEnd(int& sPos, int& ePos);
-    
-    virtual void paint(RenderObject::PaintInfo& i, int tx, int ty);
-    virtual bool nodeAtPoint(RenderObject::NodeInfo& i, int x, int y, int tx, int ty);
+
+    virtual void paint(RenderObject::PaintInfo&, int tx, int ty);
+    virtual bool nodeAtPoint(RenderObject::NodeInfo&, int x, int y, int tx, int ty);
 
     RenderText* textObject() const;
-    
-    virtual void deleteLine(RenderArena* arena);
+
+    virtual void deleteLine(RenderArena*);
     virtual void extractLine();
     virtual void attachLine();
 
@@ -89,14 +88,14 @@ public:
 
     // Overloaded new operator.  Derived classes must override operator new
     // in order to allocate out of the RenderArena.
-    void* operator new(size_t sz, RenderArena* renderArena) throw();    
+    void* operator new(size_t, RenderArena*) throw();  
 
     // Overridden to prevent the normal delete from being called.
-    void operator delete(void* ptr, size_t sz);
-    
+    void operator delete(void*, size_t);
+
 private:
     // The normal operator new is disallowed.
-    void* operator new(size_t sz) throw();
+    void* operator new(size_t) throw();
 
 public:
     void setSpaceAdd(int add) { m_width -= m_toAdd; m_toAdd = add; m_width += m_toAdd; }
@@ -105,7 +104,7 @@ public:
     virtual bool isInlineTextBox() { return true; }    
     virtual bool isText() const { return m_treatAsText; }
     void setIsText(bool b) { m_treatAsText = b; }
-    
+
     void paintDecoration(GraphicsContext*, int tx, int ty, int decoration);
     void paintSelection(GraphicsContext*, int tx, int ty, RenderStyle*, const Font*);
     void paintMarkedTextBackground(GraphicsContext*, int tx, int ty, RenderStyle*, const Font*, int startPos, int endPos);
@@ -119,16 +118,16 @@ public:
     virtual int caretMinOffset() const;
     virtual int caretMaxOffset() const;
     virtual unsigned caretMaxRenderedOffset() const;
-    
+
     int textPos() const;
-    int offsetForPosition(int _x, bool includePartialGlyphs = true) const;
+    int offsetForPosition(int x, bool includePartialGlyphs = true) const;
     int positionForOffset(int offset) const;
 
     bool containsCaretOffset(int offset) const; // false for offset after line break
-    
+
     int m_start;
     unsigned short m_len;
-    
+
     int m_truncation; // Where to truncate when text overflow is applied.  We use special constants to
                       // denote no truncation (the whole run paints) and full truncation (nothing paints at all).
 
@@ -141,11 +140,11 @@ private:
     friend class RenderText;
 };
 
-inline RenderText *InlineTextBox::textObject() const
+inline RenderTextInlineTextBox::textObject() const
 {
-    return static_cast<RenderText *>(m_object);
+    return static_cast<RenderText*>(m_object);
 }
 
-}
+} // namespace WebCore
 
-#endif
+#endif // InlineTextBox_H
index b60181e..6a3b0fb 100644 (file)
@@ -20,8 +20,8 @@
     Boston, MA 02111-1307, USA.
 */
 
-#ifndef LENGTH_H
-#define LENGTH_H
+#ifndef Length_H
+#define Length_H
 
 namespace WebCore {
 
@@ -30,10 +30,20 @@ namespace WebCore {
     enum LengthType { Auto, Relative, Percent, Fixed, Static, Intrinsic, MinIntrinsic };
 
     struct Length {
-        Length() : m_value(0) { }
-        Length(LengthType t) : m_value(t) { }
-        Length(int v, LengthType t, bool q = false) : m_value((v * 16) | (q << 3) | t) { }
-            // FIXME: Doesn't work if the passed-in value is very large!
+        Length()
+            : m_value(0)
+        {
+        }
+
+        Length(LengthType t)
+            : m_value(t)
+        {
+        }
+
+        Length(int v, LengthType t, bool q = false)
+            : m_value((v * 16) | (q << 3) | t) // FIXME: Doesn't work if the passed-in value is very large!
+        {
+        }
 
         bool operator==(const Length& o) const { return m_value == o.m_value; }
         bool operator!=(const Length& o) const { return m_value != o.m_value; }
@@ -84,6 +94,6 @@ namespace WebCore {
         int m_value;
     };
 
-}
+} // namespace WebCore
 
-#endif
+#endif // Length_H
index b350b6c..73ff514 100644 (file)
@@ -22,6 +22,7 @@
  * Boston, MA 02111-1307, USA.
  *
  */
+
 #include "config.h"
 #include "ListMarkerBox.h"
 
@@ -41,4 +42,4 @@ bool ListMarkerBox::isText() const
     return !static_cast<RenderListMarker*>(object())->listImage();
 }
 
-}
+} // namespace WebCore
index 6e78cdd..a86af7e 100644 (file)
 
 namespace WebCore {
 
-class ListMarkerBox : public InlineBox
-{
+class ListMarkerBox : public InlineBox {
 public:
     ListMarkerBox(RenderObject*);
-    
+
     virtual bool isText() const;
 };
 
-} //namespace
+} // namespace WebCore
 
-#endif
+#endif // ListMarkerBox_H
index 5d600d0..4ccb50c 100644 (file)
@@ -29,8 +29,7 @@
 
 namespace WebCore {
 
-class RenderPartObject : public RenderPart
-{
+class RenderPartObject : public RenderPart {
 public:
     RenderPartObject(HTMLElement*);
 
@@ -42,6 +41,6 @@ public:
     virtual void viewCleared();
 };
 
-}
+} // namespace WebCore
 
-#endif
+#endif // RenderPartObject_H
index 2de20d1..f74ae58 100644 (file)
  * the Free Software Foundation, Inc., 59 Temple Place - Suite 330,
  * Boston, MA 02111-1307, USA.
  */
+
 #include "config.h"
 #include "RootInlineBox.h"
 
+#include "Document.h"
 #include "EllipsisBox.h"
-#include "RenderBlock.h"
 #include "GraphicsContext.h"
-#include "Document.h"
+#include "RenderBlock.h"
+
 #if PLATFORM(MAC)
 #include "FrameMac.h"
 #endif
@@ -319,4 +321,4 @@ void RootInlineBox::setLineBreakInfo(RenderObject* obj, unsigned breakPos, BidiS
         m_lineBreakBidiStatus = *status;
 }
 
-}
+} // namespace WebCore
index 7eebd1b..29be21c 100644 (file)
@@ -31,8 +31,7 @@ namespace WebCore {
 class EllipsisBox;
 struct GapRects;
 
-class RootInlineBox : public InlineFlowBox
-{
+class RootInlineBox : public InlineFlowBox {
 public:
     RootInlineBox(RenderObject* obj)
         : InlineFlowBox(obj)
@@ -86,29 +85,28 @@ public:
     virtual int placeEllipsisBox(bool ltr, int blockEdge, int ellipsisWidth, bool&);
 
     EllipsisBox* ellipsisBox() const { return m_ellipsisBox; }
-    void paintEllipsisBox(RenderObject::PaintInfo& i, int _tx, int _ty) const;
-    bool hitTestEllipsisBox(RenderObject::NodeInfo& info, int _x, int _y, int _tx, int _ty,
-                            HitTestAction hitTestAction, bool inBox);
+    void paintEllipsisBox(RenderObject::PaintInfo&, int tx, int ty) const;
+    bool hitTestEllipsisBox(RenderObject::NodeInfo&, int x, int y, int tx, int ty, HitTestAction, bool inBox);
     
     virtual void clearTruncation();
 
 #if PLATFORM(MAC)
     void addHighlightOverflow();
-    void paintCustomHighlight(RenderObject::PaintInfo& i, int tx, int ty, const AtomicString& highlightType);
+    void paintCustomHighlight(RenderObject::PaintInfo&, int tx, int ty, const AtomicString& highlightType);
 #endif
 
-    virtual void paint(RenderObject::PaintInfo& i, int _tx, int _ty);
-    virtual bool nodeAtPoint(RenderObject::NodeInfo& i, int x, int y, int tx, int ty);
+    virtual void paint(RenderObject::PaintInfo&, int tx, int ty);
+    virtual bool nodeAtPoint(RenderObject::NodeInfo&, int x, int y, int tx, int ty);
 
     bool hasSelectedChildren() const { return m_hasSelectedChildren; }
     void setHasSelectedChildren(bool b);
-    
+
     virtual RenderObject::SelectionState selectionState();
     InlineBox* firstSelectedBox();
     InlineBox* lastSelectedBox();
     
     GapRects fillLineSelectionGap(int selTop, int selHeight, RenderBlock* rootBlock, int blockX, int blockY, 
-                                  int tx, int ty, const RenderObject::PaintInfo* i);
+                                  int tx, int ty, const RenderObject::PaintInfo*);
     
     RenderBlock* block() const;
 
@@ -116,7 +114,7 @@ public:
     int selectionBottom() { return m_selectionBottom; }
     int selectionHeight() { return max(0, selectionBottom() - selectionTop()); }
  
-    InlineBox* closestLeafChildForXPos(int _x, int _tx);
+    InlineBox* closestLeafChildForXPos(int x, int tx);
 
 protected:
     // Normally we are only as tall as the style on our block dictates, but we might have content
@@ -152,6 +150,6 @@ protected:
     EllipsisBox* m_ellipsisBox;
 };
 
-} //namespace
+} // namespace WebCore
 
-#endif
+#endif // RootInlineBox_H
index 6bfa98c..0a2dc4f 100644 (file)
  *
  */
 
+#include "config.h"
 
 #ifdef SVG_SUPPORT
 
-#include "config.h"
 #include "SVGInlineFlowBox.h"
-#include "KRenderingDevice.h"
-#include "KCanvasRenderingStyle.h"
+
+#include "GraphicsContext.h"
+#include "InlineTextBox.h"
 #include "KCanvasClipper.h"
 #include "KCanvasMasker.h"
-#include "InlineTextBox.h"
+#include "KCanvasRenderingStyle.h"
+#include "KRenderingDevice.h"
 #include "RootInlineBox.h"
-#include "SVGTextPositioningElement.h"
 #include "SVGLengthList.h"
-#include "GraphicsContext.h"
+#include "SVGTextPositioningElement.h"
 #include <wtf/OwnPtr.h>
 
 using namespace std;
@@ -290,5 +291,6 @@ void placeSVGFlowVertically(InlineFlowBox* flow, int& heightOfBlock)
     flow->setVerticalSelectionPositions(top, bottom);
 }
 
-}
+} // namespace WebCore
+
 #endif // SVG_SUPPORT
index f5b8eb4..571ec36 100644 (file)
 
 namespace WebCore {
 
-void paintSVGInlineFlow(InlineFlowBox* flow, RenderObject* object, RenderObject::PaintInfo& i, 
-                        int _tx, int _ty);
-int placeSVGFlowHorizontally(InlineFlowBox* flow, int x, int& leftPosition, int& rightPosition, bool& needsWordSpacing);
-void placeSVGFlowVertically(InlineFlowBox* flow, int& heightOfBlock);
+void paintSVGInlineFlow(InlineFlowBox*, RenderObject*, RenderObject::PaintInfo&, int tx, int ty);
+int placeSVGFlowHorizontally(InlineFlowBox*, int x, int& leftPosition, int& rightPosition, bool& needsWordSpacing);
+void placeSVGFlowVertically(InlineFlowBox*, int& heightOfBlock);
 
 class SVGInlineFlowBox : public InlineFlowBox {
 public:
-    SVGInlineFlowBox(RenderObject* obj) : InlineFlowBox(obj) {}
-    virtual void paint(RenderObject::PaintInfo& i, int _tx, int _ty);
+    SVGInlineFlowBox(RenderObject* obj)
+        : InlineFlowBox(obj)
+    {
+    }
+
+    virtual void paint(RenderObject::PaintInfo&, int tx, int ty);
     virtual int placeBoxesHorizontally(int x, int& leftPosition, int& rightPosition, bool& needsWordSpacing);
     virtual void verticallyAlignBoxes(int& heightOfBlock);
 };
 
-}
+} // namespace WebCore
 
 #endif // SVG_SUPPORT
 
-#endif // !SVGInlineFlowBox_H
+#endif // SVGInlineFlowBox_H
index adc1b2f..9085b53 100644 (file)
  *
  */
 
+#include "config.h"
 
 #ifdef SVG_SUPPORT
 
-#include "config.h"
 #include "SVGRootInlineBox.h"
+
 #include "SVGInlineFlowBox.h"
 
 namespace WebCore {
@@ -45,5 +46,6 @@ void SVGRootInlineBox::verticallyAlignBoxes(int& heightOfBlock)
     placeSVGFlowVertically(this, heightOfBlock);
 }
 
-}
+} // namespace WebCore
+
 #endif // SVG_SUPPORT
index 2e841fa..b2fa958 100644 (file)
@@ -32,14 +32,18 @@ namespace WebCore {
     
 class SVGRootInlineBox : public RootInlineBox {
 public:
-    SVGRootInlineBox(RenderObject* obj) : RootInlineBox(obj) {}
-    virtual void paint(RenderObject::PaintInfo& i, int _tx, int _ty);
+    SVGRootInlineBox(RenderObject* obj)
+        : RootInlineBox(obj)
+    {
+    }
+
+    virtual void paint(RenderObject::PaintInfo&, int tx, int ty);
     virtual int placeBoxesHorizontally(int x, int& leftPosition, int& rightPosition, bool& needsWordSpacing);
     virtual void verticallyAlignBoxes(int& heightOfBlock);
 };
 
-}
+} // namespace WebCore
 
 #endif // SVG_SUPPORT
 
-#endif // !RootInlineBox_H
+#endif // SVGRootInlineBox_H
index c8d5ec1..c4137c2 100644 (file)
@@ -27,11 +27,13 @@ namespace WebCore {
 
 class RenderTable;
 
-class TableLayout
-{
+class TableLayout {
 public:
-    TableLayout(RenderTable* t) : m_table(t) {}
-    virtual ~TableLayout() {};
+    TableLayout(RenderTable* table)
+        : m_table(table)
+    {
+    }
+    virtual ~TableLayout() { }
 
     virtual void calcMinMaxWidth() = 0;
     virtual void layout() = 0;
@@ -40,6 +42,6 @@ protected:
     RenderTable* m_table;
 };
 
-}
+} // namespace WebCore
 
-#endif
+#endif // TableLayout_H
index 8ee1637..acb8145 100644 (file)
@@ -34,13 +34,18 @@ namespace WebCore {
     class InlineBox;
 
     struct BidiStatus {
-        BidiStatus() : eor(U_OTHER_NEUTRAL), lastStrong(U_OTHER_NEUTRAL), last(U_OTHER_NEUTRAL) {}
-        
+        BidiStatus()
+            : eor(U_OTHER_NEUTRAL)
+            , lastStrong(U_OTHER_NEUTRAL)
+            , last(U_OTHER_NEUTRAL)
+        {
+        }
+
         UCharDirection eor;
         UCharDirection lastStrong;
         UCharDirection last;
     };
-        
+
     class BidiContext {
     public:
         BidiContext(unsigned char level, UCharDirection embedding, BidiContext* parent = 0, bool override = false);
@@ -65,7 +70,12 @@ namespace WebCore {
 
     struct BidiRun {
         BidiRun(int start_, int stop_, RenderObject* o, BidiContext* context, UCharDirection dir)
-            :  start(start_), stop(stop_), obj(o), box(0), override(context->override), nextRun(0)
+            : start(start_)
+            , stop(stop_)
+            , obj(o)
+            , box(0)
+            , override(context->override)
+            , nextRun(0)
         {
             if (dir == U_OTHER_NEUTRAL) 
                 dir = context->dir();
@@ -100,14 +110,14 @@ namespace WebCore {
         int start;
         int stop;
 
-        RenderObject *obj;
+        RenderObjectobj;
         InlineBox* box;
-        
+
         // explicit + implicit levels here
         unsigned char level;
         bool override : 1;
         bool compact : 1;
-        
+
         BidiRun* nextRun;
     };
 
index 44f6c3e..b5c52f1 100644 (file)
@@ -20,6 +20,9 @@
  *
  */
 
+#ifndef break_lines_h
+#define break_lines_h
+
 #include "UChar.h"
 
 namespace WebCore {
@@ -33,4 +36,6 @@ namespace WebCore {
         return pos == nextBreakable;
     }
 
-}
+} // namespace WebCore
+
+#endif // break_lines_h