Add gfx::ImageSkia and icon support to aura::PropertyConverter.

ui/views/mus/native_widget_mus_unittest.cc

 // Copyright 2016 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 "base/callback.h"
 #include "base/macros.h"
 #include "services/ui/public/cpp/property_type_converters.h"
 #include "services/ui/public/cpp/tests/window_tree_client_private.h"
 #include "services/ui/public/cpp/window.h"
@@ skipping 85 matching lines @@
     changes_.push_back(active);
   }
 
  private:
   Widget* widget_;
   std::vector<bool> changes_;
 
   DISALLOW_COPY_AND_ASSIGN(WidgetActivationObserver);
 };
 
-// A WidgetDelegate that supplies an app icon.
+// A WidgetDelegate that supplies app and window icons.
 class TestWidgetDelegate : public WidgetDelegateView {
  public:
-  explicit TestWidgetDelegate(const SkBitmap& icon)
-      : app_icon_(gfx::ImageSkia::CreateFrom1xBitmap(icon)) {}
+  explicit TestWidgetDelegate(const SkBitmap& app_icon,
+                              const SkBitmap& window_icon)
+      : app_icon_(gfx::ImageSkia::CreateFrom1xBitmap(app_icon)),
+        window_icon_(gfx::ImageSkia::CreateFrom1xBitmap(window_icon)) {}
 
   ~TestWidgetDelegate() override {}
 
-  void SetIcon(const SkBitmap& icon) {
+  void SetAppIcon(const SkBitmap& icon) {
     app_icon_ = gfx::ImageSkia::CreateFrom1xBitmap(icon);
   }
 
+  void SetWindowIcon(const SkBitmap& icon) {
+    window_icon_ = gfx::ImageSkia::CreateFrom1xBitmap(icon);
+  }
+
   // views::WidgetDelegate:
   gfx::ImageSkia GetWindowAppIcon() override { return app_icon_; }
+  gfx::ImageSkia GetWindowIcon() override { return window_icon_; }
 
  private:
   gfx::ImageSkia app_icon_;
+  gfx::ImageSkia window_icon_;
 
   DISALLOW_COPY_AND_ASSIGN(TestWidgetDelegate);
 };
 
 class WidgetDelegateWithHitTestMask : public WidgetDelegateView {
  public:
   explicit WidgetDelegateWithHitTestMask(const gfx::Rect& mask_rect)
       : mask_rect_(mask_rect) {}
 
   ~WidgetDelegateWithHitTestMask() override {}
@@ skipping 108 matching lines @@
   widget.Init(params);
   widget.Show();
 
   // The widget is not currently active.
   EXPECT_FALSE(widget.IsActive());
 
   // The widget was never active.
   EXPECT_EQ(0u, activation_observer.changes().size());
 }
 
-// Tests that a window with an icon sets the ui::Window icon property.
-TEST_F(NativeWidgetMusTest, AppIcon) {
-  // Create a Widget with a bitmap as the icon.
-  SkBitmap source_bitmap = MakeBitmap(SK_ColorRED);
+// Tests that a window with app/window icons sets ui::Window icon properties.
+TEST_F(NativeWidgetMusTest, AppAndWindowIcons) {
+  // Create a Widget with app and window icons.
+  SkBitmap app_icon_input = MakeBitmap(SK_ColorRED);
+  SkBitmap window_icon_input = MakeBitmap(SK_ColorGREEN);
   std::unique_ptr<Widget> widget(
-      CreateWidget(new TestWidgetDelegate(source_bitmap)));
+      CreateWidget(new TestWidgetDelegate(app_icon_input, window_icon_input)));
 
-  // The ui::Window has the icon property.
+  // The ui::Window has the expected icon properties.
   ui::Window* window =
       static_cast<NativeWidgetMus*>(widget->native_widget_private())->window();
+  EXPECT_TRUE(
+      window->HasSharedProperty(ui::mojom::WindowManager::kAppIcon_Property));
   EXPECT_TRUE(window->HasSharedProperty(
-      ui::mojom::WindowManager::kWindowAppIcon_Property));
-
-  // The icon is the expected icon.
-  SkBitmap icon = window->GetSharedProperty<SkBitmap>(
-      ui::mojom::WindowManager::kWindowAppIcon_Property);
-  EXPECT_TRUE(gfx::BitmapsAreEqual(source_bitmap, icon));
+      ui::mojom::WindowManager::kWindowIcon_Property));
+  SkBitmap app_icon = window->GetSharedProperty<SkBitmap>(
+      ui::mojom::WindowManager::kAppIcon_Property);
+  EXPECT_TRUE(gfx::BitmapsAreEqual(app_icon_input, app_icon));
+  SkBitmap window_icon = window->GetSharedProperty<SkBitmap>(
+      ui::mojom::WindowManager::kWindowIcon_Property);
+  EXPECT_TRUE(gfx::BitmapsAreEqual(window_icon_input, window_icon));
 }
 
