Title: [159010] trunk/Source/WebCore
Revision
159010
Author
[email protected]
Date
2013-11-09 21:47:40 -0800 (Sat, 09 Nov 2013)

Log Message

Use RENDER_OBJECT_TYPE_CASTS for more types.
<https://webkit.org/b/124112>

Generate toRenderFoo() type casting helpers for these classes:

    - RenderCombineText
    - RenderDetailsMarker
    - RenderListMarker
    - RenderVideo
    - RenderView

Reviewed by Anders Carlsson.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (159009 => 159010)


--- trunk/Source/WebCore/ChangeLog	2013-11-10 05:08:43 UTC (rev 159009)
+++ trunk/Source/WebCore/ChangeLog	2013-11-10 05:47:40 UTC (rev 159010)
@@ -1,5 +1,20 @@
 2013-11-09  Andreas Kling  <[email protected]>
 
+        Use RENDER_OBJECT_TYPE_CASTS for more types.
+        <https://webkit.org/b/124112>
+
+        Generate toRenderFoo() type casting helpers for these classes:
+
+            - RenderCombineText
+            - RenderDetailsMarker
+            - RenderListMarker
+            - RenderVideo
+            - RenderView
+
+        Reviewed by Anders Carlsson.
+
+2013-11-09  Andreas Kling  <[email protected]>
+
         Move MathML type checking virtuals to RenderObject.
         <https://webkit.org/b/124111>
 

Modified: trunk/Source/WebCore/rendering/RenderCombineText.h (159009 => 159010)


--- trunk/Source/WebCore/rendering/RenderCombineText.h	2013-11-10 05:08:43 UTC (rev 159009)
+++ trunk/Source/WebCore/rendering/RenderCombineText.h	2013-11-10 05:47:40 UTC (rev 159010)
@@ -57,20 +57,8 @@
     bool m_needsFontUpdate : 1;
 };
 
-inline RenderCombineText& toRenderCombineText(RenderObject& object)
-{ 
-    ASSERT_WITH_SECURITY_IMPLICATION(object.isCombineText());
-    return static_cast<RenderCombineText&>(object);
-}
+RENDER_OBJECT_TYPE_CASTS(RenderCombineText, isCombineText());
 
-inline const RenderCombineText& toRenderCombineText(const RenderObject& object)
-{ 
-    ASSERT_WITH_SECURITY_IMPLICATION(object.isCombineText());
-    return static_cast<const RenderCombineText&>(object);
-}
-
-void toRenderCombineText(const RenderCombineText&);
-
 } // namespace WebCore
 
 #endif // RenderCombineText_h

Modified: trunk/Source/WebCore/rendering/RenderDetailsMarker.h (159009 => 159010)


--- trunk/Source/WebCore/rendering/RenderDetailsMarker.h	2013-11-10 05:08:43 UTC (rev 159009)
+++ trunk/Source/WebCore/rendering/RenderDetailsMarker.h	2013-11-10 05:47:40 UTC (rev 159010)
@@ -45,14 +45,8 @@
     Path getPath(const LayoutPoint& origin) const;
 };
 
-inline const RenderDetailsMarker& toRenderDetailsMarker(const RenderObject& object)
-{
-    ASSERT_WITH_SECURITY_IMPLICATION(object.isDetailsMarker());
-    return static_cast<const RenderDetailsMarker&>(object);
-}
+RENDER_OBJECT_TYPE_CASTS(RenderDetailsMarker, isDetailsMarker());
 
-void toRenderDetailsMarker(const RenderDetailsMarker&);
-
 }
 
 #endif

Modified: trunk/Source/WebCore/rendering/RenderListMarker.h (159009 => 159010)


--- trunk/Source/WebCore/rendering/RenderListMarker.h	2013-11-10 05:08:43 UTC (rev 159009)
+++ trunk/Source/WebCore/rendering/RenderListMarker.h	2013-11-10 05:47:40 UTC (rev 159010)
@@ -85,20 +85,8 @@
     RenderListItem& m_listItem;
 };
 
-inline RenderListMarker& toRenderListMarker(RenderObject& object)
-{
-    ASSERT_WITH_SECURITY_IMPLICATION(object.isListMarker());
-    return static_cast<RenderListMarker&>(object);
-}
+RENDER_OBJECT_TYPE_CASTS(RenderListMarker, isListMarker());
 
-inline const RenderListMarker& toRenderListMarker(const RenderObject& object)
-{
-    ASSERT_WITH_SECURITY_IMPLICATION(object.isListMarker());
-    return static_cast<const RenderListMarker&>(object);
-}
-
-void toRenderListMarker(const RenderListMarker&);
-
 } // namespace WebCore
 
 #endif // RenderListMarker_h

Modified: trunk/Source/WebCore/rendering/RenderVideo.h (159009 => 159010)


--- trunk/Source/WebCore/rendering/RenderVideo.h	2013-11-10 05:08:43 UTC (rev 159009)
+++ trunk/Source/WebCore/rendering/RenderVideo.h	2013-11-10 05:47:40 UTC (rev 159010)
@@ -92,20 +92,8 @@
     LayoutSize m_cachedImageSize;
 };
 
-inline RenderVideo& toRenderVideo(RenderObject& object)
-{
-    ASSERT_WITH_SECURITY_IMPLICATION(object.isVideo());
-    return static_cast<RenderVideo&>(object);
-}
+RENDER_OBJECT_TYPE_CASTS(RenderVideo, isVideo());
 
-inline const RenderVideo& toRenderVideo(const RenderObject& object)
-{
-    ASSERT_WITH_SECURITY_IMPLICATION(object.isVideo());
-    return static_cast<const RenderVideo&>(object);
-}
-
-void toRenderVideo(const RenderVideo&);
-
 } // namespace WebCore
 
 #endif

Modified: trunk/Source/WebCore/rendering/RenderView.h (159009 => 159010)


--- trunk/Source/WebCore/rendering/RenderView.h	2013-11-10 05:08:43 UTC (rev 159009)
+++ trunk/Source/WebCore/rendering/RenderView.h	2013-11-10 05:47:40 UTC (rev 159010)
@@ -348,20 +348,8 @@
 #endif
 };
 
-inline RenderView& toRenderView(RenderObject& object)
-{
-    ASSERT_WITH_SECURITY_IMPLICATION(object.isRenderView());
-    return static_cast<RenderView&>(object);
-}
+RENDER_OBJECT_TYPE_CASTS(RenderView, isRenderView());
 
-inline const RenderView& toRenderView(const RenderObject& object)
-{
-    ASSERT_WITH_SECURITY_IMPLICATION(object.isRenderView());
-    return static_cast<const RenderView&>(object);
-}
-
-void toRenderView(const RenderView&);
-
 // Stack-based class to assist with LayoutState push/pop
 class LayoutStateMaintainer {
     WTF_MAKE_NONCOPYABLE(LayoutStateMaintainer);
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to