Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1539)

Unified Diff: Source/core/rendering/RenderLayerReflectionInfo.cpp

Issue 423093002: Oilpan: Prepare to move RenderObject and RenderObjectChildList to Oilpan heap. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: dispose -> destroy Created 6 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: Source/core/rendering/RenderLayerReflectionInfo.cpp
diff --git a/Source/core/rendering/RenderLayerReflectionInfo.cpp b/Source/core/rendering/RenderLayerReflectionInfo.cpp
index 0032b482ad88e730ec4d4838737d97e909c54ce6..a14ac1c76b4217ef5534e1ad38d618ddfe85b757 100644
--- a/Source/core/rendering/RenderLayerReflectionInfo.cpp
+++ b/Source/core/rendering/RenderLayerReflectionInfo.cpp
@@ -59,22 +59,27 @@ RenderLayerReflectionInfo::RenderLayerReflectionInfo(RenderBox& renderer)
: m_box(renderer)
, m_isPaintingInsideReflection(false)
{
- UseCounter::count(m_box.document(), UseCounter::Reflection);
+ UseCounter::count(box().document(), UseCounter::Reflection);
- m_reflection = RenderReplica::createAnonymous(&(m_box.document()));
- m_reflection->setParent(&m_box); // We create a 1-way connection.
+ m_reflection = RenderReplica::createAnonymous(&box().document());
+ m_reflection->setParent(m_box); // We create a 1-way connection.
}
-RenderLayerReflectionInfo::~RenderLayerReflectionInfo()
+void RenderLayerReflectionInfo::destroy()
{
if (!m_reflection->documentBeingDestroyed())
- m_reflection->removeLayers(m_box.layer());
+ m_reflection->removeLayers(box().layer());
m_reflection->setParent(0);
m_reflection->destroy();
- m_reflection = 0;
+ m_reflection = nullptr;
}
+void RenderLayerReflectionInfo::trace(Visitor* visitor)
+{
+ visitor->trace(m_box);
+ visitor->trace(m_reflection);
+}
RenderLayer* RenderLayerReflectionInfo::reflectionLayer() const
{
@@ -84,16 +89,16 @@ RenderLayer* RenderLayerReflectionInfo::reflectionLayer() const
void RenderLayerReflectionInfo::updateAfterStyleChange(const RenderStyle* oldStyle)
{
RefPtr<RenderStyle> newStyle = RenderStyle::create();
- newStyle->inheritFrom(m_box.style());
+ newStyle->inheritFrom(box().style());
// Map in our transform.
TransformOperations transform;
- switch (m_box.style()->boxReflect()->direction()) {
+ switch (box().style()->boxReflect()->direction()) {
case ReflectionBelow:
transform.operations().append(TranslateTransformOperation::create(Length(0, Fixed),
Length(100., Percent), TransformOperation::Translate));
transform.operations().append(TranslateTransformOperation::create(Length(0, Fixed),
- m_box.style()->boxReflect()->offset(), TransformOperation::Translate));
+ box().style()->boxReflect()->offset(), TransformOperation::Translate));
transform.operations().append(ScaleTransformOperation::create(1.0, -1.0, ScaleTransformOperation::Scale));
break;
@@ -102,14 +107,14 @@ void RenderLayerReflectionInfo::updateAfterStyleChange(const RenderStyle* oldSty
transform.operations().append(TranslateTransformOperation::create(Length(0, Fixed),
Length(100., Percent), TransformOperation::Translate));
transform.operations().append(TranslateTransformOperation::create(Length(0, Fixed),
- m_box.style()->boxReflect()->offset(), TransformOperation::Translate));
+ box().style()->boxReflect()->offset(), TransformOperation::Translate));
break;
case ReflectionRight:
transform.operations().append(TranslateTransformOperation::create(Length(100., Percent),
Length(0, Fixed), TransformOperation::Translate));
transform.operations().append(TranslateTransformOperation::create(
- m_box.style()->boxReflect()->offset(), Length(0, Fixed), TransformOperation::Translate));
+ box().style()->boxReflect()->offset(), Length(0, Fixed), TransformOperation::Translate));
transform.operations().append(ScaleTransformOperation::create(-1.0, 1.0, ScaleTransformOperation::Scale));
break;
@@ -118,13 +123,13 @@ void RenderLayerReflectionInfo::updateAfterStyleChange(const RenderStyle* oldSty
transform.operations().append(TranslateTransformOperation::create(Length(100., Percent),
Length(0, Fixed), TransformOperation::Translate));
transform.operations().append(TranslateTransformOperation::create(
- m_box.style()->boxReflect()->offset(), Length(0, Fixed), TransformOperation::Translate));
+ box().style()->boxReflect()->offset(), Length(0, Fixed), TransformOperation::Translate));
break;
}
newStyle->setTransform(transform);
// Map in our mask.
- newStyle->setMaskBoxImage(m_box.style()->boxReflect()->mask());
+ newStyle->setMaskBoxImage(box().style()->boxReflect()->mask());
m_reflection->setStyle(newStyle.release());
}
@@ -142,7 +147,7 @@ void RenderLayerReflectionInfo::paint(GraphicsContext* context, const LayerPaint
String RenderLayerReflectionInfo::debugName() const
{
- return m_box.debugName() + " (reflection)";
+ return box().debugName() + " (reflection)";
}
} // namespace blink
« no previous file with comments | « Source/core/rendering/RenderLayerReflectionInfo.h ('k') | Source/core/rendering/RenderLayerScrollableArea.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698