Use SkLayerDrawLooper::Builder to construct SkLayerDrawLooper.

Source/core/rendering/EllipsisBox.cpp

Index: Source/core/rendering/EllipsisBox.cpp
diff --git a/Source/core/rendering/EllipsisBox.cpp b/Source/core/rendering/EllipsisBox.cpp
index cd35c3d2e88526bc17797c4b35d4f48bf62abffe..467d1e6bc4e3ccdba7fd38b0c69ac22fcc52fd8e 100644
--- a/Source/core/rendering/EllipsisBox.cpp
+++ b/Source/core/rendering/EllipsisBox.cpp
@@ -67,17 +67,17 @@ void EllipsisBox::paint(PaintInfo& paintInfo, const LayoutPoint& paintOffset, La
     const ShadowList* shadowList = context->printing() ? 0 : style->textShadow();
     bool hasShadow = shadowList;
     if (hasShadow) {
-        DrawLooper drawLooper;
+        DrawLooper::Builder drawLooperBuilder;
         for (size_t i = shadowList->shadows().size(); i--; ) {
             const ShadowData& shadow = shadowList->shadows()[i];
             float shadowX = isHorizontal() ? shadow.x() : shadow.y();
             float shadowY = isHorizontal() ? shadow.y() : -shadow.x();
             FloatSize offset(shadowX, shadowY);
-            drawLooper.addShadow(offset, shadow.blur(), shadow.color(),
+            drawLooperBuilder.addShadow(offset, shadow.blur(), shadow.color(),
                 DrawLooper::ShadowRespectsTransforms, DrawLooper::ShadowIgnoresAlpha);
         }
-        drawLooper.addUnmodifiedContent();
-        context->setDrawLooper(drawLooper);
+        drawLooperBuilder.addUnmodifiedContent();
+        context->setDrawLooper(*drawLooperBuilder.detachDrawLooper().get());

[jbroman, 2014/02/24 17:47:44]

This seems awkward. If nothing else, you can dereference a PassRefPtr directly,
so the .get() is unnecessary.

I think my preference would be something like:

OwnPtr<DrawLooper> drawLooper = drawLooperBuilder.detachDrawLooper();
context->setDrawLooper(*drawLooper);

     }
 
     TextRun textRun = RenderBlockFlow::constructTextRun(renderer(), font, m_str, style, TextRun::AllowTrailingExpansion);