-// Tests that a window without an icon does not set the ui::Window icon
-// property.
-TEST_F(NativeWidgetMusTest, NoAppIcon) {
-  // Create a Widget without a special icon.
+// Tests that a window without icons does not set ui::Window icon properties.
+TEST_F(NativeWidgetMusTest, NoAppNorWindowIcon) {
+  // Create a Widget without app or window icons, by supplying a null delegate.
   std::unique_ptr<Widget> widget(CreateWidget(nullptr));
 
-  // The ui::Window does not have an icon property.
+  // The ui::Window does not have either icon property.
   ui::Window* window =
       static_cast<NativeWidgetMus*>(widget->native_widget_private())->window();
+  EXPECT_FALSE(
+      window->HasSharedProperty(ui::mojom::WindowManager::kAppIcon_Property));
   EXPECT_FALSE(window->HasSharedProperty(
-      ui::mojom::WindowManager::kWindowAppIcon_Property));
+      ui::mojom::WindowManager::kWindowIcon_Property));
 }
 
-// Tests that changing the icon on a Widget updates the ui::Window icon
-// property.
-TEST_F(NativeWidgetMusTest, ChangeAppIcon) {
-  // Create a Widget with an icon.
+// Tests that changing icons on a Widget updates ui::Window icon properties.
+TEST_F(NativeWidgetMusTest, ChangeAppAndWindowIcons) {
+  // Create a Widget with app and window icons.
   SkBitmap bitmap1 = MakeBitmap(SK_ColorRED);
-  TestWidgetDelegate* delegate = new TestWidgetDelegate(bitmap1);
+  TestWidgetDelegate* delegate = new TestWidgetDelegate(bitmap1, bitmap1);
   std::unique_ptr<Widget> widget(CreateWidget(delegate));
-
-  // Update the icon to a new image.
-  SkBitmap bitmap2 = MakeBitmap(SK_ColorGREEN);
-  delegate->SetIcon(bitmap2);
-  widget->UpdateWindowIcon();
-
-  // The window has the updated icon.
   ui::Window* window =
       static_cast<NativeWidgetMus*>(widget->native_widget_private())->window();
-  SkBitmap icon = window->GetSharedProperty<SkBitmap>(
-      ui::mojom::WindowManager::kWindowAppIcon_Property);
-  EXPECT_TRUE(gfx::BitmapsAreEqual(bitmap2, icon));
+
+  // Update the app icon image; verify the window has the updated icon.
+  SkBitmap bitmap2 = MakeBitmap(SK_ColorGREEN);
+  delegate->SetAppIcon(bitmap2);
+  widget->UpdateWindowIcon();
+  SkBitmap app_icon = window->GetSharedProperty<SkBitmap>(
+      ui::mojom::WindowManager::kAppIcon_Property);
+  EXPECT_TRUE(gfx::BitmapsAreEqual(bitmap2, app_icon));
+
+  // Update the window icon image; verify the window has the updated icon.
+  SkBitmap bitmap3 = MakeBitmap(SK_ColorBLUE);
+  delegate->SetWindowIcon(bitmap3);
+  widget->UpdateWindowIcon();
+  SkBitmap window_icon = window->GetSharedProperty<SkBitmap>(
+      ui::mojom::WindowManager::kWindowIcon_Property);
+  EXPECT_TRUE(gfx::BitmapsAreEqual(bitmap3, window_icon));
 }
 
 TEST_F(NativeWidgetMusTest, ValidLayerTree) {
   std::unique_ptr<Widget> widget(CreateWidget(nullptr));
   View* content = new View;
   content->SetPaintToLayer(true);
   widget->GetContentsView()->AddChildView(content);
   EXPECT_TRUE(widget->GetNativeWindow()->layer()->Contains(content->layer()));
 }
 
@@ skipping 341 matching lines @@
   not_child_params.name = "Not Child Widget";
   not_child_params.native_widget =
       new NativeWidgetMus(not_child_widget.get(), not_child_window,
                           ui::mojom::CompositorFrameSinkType::DEFAULT);
   not_child_widget->Init(not_child_params);
 
   EXPECT_NE(not_child_window->parent(), parent_window);
 }
 
 }  // namespace views