Chromium Code Reviews| Index: third_party/WebKit/Source/platform/graphics/paint/PropertyTreeState.h |
| diff --git a/third_party/WebKit/Source/platform/graphics/paint/PropertyTreeState.h b/third_party/WebKit/Source/platform/graphics/paint/PropertyTreeState.h |
| index 0204844235482c817b3b3089078fd63d1a110951..6e47c464f62e593a1f6bf44103754c691c8e2b4e 100644 |
| --- a/third_party/WebKit/Source/platform/graphics/paint/PropertyTreeState.h |
| +++ b/third_party/WebKit/Source/platform/graphics/paint/PropertyTreeState.h |
| @@ -38,36 +38,30 @@ class PropertyTreeState { |
| DCHECK(!m_transform || !m_transform->hasOneRef()); |
| return m_transform.get(); |
| } |
| - void setTransform(const TransformPaintPropertyNode* node) { |
| + void setTransform(PassRefPtr<const TransformPaintPropertyNode> node) { |
| m_transform = node; |
| - DCHECK(!node->hasOneRef()); |
|
trchen
2016/12/15 04:15:52
Just realized this DCHECK makes no sense. The only
|
| } |
| const ClipPaintPropertyNode* clip() const { |
| DCHECK(!m_clip || !m_clip->hasOneRef()); |
| return m_clip.get(); |
| } |
| - void setClip(const ClipPaintPropertyNode* node) { |
| - m_clip = node; |
| - DCHECK(!node->hasOneRef()); |
| - } |
| + void setClip(PassRefPtr<const ClipPaintPropertyNode> node) { m_clip = node; } |
| const EffectPaintPropertyNode* effect() const { |
| DCHECK(!m_effect || !m_effect->hasOneRef()); |
| return m_effect.get(); |
| } |
| - void setEffect(const EffectPaintPropertyNode* node) { |
| + void setEffect(PassRefPtr<const EffectPaintPropertyNode> node) { |
| m_effect = node; |
| - DCHECK(!node->hasOneRef()); |
| } |
| const ScrollPaintPropertyNode* scroll() const { |
| DCHECK(!m_scroll || !m_scroll->hasOneRef()); |
| return m_scroll.get(); |
| } |
| - void setScroll(const ScrollPaintPropertyNode* node) { |
| + void setScroll(PassRefPtr<const ScrollPaintPropertyNode> node) { |
| m_scroll = node; |
| - DCHECK(!node->hasOneRef()); |
| } |
| private: |