Use std::unique_ptr to signal ownership transfer in WebCompositorSupport

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

 /*
  * Copyright (C) 2012 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
  * 2.  Redistributions in binary form must reproduce the above copyright
@@ skipping 544 matching lines @@
 
   if (m_surface) {
     // Always save an initial frame, to support resetting the top level matrix
     // and clip.
     m_surface->getCanvas()->save();
   } else {
     reportSurfaceCreationFailure();
   }
 
   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());
     m_layer->setNearestNeighbor(m_filterQuality == kNone_SkFilterQuality);
   }
 
   if (m_surface && isHibernating()) {
     if (surfaceIsAccelerated) {
       m_logger->reportHibernationEvent(HibernationEndedNormally);
     } else {
@@ skipping 554 matching lines @@
     default;
 
 void Canvas2DLayerBridge::Logger::reportHibernationEvent(
     HibernationEvent event) {
   DEFINE_STATIC_LOCAL(EnumerationHistogram, hibernationHistogram,
                       ("Canvas.HibernationEvents", HibernationEventCount));
   hibernationHistogram.count(event);
 }
 
 }  // namespace blink