Remove redundant helper from RenderRegion.
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 23 Jun 2013 06:24:15 +0000 (06:24 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 23 Jun 2013 06:24:15 +0000 (06:24 +0000)
https://bugs.webkit.org/show_bug.cgi?id=117915

Patch by Zalan Bujtas <zalan@apple.com> on 2013-06-22
Reviewed by Andreas Kling.

Minor code cleanup.

* rendering/RenderRegion.cpp:
(WebCore::RenderRegion::regionOversetState):
(WebCore::RenderRegion::setRegionOversetState):
* rendering/RenderRegion.h:

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

Source/WebCore/ChangeLog
Source/WebCore/rendering/RenderRegion.cpp
Source/WebCore/rendering/RenderRegion.h

index d6c4ba4..af6d68c 100644 (file)
@@ -1,3 +1,17 @@
+2013-06-22  Zalan Bujtas  <zalan@apple.com>
+
+        Remove redundant helper from RenderRegion.
+        https://bugs.webkit.org/show_bug.cgi?id=117915
+
+        Reviewed by Andreas Kling.
+
+        Minor code cleanup.
+
+        * rendering/RenderRegion.cpp:
+        (WebCore::RenderRegion::regionOversetState):
+        (WebCore::RenderRegion::setRegionOversetState):
+        * rendering/RenderRegion.h:
+
 2013-06-22  Mark Rowe  <mrowe@apple.com>
 
         Build fix after r151878.
index d87c3ce..4f86b52 100644 (file)
@@ -133,22 +133,23 @@ LayoutRect RenderRegion::overflowRectForFlowThreadPortion(const LayoutRect& flow
 
 RegionOversetState RenderRegion::regionOversetState() const
 {
-    if (isValid() && element())
-        return element()->regionOversetState();
+    ASSERT(node());
+
+    if (!isValid())
+        return RegionUndefined;
+
+    if (Element* element = toElement(node()))
+        return element->regionOversetState();
     
     return RegionUndefined;
 }
 
 void RenderRegion::setRegionOversetState(RegionOversetState state)
 {
-    if (element())
-        element()->setRegionOversetState(state);
-}
+    ASSERT(node());
 
-Element* RenderRegion::element() const
-{
-    ASSERT(node() && node()->isElementNode());
-    return toElement(node());
+    if (Element* element = toElement(node()))
+        element->setRegionOversetState(state);
 }
 
 LayoutUnit RenderRegion::pageLogicalTopForOffset(LayoutUnit /* offset */) const
index a34cd23..cff7ca5 100644 (file)
@@ -85,8 +85,6 @@ public:
     RegionOversetState regionOversetState() const;
     void setRegionOversetState(RegionOversetState);
 
-    Element* element() const;
-    
     // These methods represent the width and height of a "page" and for a RenderRegion they are just the
     // content width and content height of a region. For RenderRegionSets, however, they will be the width and
     // height of a single column or page in the set.