RenderObject: Inline isBody() and isHR().

<https://webkit.org/b/124901>

Together these account for ~0.3% of samples on HTML5-8266.
Almost all of it is call overhead.

Reviewed by Anders Carlsson.

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

diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
index 508689d331ba8ff4c873bfb3cb446d4fd8a6afc7..a80f8a447fc83b0602cbe9ad78c620c54e61f543 100644 (file)
--- a/Source/WebCore/ChangeLog
+++ b/Source/WebCore/ChangeLog
@@ -1,3 +1,13 @@
+2013-11-26  Andreas Kling  <akling@apple.com>
+
+        RenderObject: Inline isBody() and isHR().
+        <https://webkit.org/b/124901>
+
+        Together these account for ~0.3% of samples on HTML5-8266.
+        Almost all of it is call overhead.
+
+        Reviewed by Anders Carlsson.
+
 2013-11-26  Bear Travis  <betravis@adobe.com>
 
         [CSS Shapes] Layout using [<box> || <shape>] value

diff --git a/Source/WebCore/rendering/RenderObject.cpp b/Source/WebCore/rendering/RenderObject.cpp
index e9597609b19fdadf97064b72353058b85e594d78..49c9f7b8d1491f36a7e06cd242352b827c6d6fec 100644 (file)
--- a/Source/WebCore/rendering/RenderObject.cpp
+++ b/Source/WebCore/rendering/RenderObject.cpp
@@ -142,16 +142,6 @@ bool RenderObject::isDescendantOf(const RenderObject* obj) const
     return false;
 }
 
-bool RenderObject::isBody() const
-{
-    return node() && node()->hasTagName(bodyTag);
-}
-
-bool RenderObject::isHR() const
-{
-    return node() && node()->hasTagName(hrTag);
-}
-
 bool RenderObject::isLegend() const
 {
     return node() && node()->hasTagName(legendTag);

diff --git a/Source/WebCore/rendering/RenderObject.h b/Source/WebCore/rendering/RenderObject.h
index 71a1e65d5dee201247a8565240c11a63902ea9ed..9bf5cfb884d170464ecef7bf293521bff04f5062 100644 (file)
--- a/Source/WebCore/rendering/RenderObject.h
+++ b/Source/WebCore/rendering/RenderObject.h
@@ -368,8 +368,8 @@ public:
     virtual bool isRenderScrollbarPart() const { return false; }
 
     bool isRoot() const { return document().documentElement() == &m_node; }
-    bool isBody() const;
-    bool isHR() const;
+    bool isBody() const { return node() && node()->hasTagName(HTMLNames::bodyTag); }
+    bool isHR() const { return node() && node()->hasTagName(HTMLNames::hrTag); }
     bool isLegend() const;
 
     bool isHTMLMarquee() const;