James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 1 | // Copyright 2013 The Chromium Authors. All rights reserved. |
| 2 | // Use of this source code is governed by a BSD-style license that can be |
| 3 | // found in the LICENSE file. |
| 4 | |
| 5 | #include "ash/shelf/shelf_window_watcher.h" |
| 6 | |
Mitsuru Oshima | 04b54d0 | 2017-10-09 14:22:45 | [diff] [blame] | 7 | #include <memory> |
| 8 | |
sky | e5fd122 | 2017-04-12 18:43:23 | [diff] [blame] | 9 | #include "ash/public/cpp/config.h" |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 10 | #include "ash/public/cpp/shelf_item.h" |
msw | 109806d | 2017-06-02 20:11:57 | [diff] [blame] | 11 | #include "ash/public/cpp/shelf_model.h" |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 12 | #include "ash/public/cpp/shell_window_ids.h" |
| 13 | #include "ash/public/cpp/window_properties.h" |
| 14 | #include "ash/root_window_controller.h" |
| 15 | #include "ash/session/session_controller.h" |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 16 | #include "ash/shell.h" |
| 17 | #include "ash/test/ash_test_base.h" |
| 18 | #include "ash/wm/window_resizer.h" |
| 19 | #include "ash/wm/window_state.h" |
msw | 19b30c2c | 2017-06-01 03:21:40 | [diff] [blame] | 20 | #include "third_party/skia/include/core/SkBitmap.h" |
| 21 | #include "ui/aura/client/aura_constants.h" |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 22 | #include "ui/aura/window.h" |
| 23 | #include "ui/base/hit_test.h" |
msw | 19b30c2c | 2017-06-01 03:21:40 | [diff] [blame] | 24 | #include "ui/base/resource/resource_bundle.h" |
| 25 | #include "ui/gfx/image/image_skia.h" |
| 26 | #include "ui/resources/grit/ui_resources.h" |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 27 | #include "ui/views/widget/widget.h" |
msw | 70ac45f | 2017-06-05 02:02:45 | [diff] [blame] | 28 | #include "ui/wm/core/transient_window_controller.h" |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 29 | |
| 30 | namespace ash { |
msw | 19b30c2c | 2017-06-01 03:21:40 | [diff] [blame] | 31 | namespace { |
| 32 | |
| 33 | // Create a test 1x1 icon image with a given |color|. |
| 34 | gfx::ImageSkia CreateImageSkiaIcon(SkColor color) { |
| 35 | SkBitmap bitmap; |
| 36 | bitmap.allocN32Pixels(1, 1); |
| 37 | bitmap.eraseColor(color); |
| 38 | return gfx::ImageSkia::CreateFrom1xBitmap(bitmap); |
| 39 | } |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 40 | |
James Cook | 317781a | 2017-07-18 02:08:06 | [diff] [blame] | 41 | class ShelfWindowWatcherTest : public AshTestBase { |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 42 | public: |
| 43 | ShelfWindowWatcherTest() : model_(nullptr) {} |
Chris Watkins | c24daf6 | 2017-11-28 03:43:09 | [diff] [blame^] | 44 | ~ShelfWindowWatcherTest() override = default; |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 45 | |
| 46 | void SetUp() override { |
James Cook | 317781a | 2017-07-18 02:08:06 | [diff] [blame] | 47 | AshTestBase::SetUp(); |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 48 | model_ = Shell::Get()->shelf_model(); |
Mike Wasserman | 717cd57 | 2017-09-23 17:01:40 | [diff] [blame] | 49 | // ShelfModel creates an app list item. |
| 50 | ASSERT_EQ(1, model_->item_count()); |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 51 | } |
| 52 | |
| 53 | void TearDown() override { |
| 54 | model_ = nullptr; |
James Cook | 317781a | 2017-07-18 02:08:06 | [diff] [blame] | 55 | AshTestBase::TearDown(); |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 56 | } |
| 57 | |
msw | 6958e7f | 2017-05-15 22:55:06 | [diff] [blame] | 58 | static ShelfID CreateShelfItem(aura::Window* window) { |
msw | 84b8a5f | 2017-05-05 00:13:36 | [diff] [blame] | 59 | static int id = 0; |
msw | 70ac45f | 2017-06-05 02:02:45 | [diff] [blame] | 60 | ShelfID shelf_id(std::to_string(id++)); |
msw | 6958e7f | 2017-05-15 22:55:06 | [diff] [blame] | 61 | window->SetProperty(kShelfIDKey, new std::string(shelf_id.Serialize())); |
| 62 | window->SetProperty(kShelfItemTypeKey, static_cast<int32_t>(TYPE_DIALOG)); |
msw | 84b8a5f | 2017-05-05 00:13:36 | [diff] [blame] | 63 | return shelf_id; |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 64 | } |
| 65 | |
| 66 | protected: |
| 67 | ShelfModel* model_; |
| 68 | |
| 69 | private: |
| 70 | DISALLOW_COPY_AND_ASSIGN(ShelfWindowWatcherTest); |
| 71 | }; |
| 72 | |
| 73 | // Ensure shelf items are added and removed as windows are opened and closed. |
| 74 | TEST_F(ShelfWindowWatcherTest, OpenAndClose) { |
msw | 70ac45f | 2017-06-05 02:02:45 | [diff] [blame] | 75 | // Windows with valid ShelfItemType and ShelfID properties get shelf items. |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 76 | std::unique_ptr<views::Widget> widget1 = |
| 77 | CreateTestWidget(nullptr, kShellWindowId_DefaultContainer, gfx::Rect()); |
msw | 6958e7f | 2017-05-15 22:55:06 | [diff] [blame] | 78 | CreateShelfItem(widget1->GetNativeWindow()); |
Mike Wasserman | 717cd57 | 2017-09-23 17:01:40 | [diff] [blame] | 79 | EXPECT_EQ(2, model_->item_count()); |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 80 | std::unique_ptr<views::Widget> widget2 = |
| 81 | CreateTestWidget(nullptr, kShellWindowId_DefaultContainer, gfx::Rect()); |
msw | 6958e7f | 2017-05-15 22:55:06 | [diff] [blame] | 82 | CreateShelfItem(widget2->GetNativeWindow()); |
Mike Wasserman | 717cd57 | 2017-09-23 17:01:40 | [diff] [blame] | 83 | EXPECT_EQ(3, model_->item_count()); |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 84 | |
| 85 | // Each ShelfItem is removed when the associated window is destroyed. |
| 86 | widget1.reset(); |
Michael Wasserman | 78b6f3e | 2017-07-20 19:51:20 | [diff] [blame] | 87 | EXPECT_EQ(2, model_->item_count()); |
Mike Wasserman | 717cd57 | 2017-09-23 17:01:40 | [diff] [blame] | 88 | widget2.reset(); |
| 89 | EXPECT_EQ(1, model_->item_count()); |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 90 | } |
| 91 | |
msw | feae432 | 2017-06-08 04:24:22 | [diff] [blame] | 92 | // Ensure shelf items are added and removed for some unknown windows in mash. |
msw | 70ac45f | 2017-06-05 02:02:45 | [diff] [blame] | 93 | TEST_F(ShelfWindowWatcherTest, OpenAndCloseMash) { |
| 94 | if (Shell::GetAshConfig() != Config::MASH) |
| 95 | return; |
| 96 | |
msw | 70ac45f | 2017-06-05 02:02:45 | [diff] [blame] | 97 | // Windows with no valid ShelfItemType and ShelfID properties get shelf items. |
| 98 | std::unique_ptr<views::Widget> widget1 = |
| 99 | CreateTestWidget(nullptr, kShellWindowId_DefaultContainer, gfx::Rect()); |
Mike Wasserman | 717cd57 | 2017-09-23 17:01:40 | [diff] [blame] | 100 | EXPECT_EQ(2, model_->item_count()); |
msw | 70ac45f | 2017-06-05 02:02:45 | [diff] [blame] | 101 | std::unique_ptr<views::Widget> widget2 = |
| 102 | CreateTestWidget(nullptr, kShellWindowId_DefaultContainer, gfx::Rect()); |
Mike Wasserman | 717cd57 | 2017-09-23 17:01:40 | [diff] [blame] | 103 | EXPECT_EQ(3, model_->item_count()); |
msw | 70ac45f | 2017-06-05 02:02:45 | [diff] [blame] | 104 | |
| 105 | // Each ShelfItem is removed when the associated window is destroyed. |
| 106 | widget1.reset(); |
Michael Wasserman | 78b6f3e | 2017-07-20 19:51:20 | [diff] [blame] | 107 | EXPECT_EQ(2, model_->item_count()); |
Mike Wasserman | 717cd57 | 2017-09-23 17:01:40 | [diff] [blame] | 108 | widget2.reset(); |
| 109 | EXPECT_EQ(1, model_->item_count()); |
msw | 70ac45f | 2017-06-05 02:02:45 | [diff] [blame] | 110 | |
msw | feae432 | 2017-06-08 04:24:22 | [diff] [blame] | 111 | // Windows with type WINDOW_TYPE_NORMAL get shelf items, others do not. |
| 112 | aura::client::WindowType no_item_types[] = { |
| 113 | aura::client::WINDOW_TYPE_UNKNOWN, aura::client::WINDOW_TYPE_NORMAL, |
| 114 | aura::client::WINDOW_TYPE_POPUP, aura::client::WINDOW_TYPE_CONTROL, |
| 115 | aura::client::WINDOW_TYPE_PANEL, aura::client::WINDOW_TYPE_MENU, |
| 116 | aura::client::WINDOW_TYPE_TOOLTIP}; |
| 117 | for (aura::client::WindowType type : no_item_types) { |
| 118 | std::unique_ptr<aura::Window> window = |
Mitsuru Oshima | 04b54d0 | 2017-10-09 14:22:45 | [diff] [blame] | 119 | std::make_unique<aura::Window>(nullptr, type); |
msw | feae432 | 2017-06-08 04:24:22 | [diff] [blame] | 120 | window->Init(ui::LAYER_NOT_DRAWN); |
| 121 | Shell::GetPrimaryRootWindow() |
| 122 | ->GetChildById(kShellWindowId_DefaultContainer) |
| 123 | ->AddChild(window.get()); |
| 124 | window->Show(); |
Mike Wasserman | 717cd57 | 2017-09-23 17:01:40 | [diff] [blame] | 125 | EXPECT_EQ(type == aura::client::WINDOW_TYPE_NORMAL ? 2 : 1, |
msw | feae432 | 2017-06-08 04:24:22 | [diff] [blame] | 126 | model_->item_count()); |
| 127 | } |
| 128 | |
msw | 70ac45f | 2017-06-05 02:02:45 | [diff] [blame] | 129 | // Windows with WindowState::ignored_by_shelf set do not get shelf items. |
| 130 | widget1 = |
| 131 | CreateTestWidget(nullptr, kShellWindowId_DefaultContainer, gfx::Rect()); |
| 132 | wm::GetWindowState(widget1->GetNativeWindow())->set_ignored_by_shelf(true); |
| 133 | // TODO(msw): Make the flag a window property and remove this workaround. |
| 134 | widget1->GetNativeWindow()->SetProperty(aura::client::kDrawAttentionKey, |
| 135 | true); |
Mike Wasserman | 717cd57 | 2017-09-23 17:01:40 | [diff] [blame] | 136 | EXPECT_EQ(1, model_->item_count()); |
msw | 70ac45f | 2017-06-05 02:02:45 | [diff] [blame] | 137 | } |
| 138 | |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 139 | TEST_F(ShelfWindowWatcherTest, CreateAndRemoveShelfItemProperties) { |
| 140 | // TODO: investigate failure in mash. http://crbug.com/695562. |
sky | e5fd122 | 2017-04-12 18:43:23 | [diff] [blame] | 141 | if (Shell::GetAshConfig() == Config::MASH) |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 142 | return; |
| 143 | |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 144 | // Creating windows without a valid ShelfItemType does not add items. |
Mike Wasserman | 717cd57 | 2017-09-23 17:01:40 | [diff] [blame] | 145 | EXPECT_EQ(1, model_->item_count()); |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 146 | std::unique_ptr<views::Widget> widget1 = |
| 147 | CreateTestWidget(nullptr, kShellWindowId_DefaultContainer, gfx::Rect()); |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 148 | std::unique_ptr<views::Widget> widget2 = |
| 149 | CreateTestWidget(nullptr, kShellWindowId_DefaultContainer, gfx::Rect()); |
Mike Wasserman | 717cd57 | 2017-09-23 17:01:40 | [diff] [blame] | 150 | EXPECT_EQ(1, model_->item_count()); |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 151 | |
| 152 | // Create a ShelfItem for the first window. |
msw | 6958e7f | 2017-05-15 22:55:06 | [diff] [blame] | 153 | ShelfID id_w1 = CreateShelfItem(widget1->GetNativeWindow()); |
Mike Wasserman | 717cd57 | 2017-09-23 17:01:40 | [diff] [blame] | 154 | EXPECT_EQ(2, model_->item_count()); |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 155 | |
| 156 | int index_w1 = model_->ItemIndexByID(id_w1); |
| 157 | EXPECT_EQ(STATUS_RUNNING, model_->items()[index_w1].status); |
| 158 | |
| 159 | // Create a ShelfItem for the second window. |
msw | 6958e7f | 2017-05-15 22:55:06 | [diff] [blame] | 160 | ShelfID id_w2 = CreateShelfItem(widget2->GetNativeWindow()); |
Mike Wasserman | 717cd57 | 2017-09-23 17:01:40 | [diff] [blame] | 161 | EXPECT_EQ(3, model_->item_count()); |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 162 | |
| 163 | int index_w2 = model_->ItemIndexByID(id_w2); |
James Cook | 263e93e | 2017-11-08 17:37:24 | [diff] [blame] | 164 | EXPECT_EQ(STATUS_RUNNING, model_->items()[index_w2].status); |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 165 | |
| 166 | // ShelfItem is removed when the item type window property is cleared. |
msw | 6958e7f | 2017-05-15 22:55:06 | [diff] [blame] | 167 | widget1->GetNativeWindow()->SetProperty(kShelfItemTypeKey, |
| 168 | static_cast<int32_t>(TYPE_UNDEFINED)); |
Mike Wasserman | 717cd57 | 2017-09-23 17:01:40 | [diff] [blame] | 169 | EXPECT_EQ(2, model_->item_count()); |
msw | 6958e7f | 2017-05-15 22:55:06 | [diff] [blame] | 170 | widget2->GetNativeWindow()->SetProperty(kShelfItemTypeKey, |
| 171 | static_cast<int32_t>(TYPE_UNDEFINED)); |
Mike Wasserman | 717cd57 | 2017-09-23 17:01:40 | [diff] [blame] | 172 | EXPECT_EQ(1, model_->item_count()); |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 173 | // Clearing twice doesn't do anything. |
msw | 6958e7f | 2017-05-15 22:55:06 | [diff] [blame] | 174 | widget2->GetNativeWindow()->SetProperty(kShelfItemTypeKey, |
| 175 | static_cast<int32_t>(TYPE_UNDEFINED)); |
Mike Wasserman | 717cd57 | 2017-09-23 17:01:40 | [diff] [blame] | 176 | EXPECT_EQ(1, model_->item_count()); |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 177 | } |
| 178 | |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 179 | TEST_F(ShelfWindowWatcherTest, UpdateWindowProperty) { |
Michael Wasserman | 78b6f3e | 2017-07-20 19:51:20 | [diff] [blame] | 180 | // Create a ShelfItem for a new window. |
Mike Wasserman | 717cd57 | 2017-09-23 17:01:40 | [diff] [blame] | 181 | EXPECT_EQ(1, model_->item_count()); |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 182 | std::unique_ptr<views::Widget> widget = |
| 183 | CreateTestWidget(nullptr, kShellWindowId_DefaultContainer, gfx::Rect()); |
msw | 6958e7f | 2017-05-15 22:55:06 | [diff] [blame] | 184 | ShelfID id = CreateShelfItem(widget->GetNativeWindow()); |
Mike Wasserman | 717cd57 | 2017-09-23 17:01:40 | [diff] [blame] | 185 | EXPECT_EQ(2, model_->item_count()); |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 186 | |
| 187 | int index = model_->ItemIndexByID(id); |
James Cook | 263e93e | 2017-11-08 17:37:24 | [diff] [blame] | 188 | EXPECT_EQ(STATUS_RUNNING, model_->items()[index].status); |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 189 | |
msw | 6958e7f | 2017-05-15 22:55:06 | [diff] [blame] | 190 | // Update the window's ShelfItemType. |
| 191 | widget->GetNativeWindow()->SetProperty(kShelfItemTypeKey, |
msw | feae432 | 2017-06-08 04:24:22 | [diff] [blame] | 192 | static_cast<int32_t>(TYPE_APP_PANEL)); |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 193 | // No new item is created after updating a launcher item. |
Mike Wasserman | 717cd57 | 2017-09-23 17:01:40 | [diff] [blame] | 194 | EXPECT_EQ(2, model_->item_count()); |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 195 | // index and id are not changed after updating a launcher item. |
| 196 | EXPECT_EQ(index, model_->ItemIndexByID(id)); |
| 197 | EXPECT_EQ(id, model_->items()[index].id); |
| 198 | } |
| 199 | |
| 200 | TEST_F(ShelfWindowWatcherTest, MaximizeAndRestoreWindow) { |
Michael Wasserman | 78b6f3e | 2017-07-20 19:51:20 | [diff] [blame] | 201 | // Create a ShelfItem for a new window. |
Mike Wasserman | 717cd57 | 2017-09-23 17:01:40 | [diff] [blame] | 202 | EXPECT_EQ(1, model_->item_count()); |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 203 | std::unique_ptr<views::Widget> widget = |
| 204 | CreateTestWidget(nullptr, kShellWindowId_DefaultContainer, gfx::Rect()); |
msw | 6958e7f | 2017-05-15 22:55:06 | [diff] [blame] | 205 | ShelfID id = CreateShelfItem(widget->GetNativeWindow()); |
Mike Wasserman | 717cd57 | 2017-09-23 17:01:40 | [diff] [blame] | 206 | EXPECT_EQ(2, model_->item_count()); |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 207 | |
| 208 | int index = model_->ItemIndexByID(id); |
James Cook | 263e93e | 2017-11-08 17:37:24 | [diff] [blame] | 209 | EXPECT_EQ(STATUS_RUNNING, model_->items()[index].status); |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 210 | |
msw | 6958e7f | 2017-05-15 22:55:06 | [diff] [blame] | 211 | // Maximize the window. |
Michael Wasserman | 78b6f3e | 2017-07-20 19:51:20 | [diff] [blame] | 212 | wm::WindowState* window_state = wm::GetWindowState(widget->GetNativeWindow()); |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 213 | EXPECT_FALSE(window_state->IsMaximized()); |
| 214 | window_state->Maximize(); |
| 215 | EXPECT_TRUE(window_state->IsMaximized()); |
msw | 6958e7f | 2017-05-15 22:55:06 | [diff] [blame] | 216 | // No new item is created after maximizing the window. |
Mike Wasserman | 717cd57 | 2017-09-23 17:01:40 | [diff] [blame] | 217 | EXPECT_EQ(2, model_->item_count()); |
msw | 6958e7f | 2017-05-15 22:55:06 | [diff] [blame] | 218 | // index and id are not changed after maximizing the window. |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 219 | EXPECT_EQ(index, model_->ItemIndexByID(id)); |
| 220 | EXPECT_EQ(id, model_->items()[index].id); |
| 221 | |
msw | 6958e7f | 2017-05-15 22:55:06 | [diff] [blame] | 222 | // Restore the window. |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 223 | window_state->Restore(); |
| 224 | EXPECT_FALSE(window_state->IsMaximized()); |
msw | 6958e7f | 2017-05-15 22:55:06 | [diff] [blame] | 225 | // No new item is created after restoring the window. |
Mike Wasserman | 717cd57 | 2017-09-23 17:01:40 | [diff] [blame] | 226 | EXPECT_EQ(2, model_->item_count()); |
msw | 6958e7f | 2017-05-15 22:55:06 | [diff] [blame] | 227 | // Index and id are not changed after maximizing the window. |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 228 | EXPECT_EQ(index, model_->ItemIndexByID(id)); |
| 229 | EXPECT_EQ(id, model_->items()[index].id); |
| 230 | } |
| 231 | |
| 232 | // Check |window|'s item is not changed during the dragging. |
| 233 | // TODO(simonhong): Add a test for removing a Window during the dragging. |
| 234 | TEST_F(ShelfWindowWatcherTest, DragWindow) { |
Michael Wasserman | 78b6f3e | 2017-07-20 19:51:20 | [diff] [blame] | 235 | // Create a ShelfItem for a new window. |
Mike Wasserman | 717cd57 | 2017-09-23 17:01:40 | [diff] [blame] | 236 | EXPECT_EQ(1, model_->item_count()); |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 237 | std::unique_ptr<views::Widget> widget = |
| 238 | CreateTestWidget(nullptr, kShellWindowId_DefaultContainer, gfx::Rect()); |
msw | 6958e7f | 2017-05-15 22:55:06 | [diff] [blame] | 239 | ShelfID id = CreateShelfItem(widget->GetNativeWindow()); |
Mike Wasserman | 717cd57 | 2017-09-23 17:01:40 | [diff] [blame] | 240 | EXPECT_EQ(2, model_->item_count()); |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 241 | |
| 242 | int index = model_->ItemIndexByID(id); |
James Cook | 263e93e | 2017-11-08 17:37:24 | [diff] [blame] | 243 | EXPECT_EQ(STATUS_RUNNING, model_->items()[index].status); |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 244 | |
msw | 6958e7f | 2017-05-15 22:55:06 | [diff] [blame] | 245 | // Simulate dragging of the window and check its item is not changed. |
sky | a5e9a12 | 2017-05-22 20:45:35 | [diff] [blame] | 246 | std::unique_ptr<WindowResizer> resizer( |
| 247 | CreateWindowResizer(widget->GetNativeWindow(), gfx::Point(), HTCAPTION, |
Thiago Farina | 3b086a0 | 2017-05-30 22:32:50 | [diff] [blame] | 248 | ::wm::WINDOW_MOVE_SOURCE_MOUSE)); |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 249 | ASSERT_TRUE(resizer.get()); |
| 250 | resizer->Drag(gfx::Point(50, 50), 0); |
| 251 | resizer->CompleteDrag(); |
| 252 | |
msw | 6958e7f | 2017-05-15 22:55:06 | [diff] [blame] | 253 | // Index and id are not changed after dragging the window. |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 254 | EXPECT_EQ(index, model_->ItemIndexByID(id)); |
| 255 | EXPECT_EQ(id, model_->items()[index].id); |
| 256 | } |
| 257 | |
msw | feae432 | 2017-06-08 04:24:22 | [diff] [blame] | 258 | // Ensure panels and dialogs get shelf items. |
| 259 | TEST_F(ShelfWindowWatcherTest, PanelAndDialogWindows) { |
msw | feae432 | 2017-06-08 04:24:22 | [diff] [blame] | 260 | // An item is created for a dialog window. |
Mike Wasserman | 717cd57 | 2017-09-23 17:01:40 | [diff] [blame] | 261 | EXPECT_EQ(1, model_->item_count()); |
msw | feae432 | 2017-06-08 04:24:22 | [diff] [blame] | 262 | std::unique_ptr<views::Widget> dialog_widget = |
msw | 6958e7f | 2017-05-15 22:55:06 | [diff] [blame] | 263 | CreateTestWidget(nullptr, kShellWindowId_DefaultContainer, gfx::Rect()); |
msw | feae432 | 2017-06-08 04:24:22 | [diff] [blame] | 264 | aura::Window* dialog = dialog_widget->GetNativeWindow(); |
| 265 | dialog->SetProperty(kShelfIDKey, new std::string(ShelfID("a").Serialize())); |
| 266 | dialog->SetProperty(kShelfItemTypeKey, static_cast<int32_t>(TYPE_DIALOG)); |
Mike Wasserman | 717cd57 | 2017-09-23 17:01:40 | [diff] [blame] | 267 | EXPECT_EQ(2, model_->item_count()); |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 268 | |
msw | feae432 | 2017-06-08 04:24:22 | [diff] [blame] | 269 | // An item is created for a panel window. |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 270 | views::Widget panel_widget; |
| 271 | views::Widget::InitParams panel_params(views::Widget::InitParams::TYPE_PANEL); |
| 272 | panel_params.ownership = views::Widget::InitParams::WIDGET_OWNS_NATIVE_WIDGET; |
Scott Violet | 66193f7 | 2017-06-26 22:24:18 | [diff] [blame] | 273 | panel_params.parent = Shell::GetPrimaryRootWindow()->GetChildById( |
| 274 | kShellWindowId_PanelContainer); |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 275 | panel_widget.Init(panel_params); |
| 276 | panel_widget.Show(); |
msw | feae432 | 2017-06-08 04:24:22 | [diff] [blame] | 277 | aura::Window* panel = panel_widget.GetNativeWindow(); |
| 278 | panel->SetProperty(kShelfIDKey, new std::string(ShelfID("b").Serialize())); |
| 279 | panel->SetProperty(kShelfItemTypeKey, static_cast<int32_t>(TYPE_APP_PANEL)); |
Mike Wasserman | 717cd57 | 2017-09-23 17:01:40 | [diff] [blame] | 280 | EXPECT_EQ(3, model_->item_count()); |
msw | feae432 | 2017-06-08 04:24:22 | [diff] [blame] | 281 | |
| 282 | // An item is not created for an app window. |
| 283 | std::unique_ptr<views::Widget> app_widget = |
| 284 | CreateTestWidget(nullptr, kShellWindowId_DefaultContainer, gfx::Rect()); |
| 285 | aura::Window* app = app_widget->GetNativeWindow(); |
| 286 | app->SetProperty(kShelfIDKey, new std::string(ShelfID("c").Serialize())); |
| 287 | app->SetProperty(kShelfItemTypeKey, static_cast<int32_t>(TYPE_APP)); |
Mike Wasserman | 717cd57 | 2017-09-23 17:01:40 | [diff] [blame] | 288 | EXPECT_EQ(3, model_->item_count()); |
msw | feae432 | 2017-06-08 04:24:22 | [diff] [blame] | 289 | app_widget.reset(); |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 290 | |
| 291 | // Each ShelfItem is removed when the associated window is destroyed. |
| 292 | panel_widget.CloseNow(); |
Michael Wasserman | 78b6f3e | 2017-07-20 19:51:20 | [diff] [blame] | 293 | EXPECT_EQ(2, model_->item_count()); |
Mike Wasserman | 717cd57 | 2017-09-23 17:01:40 | [diff] [blame] | 294 | dialog_widget.reset(); |
| 295 | EXPECT_EQ(1, model_->item_count()); |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 296 | } |
| 297 | |
msw | 19b30c2c | 2017-06-01 03:21:40 | [diff] [blame] | 298 | // Ensure items use the app icon and window icon aura::Window properties. |
| 299 | TEST_F(ShelfWindowWatcherTest, ItemIcon) { |
msw | 19b30c2c | 2017-06-01 03:21:40 | [diff] [blame] | 300 | // Create a ShelfItem for a window; it should have a default icon. |
Mike Wasserman | 717cd57 | 2017-09-23 17:01:40 | [diff] [blame] | 301 | EXPECT_EQ(1, model_->item_count()); |
msw | 19b30c2c | 2017-06-01 03:21:40 | [diff] [blame] | 302 | std::unique_ptr<views::Widget> widget = |
| 303 | CreateTestWidget(nullptr, kShellWindowId_DefaultContainer, gfx::Rect()); |
| 304 | aura::Window* window = widget->GetNativeWindow(); |
| 305 | ShelfID id = CreateShelfItem(window); |
Mike Wasserman | 717cd57 | 2017-09-23 17:01:40 | [diff] [blame] | 306 | EXPECT_EQ(2, model_->item_count()); |
msw | 19b30c2c | 2017-06-01 03:21:40 | [diff] [blame] | 307 | ui::ResourceBundle& rb = ui::ResourceBundle::GetSharedInstance(); |
| 308 | gfx::Image default_image = rb.GetImageNamed(IDR_DEFAULT_FAVICON_32); |
Mike Wasserman | 717cd57 | 2017-09-23 17:01:40 | [diff] [blame] | 309 | EXPECT_TRUE(model_->items()[1].image.BackedBySameObjectAs( |
msw | 19b30c2c | 2017-06-01 03:21:40 | [diff] [blame] | 310 | default_image.AsImageSkia())); |
| 311 | |
| 312 | // Setting a window icon should update the item icon. |
| 313 | const gfx::ImageSkia red = CreateImageSkiaIcon(SK_ColorRED); |
| 314 | window->SetProperty(aura::client::kWindowIconKey, new gfx::ImageSkia(red)); |
Mike Wasserman | 717cd57 | 2017-09-23 17:01:40 | [diff] [blame] | 315 | EXPECT_EQ(SK_ColorRED, model_->items()[1].image.bitmap()->getColor(0, 0)); |
msw | 19b30c2c | 2017-06-01 03:21:40 | [diff] [blame] | 316 | |
| 317 | // Setting an app icon should override the window icon. |
| 318 | const gfx::ImageSkia blue = CreateImageSkiaIcon(SK_ColorBLUE); |
| 319 | window->SetProperty(aura::client::kAppIconKey, new gfx::ImageSkia(blue)); |
Mike Wasserman | 717cd57 | 2017-09-23 17:01:40 | [diff] [blame] | 320 | EXPECT_EQ(SK_ColorBLUE, model_->items()[1].image.bitmap()->getColor(0, 0)); |
msw | 19b30c2c | 2017-06-01 03:21:40 | [diff] [blame] | 321 | |
| 322 | // Clearing the app icon should restore the window icon to the shelf item. |
| 323 | window->ClearProperty(aura::client::kAppIconKey); |
Mike Wasserman | 717cd57 | 2017-09-23 17:01:40 | [diff] [blame] | 324 | EXPECT_EQ(SK_ColorRED, model_->items()[1].image.bitmap()->getColor(0, 0)); |
msw | 19b30c2c | 2017-06-01 03:21:40 | [diff] [blame] | 325 | } |
| 326 | |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 327 | TEST_F(ShelfWindowWatcherTest, DontCreateShelfEntriesForChildWindows) { |
Mike Wasserman | 717cd57 | 2017-09-23 17:01:40 | [diff] [blame] | 328 | EXPECT_EQ(1, model_->item_count()); |
msw | feae432 | 2017-06-08 04:24:22 | [diff] [blame] | 329 | std::unique_ptr<aura::Window> window = |
Mitsuru Oshima | 04b54d0 | 2017-10-09 14:22:45 | [diff] [blame] | 330 | std::make_unique<aura::Window>(nullptr, aura::client::WINDOW_TYPE_NORMAL); |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 331 | window->Init(ui::LAYER_NOT_DRAWN); |
msw | feae432 | 2017-06-08 04:24:22 | [diff] [blame] | 332 | window->SetProperty(kShelfIDKey, new std::string(ShelfID("a").Serialize())); |
| 333 | window->SetProperty(kShelfItemTypeKey, static_cast<int32_t>(TYPE_DIALOG)); |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 334 | Shell::GetPrimaryRootWindow() |
| 335 | ->GetChildById(kShellWindowId_DefaultContainer) |
| 336 | ->AddChild(window.get()); |
| 337 | window->Show(); |
Mike Wasserman | 717cd57 | 2017-09-23 17:01:40 | [diff] [blame] | 338 | EXPECT_EQ(2, model_->item_count()); |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 339 | |
msw | feae432 | 2017-06-08 04:24:22 | [diff] [blame] | 340 | std::unique_ptr<aura::Window> child = |
Mitsuru Oshima | 04b54d0 | 2017-10-09 14:22:45 | [diff] [blame] | 341 | std::make_unique<aura::Window>(nullptr, aura::client::WINDOW_TYPE_NORMAL); |
msw | feae432 | 2017-06-08 04:24:22 | [diff] [blame] | 342 | child->Init(ui::LAYER_NOT_DRAWN); |
| 343 | child->SetProperty(kShelfIDKey, new std::string(ShelfID("b").Serialize())); |
| 344 | child->SetProperty(kShelfItemTypeKey, static_cast<int32_t>(TYPE_DIALOG)); |
| 345 | window->AddChild(child.get()); |
| 346 | child->Show(); |
| 347 | // There should not be a new shelf item for |child|. |
Mike Wasserman | 717cd57 | 2017-09-23 17:01:40 | [diff] [blame] | 348 | EXPECT_EQ(2, model_->item_count()); |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 349 | |
msw | feae432 | 2017-06-08 04:24:22 | [diff] [blame] | 350 | child.reset(); |
Michael Wasserman | 78b6f3e | 2017-07-20 19:51:20 | [diff] [blame] | 351 | EXPECT_EQ(2, model_->item_count()); |
Mike Wasserman | 717cd57 | 2017-09-23 17:01:40 | [diff] [blame] | 352 | window.reset(); |
| 353 | EXPECT_EQ(1, model_->item_count()); |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 354 | } |
| 355 | |
msw | feae432 | 2017-06-08 04:24:22 | [diff] [blame] | 356 | TEST_F(ShelfWindowWatcherTest, CreateShelfEntriesForTransientWindows) { |
Mike Wasserman | 717cd57 | 2017-09-23 17:01:40 | [diff] [blame] | 357 | EXPECT_EQ(1, model_->item_count()); |
msw | feae432 | 2017-06-08 04:24:22 | [diff] [blame] | 358 | std::unique_ptr<aura::Window> window = |
Mitsuru Oshima | 04b54d0 | 2017-10-09 14:22:45 | [diff] [blame] | 359 | std::make_unique<aura::Window>(nullptr, aura::client::WINDOW_TYPE_NORMAL); |
msw | 70ac45f | 2017-06-05 02:02:45 | [diff] [blame] | 360 | window->Init(ui::LAYER_NOT_DRAWN); |
msw | feae432 | 2017-06-08 04:24:22 | [diff] [blame] | 361 | window->SetProperty(kShelfIDKey, new std::string(ShelfID("a").Serialize())); |
| 362 | window->SetProperty(kShelfItemTypeKey, static_cast<int32_t>(TYPE_DIALOG)); |
msw | 70ac45f | 2017-06-05 02:02:45 | [diff] [blame] | 363 | Shell::GetPrimaryRootWindow() |
| 364 | ->GetChildById(kShellWindowId_DefaultContainer) |
| 365 | ->AddChild(window.get()); |
| 366 | window->Show(); |
Mike Wasserman | 717cd57 | 2017-09-23 17:01:40 | [diff] [blame] | 367 | EXPECT_EQ(2, model_->item_count()); |
msw | 70ac45f | 2017-06-05 02:02:45 | [diff] [blame] | 368 | |
msw | feae432 | 2017-06-08 04:24:22 | [diff] [blame] | 369 | std::unique_ptr<aura::Window> transient = |
Mitsuru Oshima | 04b54d0 | 2017-10-09 14:22:45 | [diff] [blame] | 370 | std::make_unique<aura::Window>(nullptr, aura::client::WINDOW_TYPE_NORMAL); |
msw | feae432 | 2017-06-08 04:24:22 | [diff] [blame] | 371 | transient->Init(ui::LAYER_NOT_DRAWN); |
| 372 | transient->SetProperty(kShelfIDKey, |
| 373 | new std::string(ShelfID("b").Serialize())); |
| 374 | transient->SetProperty(kShelfItemTypeKey, static_cast<int32_t>(TYPE_DIALOG)); |
| 375 | Shell::GetPrimaryRootWindow() |
| 376 | ->GetChildById(kShellWindowId_DefaultContainer) |
| 377 | ->AddChild(transient.get()); |
| 378 | ::wm::TransientWindowController::Get()->AddTransientChild(window.get(), |
| 379 | transient.get()); |
| 380 | transient->Show(); |
| 381 | // There should be a new shelf item for |transient|. |
Mike Wasserman | 717cd57 | 2017-09-23 17:01:40 | [diff] [blame] | 382 | EXPECT_EQ(3, model_->item_count()); |
msw | 70ac45f | 2017-06-05 02:02:45 | [diff] [blame] | 383 | |
msw | feae432 | 2017-06-08 04:24:22 | [diff] [blame] | 384 | transient.reset(); |
Michael Wasserman | 78b6f3e | 2017-07-20 19:51:20 | [diff] [blame] | 385 | EXPECT_EQ(2, model_->item_count()); |
Mike Wasserman | 717cd57 | 2017-09-23 17:01:40 | [diff] [blame] | 386 | window.reset(); |
| 387 | EXPECT_EQ(1, model_->item_count()); |
msw | 70ac45f | 2017-06-05 02:02:45 | [diff] [blame] | 388 | } |
| 389 | |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 390 | // Ensures ShelfWindowWatcher supports windows opened prior to session start. |
James Cook | 317781a | 2017-07-18 02:08:06 | [diff] [blame] | 391 | using ShelfWindowWatcherSessionStartTest = NoSessionAshTestBase; |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 392 | TEST_F(ShelfWindowWatcherSessionStartTest, PreExistingWindow) { |
| 393 | ShelfModel* model = Shell::Get()->shelf_model(); |
| 394 | ASSERT_FALSE( |
| 395 | Shell::Get()->session_controller()->IsActiveUserSessionStarted()); |
| 396 | |
Mike Wasserman | 717cd57 | 2017-09-23 17:01:40 | [diff] [blame] | 397 | // ShelfModel creates an app list item. |
| 398 | EXPECT_EQ(1, model->item_count()); |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 399 | |
| 400 | // Construct a window that should get a shelf item once the session starts. |
| 401 | std::unique_ptr<views::Widget> widget = |
| 402 | CreateTestWidget(nullptr, kShellWindowId_DefaultContainer, gfx::Rect()); |
msw | 6958e7f | 2017-05-15 22:55:06 | [diff] [blame] | 403 | ShelfWindowWatcherTest::CreateShelfItem(widget->GetNativeWindow()); |
Mike Wasserman | 717cd57 | 2017-09-23 17:01:40 | [diff] [blame] | 404 | EXPECT_EQ(1, model->item_count()); |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 405 | |
| 406 | // Start the test user session; ShelfWindowWatcher will find the open window. |
Xiyuan Xia | ffbc9203 | 2017-08-21 18:36:13 | [diff] [blame] | 407 | CreateUserSessions(1); |
Mike Wasserman | 717cd57 | 2017-09-23 17:01:40 | [diff] [blame] | 408 | EXPECT_EQ(2, model->item_count()); |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 409 | } |
| 410 | |
msw | 19b30c2c | 2017-06-01 03:21:40 | [diff] [blame] | 411 | } // namespace |
James Cook | b0bf8e8 | 2017-04-09 17:01:44 | [diff] [blame] | 412 | } // namespace ash |