Shadows: crop corner tiles instead of hiding.

ui/wm/core/shadow_unittest.cc

Index: ui/wm/core/shadow_unittest.cc
diff --git a/ui/wm/core/shadow_unittest.cc b/ui/wm/core/shadow_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..2090b060d382d4411bda911160e1dc9f9260250b
--- /dev/null
+++ b/ui/wm/core/shadow_unittest.cc
@@ -0,0 +1,153 @@
+// Copyright 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 "ui/wm/core/shadow.h"
+
+#include "base/memory/scoped_ptr.h"
+#include "base/path_service.h"
+#include "third_party/skia/include/core/SkBitmap.h"
+#include "ui/aura/test/aura_test_base.h"
+#include "ui/aura/test/test_windows.h"
+#include "ui/aura/window.h"
+#include "ui/base/resource/resource_bundle.h"
+#include "ui/base/ui_base_paths.h"
+#include "ui/compositor/layer.h"
+#include "ui/compositor/layer_tree_owner.h"
+#include "ui/resources/grit/ui_resources.h"
+
+namespace wm {
+
+namespace {
+
+const int kSmallBitmapSize = 129;
+const int kLargeBitmapSize = 269;
+
+// Mock for the ResourceBundle::Delegate class.
+class MockResourceBundleDelegate : public ui::ResourceBundle::Delegate {
+ public:
+  MockResourceBundleDelegate() : last_resource_id_(0) {
+    SkBitmap bitmap_small, bitmap_large;
+    bitmap_small.allocPixels(
+        SkImageInfo::MakeN32Premul(kSmallBitmapSize, kSmallBitmapSize));
+    bitmap_large.allocPixels(
+        SkImageInfo::MakeN32Premul(kLargeBitmapSize, kLargeBitmapSize));
+    image_small_ = gfx::Image::CreateFrom1xBitmap(bitmap_small);
+    image_large_ = gfx::Image::CreateFrom1xBitmap(bitmap_large);
+  }
+  virtual ~MockResourceBundleDelegate() {}
+
+  // ResourceBundle::Delegate:
+  virtual base::FilePath GetPathForResourcePack(
+      const base::FilePath& pack_path,
+      ui::ScaleFactor scale_factor) OVERRIDE {
+    return base::FilePath();
+  }
+  virtual base::FilePath GetPathForLocalePack(
+      const base::FilePath& pack_path,
+      const std::string& locale) OVERRIDE {
+    return base::FilePath();
+  }
+  virtual gfx::Image GetImageNamed(int resource_id) OVERRIDE {
+    last_resource_id_ = resource_id;
+    switch (resource_id) {
+      case IDR_WINDOW_BUBBLE_SHADOW_SMALL:
+        return image_small_;
+      case IDR_AURA_SHADOW_ACTIVE:
+      case IDR_AURA_SHADOW_INACTIVE:
+        return image_large_;
+      default:
+        NOTREACHED();
+        return gfx::Image();
+    }
+  }
+  virtual gfx::Image GetNativeImageNamed(
+      int resource_id, ui::ResourceBundle::ImageRTL rtl) OVERRIDE {
+    return gfx::Image();
+  }
+  virtual base::RefCountedStaticMemory* LoadDataResourceBytes(
+      int resource_id, ui::ScaleFactor scale_factor) OVERRIDE {
+    return NULL;
+  }
+  virtual bool GetRawDataResource(
+      int resource_id, ui::ScaleFactor scale_factor,
+      base::StringPiece* value) OVERRIDE {
+    return false;
+  }
+  virtual bool GetLocalizedString(
+      int message_id, base::string16* value) OVERRIDE {
+    return false;
+  }
+  virtual scoped_ptr<gfx::Font> GetFont(
+      ui::ResourceBundle::FontStyle style) OVERRIDE {
+    return scoped_ptr<gfx::Font>();
+  }
+
+  int last_resource_id() const { return last_resource_id_; }
+
+ private:
+  gfx::Image image_small_;
+  gfx::Image image_large_;
+  int last_resource_id_;
+
+  DISALLOW_COPY_AND_ASSIGN(MockResourceBundleDelegate);
+};
+
+}  // namespace
+
+class ShadowTest: public aura::test::AuraTestBase {
+ public:
+  ShadowTest() {}
+  virtual ~ShadowTest() {}
+
+  MockResourceBundleDelegate* delegate() { return delegate_.get(); }
+
+  // aura::testAuraBase:
+  virtual void SetUp() OVERRIDE {
+    aura::test::AuraTestBase::SetUp();
+    delegate_.reset(new MockResourceBundleDelegate());
+    if (ResourceBundle::HasSharedInstance())
+      ui::ResourceBundle::CleanupSharedInstance();
+    ui::ResourceBundle::InitSharedInstanceWithLocale(

[tfarina, 2015/01/16 19:33:39]

Why do you need to initialize ResourceBundle here? If we are initializing it for
all tests in
https://chromium.googlesource.com/chromium/src/+/master/ui/wm/test/run_all_un...

+        "en-US", delegate(), ui::ResourceBundle::LOAD_COMMON_RESOURCES);
+  }
+  virtual void TearDown() OVERRIDE {
+    ui::ResourceBundle::CleanupSharedInstance();
+    base::FilePath ui_test_pak_path;
+    ASSERT_TRUE(PathService::Get(ui::UI_TEST_PAK, &ui_test_pak_path));
+    ui::ResourceBundle::InitSharedInstanceWithPakPath(ui_test_pak_path);

[tfarina, 2015/01/16 19:33:39]

Why are you initializing (again) ResourceBundle in the teardown pass of the test
setup? Sorry, without knowing the context that does not look right.

+    aura::test::AuraTestBase::TearDown();
+  }
+ private:
+  scoped_ptr<MockResourceBundleDelegate> delegate_;
+  DISALLOW_COPY_AND_ASSIGN(ShadowTest);
+};
+
+// Test if the proper image is set for the specified style.
+TEST_F(ShadowTest, UpdateImagesForStyle) {
+  Shadow shadow;
+
+  shadow.Init(Shadow::STYLE_SMALL);
+  EXPECT_EQ(delegate()->last_resource_id(), IDR_WINDOW_BUBBLE_SHADOW_SMALL);
+  shadow.SetStyle(Shadow::STYLE_ACTIVE);
+  EXPECT_EQ(delegate()->last_resource_id(), IDR_AURA_SHADOW_ACTIVE);
+  shadow.SetStyle(Shadow::STYLE_INACTIVE);
+  EXPECT_EQ(delegate()->last_resource_id(), IDR_AURA_SHADOW_INACTIVE);
+}
+
+// Test if the proper content bounds is calculated based on the current style.
+TEST_F(ShadowTest, SetContentBounds) {
+  Shadow shadow;
+
+  // Verify that layer bounds are inset from content bounds.
+  shadow.Init(Shadow::STYLE_ACTIVE);
+  gfx::Rect content_bounds(100, 100, 300, 300);
+  shadow.SetContentBounds(content_bounds);
+  EXPECT_EQ(shadow.content_bounds(), content_bounds);
+  EXPECT_EQ(shadow.layer()->bounds(), gfx::Rect(36, 36, 428, 428));
+
+  shadow.SetStyle(Shadow::STYLE_SMALL);
+  EXPECT_EQ(shadow.content_bounds(), content_bounds);
+  EXPECT_EQ(shadow.layer()->bounds(), gfx::Rect(96, 96, 308, 308));
+}
+}  // namespace wm