- improved version of last night's build fix (should not do using namespace...
authordarin <darin@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 9 Nov 2006 15:41:25 +0000 (15:41 +0000)
committerdarin <darin@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 9 Nov 2006 15:41:25 +0000 (15:41 +0000)
        * rendering/RenderObject.h:
        * rendering/RenderObject.cpp:
        (WebCore::RenderObject::maxTopMargin):
        (WebCore::RenderObject::maxBottomMargin):

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

WebCore/ChangeLog
WebCore/rendering/RenderObject.cpp
WebCore/rendering/RenderObject.h

index a6c973c222aa7a1b490f23028ef4e44c8cb951f0..70e67342abbd5d8515965f7c0d59b8ccc90f6217 100644 (file)
@@ -1,3 +1,12 @@
+2006-11-09  Darin Adler  <darin@apple.com>
+
+        - improved version of last night's build fix (should not do using namespace in a header)
+
+        * rendering/RenderObject.h:
+        * rendering/RenderObject.cpp:
+        (WebCore::RenderObject::maxTopMargin):
+        (WebCore::RenderObject::maxBottomMargin):
+
 2006-11-08  Maciej Stachowiak  <mjs@apple.com>
 
         Reviewed by Geoff.
index 06c40ab56d2591d6ebd3073b400dde97da6c0ca4..dd1352b001a1acda16575740574518e91197ead7 100644 (file)
@@ -3086,14 +3086,12 @@ unsigned RenderObject::caretMaxRenderedOffset() const
 
 int RenderObject::previousOffset(int current) const
 {
-    int previousOffset = current - 1;
-    return previousOffset;
+    return current - 1;
 }
 
 int RenderObject::nextOffset(int current) const
 {
-    int nextOffset = current + 1;
-    return nextOffset;
+    return current + 1;
 }
 
 InlineBox* RenderObject::inlineBox(int offset, EAffinity affinity)
@@ -3101,6 +3099,16 @@ InlineBox* RenderObject::inlineBox(int offset, EAffinity affinity)
     return inlineBoxWrapper();
 }
 
+int RenderObject::maxTopMargin(bool positive) const
+{
+    return positive ? max(0, marginTop()) : min(0, -marginTop());
+}
+
+int RenderObject::maxBottomMargin(bool positive) const
+{
+    return positive ? max(0, marginBottom()) : min(0, -marginBottom());
+}
+
 #ifdef SVG_SUPPORT
 
 FloatRect RenderObject::relativeBBox(bool) const
index 76ff04a5737a3c7416c703e265b9abf4ce3a2d53..2f2aa9f60afa918590282ddd6b86d774a91c2e03 100644 (file)
 #define RenderObject_h
 
 #include "CachedResourceClient.h"
+#if 0
 #include "DeprecatedValueList.h"
 #include "RenderStyle.h"
 #include "ScrollBar.h"
 #include "VisiblePosition.h"
 #include <algorithm>
 #include <wtf/HashMap.h>
-
-using namespace std;
-
-/*
- *  The painting of a layer occurs in three distinct phases.  Each phase involves
- *  a recursive descent into the layer's render objects. The first phase is the background phase.
- *  The backgrounds and borders of all blocks are painted.  Inlines are not painted at all.
- *  Floats must paint above block backgrounds but entirely below inline content that can overlap them.
- *  In the foreground phase, all inlines are fully painted.  Inline replaced elements will get all
- *  three phases invoked on them during this phase.
- */
+#endif
+#include "PlatformString.h"
 
 namespace WebCore {
 
@@ -76,6 +68,15 @@ class TextStream;
 class VisiblePosition;
 struct HitTestRequest;
 
+/*
+ *  The painting of a layer occurs in three distinct phases.  Each phase involves
+ *  a recursive descent into the layer's render objects. The first phase is the background phase.
+ *  The backgrounds and borders of all blocks are painted.  Inlines are not painted at all.
+ *  Floats must paint above block backgrounds but entirely below inline content that can overlap them.
+ *  In the foreground phase, all inlines are fully painted.  Inline replaced elements will get all
+ *  three phases invoked on them during this phase.
+ */
+
 enum PaintPhase {
     PaintPhaseBlockBackground,
     PaintPhaseChildBlockBackground,
@@ -656,8 +657,8 @@ public:
     virtual bool isTopMarginQuirk() const { return false; }
     virtual bool isBottomMarginQuirk() const { return false; }
 
-    virtual int maxTopMargin(bool positive) const { return positive ? max(0, marginTop()) : min(0, -marginTop()); }
-    virtual int maxBottomMargin(bool positive) const { return positive ? max(0, marginBottom()) : min(0, -marginBottom()); }
+    virtual int maxTopMargin(bool positive) const;
+    virtual int maxBottomMargin(bool positive) const;
 
     virtual int marginTop() const { return 0; }
     virtual int marginBottom() const { return 0; }