Change ui::wm::Shadow to use cc::NinePatchLayer.

ui/compositor/layer.cc

 // Copyright (c) 2012 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 "ui/compositor/layer.h"
 
 #include <algorithm>
 
 #include "base/command_line.h"
 #include "base/debug/trace_event.h"
 #include "base/json/json_writer.h"
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
 #include "cc/base/scoped_ptr_algorithm.h"
 #include "cc/layers/content_layer.h"
 #include "cc/layers/delegated_renderer_layer.h"
+#include "cc/layers/nine_patch_layer.h"
 #include "cc/layers/picture_layer.h"
 #include "cc/layers/solid_color_layer.h"
 #include "cc/layers/texture_layer.h"
 #include "cc/output/copy_output_request.h"
 #include "cc/output/delegated_frame_data.h"
 #include "cc/output/filter_operation.h"
 #include "cc/output/filter_operations.h"
 #include "cc/resources/transferable_resource.h"
 #include "ui/compositor/compositor_switches.h"
 #include "ui/compositor/dip_util.h"
@@ skipping 549 matching lines @@
   content_layer_ = new_layer;
 
   mailbox_ = cc::TextureMailbox();
   if (mailbox_release_callback_) {
     mailbox_release_callback_->Run(0, false);
     mailbox_release_callback_.reset();
   }
   RecomputeDrawsContentAndUVRect();
 }
 
+void Layer::SetShowNinePatch() {
+  if (nine_patch_layer_.get())
+    return;
+
+  scoped_refptr<cc::NinePatchLayer> new_layer = cc::NinePatchLayer::Create();
+  SetFillsBoundsOpaquely(false);

[sky, 2014/07/14 15:30:06]

Why the SetFillsBoundsOpaquely here? Isn't that separate from whether this layer
is drawing a nine-patch?

[hshi1, 2014/07/14 21:23:37]

Done. I'm moving this call the layer owner (Shadow).

+
+  SwitchToLayer(new_layer);
+  nine_patch_layer_ = new_layer;
+}
+
+void Layer::UpdateNinePatchBitmap(const SkBitmap& bitmap,
+                                  const gfx::Rect& aperture,
+                                  const gfx::Rect& border) {
+  if (!nine_patch_layer_.get())

[sky, 2014/07/14 15:30:06]

This be a DCHECK, right?

[hshi1, 2014/07/14 21:23:37]

Done.

+    return;
+
+  nine_patch_layer_->SetBitmap(bitmap);
+  nine_patch_layer_->SetAperture(aperture);
+  nine_patch_layer_->SetBorder(border);
+}
+
 void Layer::SetColor(SkColor color) { GetAnimator()->SetColor(color); }
 
 bool Layer::SchedulePaint(const gfx::Rect& invalid_rect) {
   if (type_ == LAYER_SOLID_COLOR || (!delegate_ && !mailbox_.IsValid()))
     return false;
 
   damaged_region_.op(invalid_rect.x(),
                      invalid_rect.y(),
                      invalid_rect.right(),
                      invalid_rect.bottom(),
@@ skipping 392 matching lines @@
       children_.end(),
       std::bind2nd(std::mem_fun(&Layer::RemoveAnimatorsInTreeFromCollection),
                    collection));
 }
 
 bool Layer::IsAnimating() const {
   return animator_ && animator_->is_animating();
 }
 
 }  // namespace ui