Use sk_sp-based APIs for SkColorFilter

third_party/WebKit/Source/platform/graphics/GraphicsContextState.cpp

 // Copyright (c) 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "platform/graphics/GraphicsContextState.h"
 
 #include "platform/graphics/skia/SkiaUtils.h"
 
 namespace blink {
 
@@ skipping 139 matching lines @@
     m_fillPaint.setLooper(0);
 }
 
 void GraphicsContextState::setLineDash(const DashArray& dashes, float dashOffset)
 {
     m_strokeData.setLineDash(dashes, dashOffset);
 }
 
 void GraphicsContextState::setColorFilter(PassRefPtr<SkColorFilter> colorFilter)
 {
-    m_colorFilter = colorFilter;
-    m_strokePaint.setColorFilter(m_colorFilter.get());
-    m_fillPaint.setColorFilter(m_colorFilter.get());
+    m_colorFilter = toSkSp(colorFilter);

[f(malita), 2016/03/28 13:39:32]

Hmm, looking at this code I wonder why we need m_colorFilter at all: we set the
CF on fill/stroke paints, and AFAICT they always stay in sync.  Possibly similar
situation for m_drawLoop and others.

Nothing actionable for this CL, just a mental note so I can go back and try to
remove them.

+    m_strokePaint.setColorFilter(m_colorFilter);
+    m_fillPaint.setColorFilter(m_colorFilter);
 }
 
 void GraphicsContextState::setInterpolationQuality(InterpolationQuality quality)
 {
     m_interpolationQuality = quality;
     m_strokePaint.setFilterQuality(filterQualityForPaint(quality));
     m_fillPaint.setFilterQuality(filterQualityForPaint(quality));
 }
 
 void GraphicsContextState::setShouldAntialias(bool shouldAntialias)
 {
     m_shouldAntialias = shouldAntialias;
     m_strokePaint.setAntiAlias(shouldAntialias);
     m_fillPaint.setAntiAlias(shouldAntialias);
 }
 
 } // namespace blink