Use std::unique_ptr to signal ownership transfer in WebCompositorSupport

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

Index: third_party/WebKit/Source/platform/graphics/Canvas2DLayerBridge.cpp
diff --git a/third_party/WebKit/Source/platform/graphics/Canvas2DLayerBridge.cpp b/third_party/WebKit/Source/platform/graphics/Canvas2DLayerBridge.cpp
index 724208bd954bec602346ec3b68b45e85eb157156..6b34991a8eb68a27cb479d77672bbca1d72c10ea 100644
--- a/third_party/WebKit/Source/platform/graphics/Canvas2DLayerBridge.cpp
+++ b/third_party/WebKit/Source/platform/graphics/Canvas2DLayerBridge.cpp
@@ -562,9 +562,9 @@ SkSurface* Canvas2DLayerBridge::getOrCreateSurface(AccelerationHint hint) {
   }
 
   if (m_surface && surfaceIsAccelerated && !m_layer) {
-    m_layer = wrapUnique(
+    m_layer =
         Platform::current()->compositorSupport()->createExternalTextureLayer(
-            this));
+            this);

[jbroman, 2016/10/12 22:18:40]

nit: this wrapping is kinda gross. I guess clang-format insists on it? I'd have
hoped for something like:

m_layer = Platform::current()->compositorSupport()
                             ->createExternalTextureLayer(this);

or something (here and elsewhere). Do you know what's up with this?

[dcheng, 2016/10/12 22:27:03]

I don't know the specifics, but I'm guessing clang-format penalizes breaking
before an arrow / dot operator (a lot?) more heavily than it does breaking after
= and (, even though the latter requires two line breaks.

     m_layer->setOpaque(m_opacityMode == Opaque);
     m_layer->setBlendBackgroundColor(m_opacityMode != Opaque);
     GraphicsLayer::registerContentsLayer(m_layer->layer());