Diff
Modified: trunk/Source/WebCore/ChangeLog (156120 => 156121)
--- trunk/Source/WebCore/ChangeLog 2013-09-19 22:10:26 UTC (rev 156120)
+++ trunk/Source/WebCore/ChangeLog 2013-09-19 22:12:26 UTC (rev 156121)
@@ -1,3 +1,34 @@
+2013-09-19 Commit Queue <[email protected]>
+
+ Unreviewed, rolling out r156119.
+ http://trac.webkit.org/changeset/156119
+ https://bugs.webkit.org/show_bug.cgi?id=121640
+
+ broke builds in various ways (Requested by anttik on #webkit).
+
+ * accessibility/AccessibilitySpinButton.cpp:
+ * bindings/scripts/CodeGeneratorObjC.pm:
+ (GenerateImplementation):
+ * css/SelectorChecker.cpp:
+ * dom/ContainerNode.h:
+ * dom/Element.h:
+ * dom/PseudoElement.cpp:
+ (WebCore::PseudoElement::didAttachRenderers):
+ * editing/SplitTextNodeContainingElementCommand.cpp:
+ (WebCore::SplitTextNodeContainingElementCommand::doApply):
+ * editing/TypingCommand.cpp:
+ * html/ImageDocument.cpp:
+ * page/PageSerializer.cpp:
+ * rendering/RenderElement.h:
+ * rendering/RenderObject.h:
+ * svg/SVGAnimatedColor.cpp:
+ (WebCore::adjustForCurrentColor):
+ * svg/SVGLocatable.cpp:
+ * svg/graphics/filters/SVGFEImage.cpp:
+ (WebCore::FEImage::determineAbsolutePaintRect):
+ (WebCore::FEImage::referencedRenderer):
+ * svg/graphics/filters/SVGFEImage.h:
+
2013-09-19 Antti Koivisto <[email protected]>
Add covariant RenderElement* Element::renderer()
Modified: trunk/Source/WebCore/accessibility/AccessibilitySpinButton.cpp (156120 => 156121)
--- trunk/Source/WebCore/accessibility/AccessibilitySpinButton.cpp 2013-09-19 22:10:26 UTC (rev 156120)
+++ trunk/Source/WebCore/accessibility/AccessibilitySpinButton.cpp 2013-09-19 22:12:26 UTC (rev 156121)
@@ -27,7 +27,7 @@
#include "AccessibilitySpinButton.h"
#include "AXObjectCache.h"
-#include "RenderElement.h"
+#include "RenderObject.h"
namespace WebCore {
Modified: trunk/Source/WebCore/bindings/scripts/CodeGeneratorObjC.pm (156120 => 156121)
--- trunk/Source/WebCore/bindings/scripts/CodeGeneratorObjC.pm 2013-09-19 22:10:26 UTC (rev 156120)
+++ trunk/Source/WebCore/bindings/scripts/CodeGeneratorObjC.pm 2013-09-19 22:12:26 UTC (rev 156121)
@@ -1289,7 +1289,7 @@
unless ($interfaceName eq "HTMLImageElement") {
push(@customGetterContent, " if (!IMPL->renderer() || !IMPL->renderer()->isImage())\n");
push(@customGetterContent, " return nil;\n");
- $implIncludes{"RenderElement.h"} = 1;
+ $implIncludes{"RenderObject.h"} = 1;
}
}
$implIncludes{"DOMPrivate.h"} = 1;
Modified: trunk/Source/WebCore/css/SelectorChecker.cpp (156120 => 156121)
--- trunk/Source/WebCore/css/SelectorChecker.cpp 2013-09-19 22:10:26 UTC (rev 156120)
+++ trunk/Source/WebCore/css/SelectorChecker.cpp 2013-09-19 22:12:26 UTC (rev 156121)
@@ -48,7 +48,7 @@
#include "InspectorInstrumentation.h"
#include "NodeRenderStyle.h"
#include "Page.h"
-#include "RenderElement.h"
+#include "RenderObject.h"
#include "RenderScrollbar.h"
#include "RenderStyle.h"
#include "ScrollableArea.h"
Modified: trunk/Source/WebCore/dom/ContainerNode.h (156120 => 156121)
--- trunk/Source/WebCore/dom/ContainerNode.h 2013-09-19 22:10:26 UTC (rev 156120)
+++ trunk/Source/WebCore/dom/ContainerNode.h 2013-09-19 22:12:26 UTC (rev 156121)
@@ -33,7 +33,7 @@
namespace WebCore {
class FloatPoint;
-
+
typedef void (*NodeCallback)(Node*, unsigned);
namespace Private {
Modified: trunk/Source/WebCore/dom/Element.h (156120 => 156121)
--- trunk/Source/WebCore/dom/Element.h 2013-09-19 22:10:26 UTC (rev 156120)
+++ trunk/Source/WebCore/dom/Element.h 2013-09-19 22:12:26 UTC (rev 156121)
@@ -44,7 +44,6 @@
class IntSize;
class Locale;
class PseudoElement;
-class RenderElement;
class RenderRegion;
class ShadowRoot;
@@ -128,8 +127,6 @@
DEFINE_ATTRIBUTE_EVENT_LISTENER(webkitfullscreenerror);
#endif
- RenderElement* renderer() const;
-
bool hasAttribute(const QualifiedName&) const;
const AtomicString& getAttribute(const QualifiedName&) const;
void setAttribute(const QualifiedName&, const AtomicString& value);
Modified: trunk/Source/WebCore/dom/PseudoElement.cpp (156120 => 156121)
--- trunk/Source/WebCore/dom/PseudoElement.cpp 2013-09-19 22:10:26 UTC (rev 156120)
+++ trunk/Source/WebCore/dom/PseudoElement.cpp 2013-09-19 22:12:26 UTC (rev 156121)
@@ -29,7 +29,7 @@
#include "ContentData.h"
#include "InspectorInstrumentation.h"
-#include "RenderElement.h"
+#include "RenderObject.h"
#include "RenderQuote.h"
namespace WebCore {
@@ -78,7 +78,7 @@
void PseudoElement::didAttachRenderers()
{
- RenderElement* renderer = this->renderer();
+ RenderObject* renderer = this->renderer();
if (!renderer || !renderer->style()->regionThread().isEmpty())
return;
Modified: trunk/Source/WebCore/editing/SplitTextNodeContainingElementCommand.cpp (156120 => 156121)
--- trunk/Source/WebCore/editing/SplitTextNodeContainingElementCommand.cpp 2013-09-19 22:10:26 UTC (rev 156120)
+++ trunk/Source/WebCore/editing/SplitTextNodeContainingElementCommand.cpp 2013-09-19 22:12:26 UTC (rev 156121)
@@ -27,8 +27,8 @@
#include "SplitTextNodeContainingElementCommand.h"
#include "Element.h"
-#include "RenderElement.h"
#include "Text.h"
+#include "RenderObject.h"
#include <wtf/Assertions.h>
namespace WebCore {
@@ -53,7 +53,7 @@
if (!parent || !parent->parentElement() || !parent->parentElement()->rendererIsEditable())
return;
- RenderElement* parentRenderer = parent->renderer();
+ RenderObject* parentRenderer = parent->renderer();
if (!parentRenderer || !parentRenderer->isInline()) {
wrapContentsInDummySpan(parent);
Node* firstChild = parent->firstChild();
Modified: trunk/Source/WebCore/editing/TypingCommand.cpp (156120 => 156121)
--- trunk/Source/WebCore/editing/TypingCommand.cpp 2013-09-19 22:10:26 UTC (rev 156120)
+++ trunk/Source/WebCore/editing/TypingCommand.cpp 2013-09-19 22:12:26 UTC (rev 156121)
@@ -37,7 +37,7 @@
#include "InsertLineBreakCommand.h"
#include "InsertParagraphSeparatorCommand.h"
#include "InsertTextCommand.h"
-#include "RenderElement.h"
+#include "RenderObject.h"
#include "TextIterator.h"
#include "VisiblePosition.h"
#include "VisibleUnits.h"
Modified: trunk/Source/WebCore/html/ImageDocument.cpp (156120 => 156121)
--- trunk/Source/WebCore/html/ImageDocument.cpp 2013-09-19 22:10:26 UTC (rev 156120)
+++ trunk/Source/WebCore/html/ImageDocument.cpp 2013-09-19 22:12:26 UTC (rev 156121)
@@ -42,7 +42,6 @@
#include "NotImplemented.h"
#include "Page.h"
#include "RawDataDocumentParser.h"
-#include "RenderElement.h"
#include "ResourceBuffer.h"
#include "Settings.h"
Modified: trunk/Source/WebCore/page/PageSerializer.cpp (156120 => 156121)
--- trunk/Source/WebCore/page/PageSerializer.cpp 2013-09-19 22:10:26 UTC (rev 156120)
+++ trunk/Source/WebCore/page/PageSerializer.cpp 2013-09-19 22:12:26 UTC (rev 156121)
@@ -50,7 +50,6 @@
#include "MIMETypeRegistry.h"
#include "MarkupAccumulator.h"
#include "Page.h"
-#include "RenderElement.h"
#include "StyleCachedImage.h"
#include "StyleImage.h"
#include "StylePropertySet.h"
Modified: trunk/Source/WebCore/rendering/RenderElement.h (156120 => 156121)
--- trunk/Source/WebCore/rendering/RenderElement.h 2013-09-19 22:10:26 UTC (rev 156120)
+++ trunk/Source/WebCore/rendering/RenderElement.h 2013-09-19 22:12:26 UTC (rev 156121)
@@ -47,39 +47,6 @@
void generatingNode() const WTF_DELETED_FUNCTION;
};
-inline RenderElement& toRenderElement(RenderObject& object)
-{
- ASSERT_WITH_SECURITY_IMPLICATION(object.isRenderElement());
- return static_cast<RenderElement&>(object);
-}
-
-inline const RenderElement& toRenderElement(const RenderObject& object)
-{
- ASSERT_WITH_SECURITY_IMPLICATION(object.isRenderElement());
- return static_cast<const RenderElement&>(object);
-}
-
-inline RenderElement* toRenderElement(RenderObject* object)
-{
- ASSERT_WITH_SECURITY_IMPLICATION(!object || object->isRenderElement());
- return static_cast<RenderElement*>(object);
-}
-
-inline const RenderElement* toRenderElement(const RenderObject* object)
-{
- ASSERT_WITH_SECURITY_IMPLICATION(!object || object->isRenderElement());
- return static_cast<const RenderElement*>(object);
-}
-
-// This will catch anyone doing an unnecessary cast.
-void toRenderElement(const RenderElement*);
-void toRenderElement(const RenderElement&);
-
-inline RenderElement* Element::renderer() const
-{
- return toRenderElement(Node::renderer());
-}
-
} // namespace WebCore
#endif // RenderElement_h
Modified: trunk/Source/WebCore/rendering/RenderObject.h (156120 => 156121)
--- trunk/Source/WebCore/rendering/RenderObject.h 2013-09-19 22:10:26 UTC (rev 156120)
+++ trunk/Source/WebCore/rendering/RenderObject.h 2013-09-19 22:12:26 UTC (rev 156121)
@@ -342,7 +342,6 @@
bool isPseudoElement() const { return node() && node()->isPseudoElement(); }
- bool isRenderElement() const { return !isText(); }
virtual bool isBoxModelObject() const { return false; }
virtual bool isCounter() const { return false; }
virtual bool isQuote() const { return false; }
Modified: trunk/Source/WebCore/svg/SVGAnimatedColor.cpp (156120 => 156121)
--- trunk/Source/WebCore/svg/SVGAnimatedColor.cpp 2013-09-19 22:10:26 UTC (rev 156120)
+++ trunk/Source/WebCore/svg/SVGAnimatedColor.cpp 2013-09-19 22:12:26 UTC (rev 156121)
@@ -23,7 +23,7 @@
#include "SVGAnimatedColor.h"
#include "ColorDistance.h"
-#include "RenderElement.h"
+#include "RenderObject.h"
#include "SVGAnimateElement.h"
#include "SVGColor.h"
@@ -52,7 +52,7 @@
{
ASSERT(targetElement);
- if (RenderElement* targetRenderer = targetElement->renderer())
+ if (RenderObject* targetRenderer = targetElement->renderer())
color = targetRenderer->style()->visitedDependentColor(CSSPropertyColor);
else
color = Color();
Modified: trunk/Source/WebCore/svg/SVGLocatable.cpp (156120 => 156121)
--- trunk/Source/WebCore/svg/SVGLocatable.cpp 2013-09-19 22:10:26 UTC (rev 156120)
+++ trunk/Source/WebCore/svg/SVGLocatable.cpp 2013-09-19 22:12:26 UTC (rev 156121)
@@ -25,7 +25,7 @@
#if ENABLE(SVG)
#include "SVGLocatable.h"
-#include "RenderElement.h"
+#include "RenderObject.h"
#include "SVGException.h"
#include "SVGGraphicsElement.h"
#include "SVGImageElement.h"
Modified: trunk/Source/WebCore/svg/graphics/filters/SVGFEImage.cpp (156120 => 156121)
--- trunk/Source/WebCore/svg/graphics/filters/SVGFEImage.cpp 2013-09-19 22:10:26 UTC (rev 156120)
+++ trunk/Source/WebCore/svg/graphics/filters/SVGFEImage.cpp 2013-09-19 22:12:26 UTC (rev 156121)
@@ -28,7 +28,7 @@
#include "AffineTransform.h"
#include "Filter.h"
#include "GraphicsContext.h"
-#include "RenderElement.h"
+#include "RenderObject.h"
#include "RenderTreeAsText.h"
#include "SVGElement.h"
#include "SVGFilter.h"
@@ -74,7 +74,7 @@
if (m_image) {
srcRect.setSize(m_image->size());
m_preserveAspectRatio.transformRect(paintRect, srcRect);
- } else if (RenderElement* renderer = referencedRenderer())
+ } else if (RenderObject* renderer = referencedRenderer())
srcRect = svgFilter->absoluteTransform().mapRect(renderer->repaintRectInLocalCoordinates());
if (clipsToBounds())
@@ -84,7 +84,7 @@
setAbsolutePaintRect(enclosingIntRect(paintRect));
}
-RenderElement* FEImage::referencedRenderer() const
+RenderObject* FEImage::referencedRenderer() const
{
if (!m_document)
return 0;
Modified: trunk/Source/WebCore/svg/graphics/filters/SVGFEImage.h (156120 => 156121)
--- trunk/Source/WebCore/svg/graphics/filters/SVGFEImage.h 2013-09-19 22:10:26 UTC (rev 156120)
+++ trunk/Source/WebCore/svg/graphics/filters/SVGFEImage.h 2013-09-19 22:12:26 UTC (rev 156121)
@@ -31,7 +31,7 @@
class Document;
class Image;
-class RenderElement;
+class RenderObject;
class FEImage : public FilterEffect {
public:
@@ -54,7 +54,7 @@
virtual ~FEImage() { }
FEImage(Filter*, PassRefPtr<Image>, const SVGPreserveAspectRatio&);
FEImage(Filter*, Document*, const String&, const SVGPreserveAspectRatio&);
- RenderElement* referencedRenderer() const;
+ RenderObject* referencedRenderer() const;
RefPtr<Image> m_image;
