Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2623)

Unified Diff: ash/mus/test/wm_test_base.cc

Issue 2539363005: Converts ash to use aura-mus (Closed)
Patch Set: merge Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ash/mus/test/wm_test_base.cc
diff --git a/ash/mus/test/wm_test_base.cc b/ash/mus/test/wm_test_base.cc
index 8822401b86c47bfc4be61cebd9917f4f8c4c9914..0640745562c6f24d3264ce6566264da0984e5a6d 100644
--- a/ash/mus/test/wm_test_base.cc
+++ b/ash/mus/test/wm_test_base.cc
@@ -13,7 +13,11 @@
#include "ash/mus/window_manager.h"
#include "ash/mus/window_manager_application.h"
#include "services/ui/public/cpp/property_type_converters.h"
-#include "services/ui/public/cpp/window_tree_client.h"
+#include "ui/aura/mus/property_converter.h"
+#include "ui/aura/mus/window_tree_client.h"
+#include "ui/aura/window.h"
+#include "ui/compositor/scoped_animation_duration_scale_mode.h"
+#include "ui/display/display.h"
namespace ash {
namespace mus {
@@ -67,14 +71,14 @@ void WmTestBase::UpdateDisplay(const std::string& display_spec) {
test_helper_->UpdateDisplay(display_spec);
}
-ui::Window* WmTestBase::GetPrimaryRootWindow() {
+aura::Window* WmTestBase::GetPrimaryRootWindow() {
std::vector<RootWindowController*> roots =
test_helper_->GetRootsOrderedByDisplayId();
DCHECK(!roots.empty());
return roots[0]->root();
}
-ui::Window* WmTestBase::GetSecondaryRootWindow() {
+aura::Window* WmTestBase::GetSecondaryRootWindow() {
std::vector<RootWindowController*> roots =
test_helper_->GetRootsOrderedByDisplayId();
return roots.size() < 2 ? nullptr : roots[1]->root();
@@ -93,71 +97,71 @@ display::Display WmTestBase::GetSecondaryDisplay() {
return roots.size() < 2 ? display::Display() : roots[1]->display();
}
-ui::Window* WmTestBase::CreateTestWindow(const gfx::Rect& bounds) {
+aura::Window* WmTestBase::CreateTestWindow(const gfx::Rect& bounds) {
return CreateTestWindow(bounds, ui::wm::WINDOW_TYPE_NORMAL);
}
-ui::Window* WmTestBase::CreateTestWindow(const gfx::Rect& bounds,
- ui::wm::WindowType window_type) {
+aura::Window* WmTestBase::CreateTestWindow(const gfx::Rect& bounds,
+ ui::wm::WindowType window_type) {
+ const ui::mojom::WindowType mus_window_type =
James Cook 2016/12/05 19:21:45 nit: move down closer to where it's used
sky 2016/12/05 21:39:20 Done.
+ MusWindowTypeFromWmWindowType(window_type);
std::map<std::string, std::vector<uint8_t>> properties;
- properties[ui::mojom::WindowManager::kWindowType_Property] =
- mojo::ConvertTo<std::vector<uint8_t>>(
- static_cast<int32_t>(MusWindowTypeFromWmWindowType(window_type)));
if (!bounds.IsEmpty()) {
properties[ui::mojom::WindowManager::kInitialBounds_Property] =
mojo::ConvertTo<std::vector<uint8_t>>(bounds);
}
+
properties[ui::mojom::WindowManager::kResizeBehavior_Property] =
mojo::ConvertTo<std::vector<uint8_t>>(
- ui::mojom::kResizeBehaviorCanResize |
- ui::mojom::kResizeBehaviorCanMaximize |
- ui::mojom::kResizeBehaviorCanMinimize);
-
- ui::Window* window = test_helper_->GetRootsOrderedByDisplayId()[0]
- ->window_manager()
- ->NewTopLevelWindow(&properties);
- window->SetVisible(true);
- // Most tests expect a minimum size of 0x0.
- WmWindowMusTestApi(WmWindowMus::Get(window)).set_use_empty_minimum_size(true);
+ static_cast<aura::PropertyConverter::PrimitiveType>(
+ ui::mojom::kResizeBehaviorCanResize |
+ ui::mojom::kResizeBehaviorCanMaximize |
+ ui::mojom::kResizeBehaviorCanMinimize));
+
+ aura::Window* window = test_helper_->GetRootsOrderedByDisplayId()[0]
+ ->window_manager()
+ ->NewTopLevelWindow(mus_window_type, &properties);
+ window->Show();
return window;
}
-ui::Window* WmTestBase::CreateFullscreenTestWindow(int64_t display_id) {
+aura::Window* WmTestBase::CreateFullscreenTestWindow(int64_t display_id) {
std::map<std::string, std::vector<uint8_t>> properties;
properties[ui::mojom::WindowManager::kShowState_Property] =
mojo::ConvertTo<std::vector<uint8_t>>(
- static_cast<int32_t>(ui::mojom::ShowState::FULLSCREEN));
-
+ static_cast<aura::PropertyConverter::PrimitiveType>(
+ ui::mojom::ShowState::FULLSCREEN));
if (display_id != display::kInvalidDisplayId) {
properties[ui::mojom::WindowManager::kInitialDisplayId_Property] =
mojo::ConvertTo<std::vector<uint8_t>>(display_id);
}
-
- ui::Window* window = test_helper_->GetRootsOrderedByDisplayId()[0]
- ->window_manager()
- ->NewTopLevelWindow(&properties);
- window->SetVisible(true);
+ aura::Window* window =
+ test_helper_->GetRootsOrderedByDisplayId()[0]
+ ->window_manager()
+ ->NewTopLevelWindow(ui::mojom::WindowType::WINDOW, &properties);
+ window->Show();
return window;
}
-ui::Window* WmTestBase::CreateChildTestWindow(ui::Window* parent,
- const gfx::Rect& bounds) {
+aura::Window* WmTestBase::CreateChildTestWindow(aura::Window* parent,
+ const gfx::Rect& bounds) {
std::map<std::string, std::vector<uint8_t>> properties;
- properties[ui::mojom::WindowManager::kWindowType_Property] =
- mojo::ConvertTo<std::vector<uint8_t>>(static_cast<int32_t>(
- MusWindowTypeFromWmWindowType(ui::wm::WINDOW_TYPE_NORMAL)));
- ui::Window* window = test_helper_->GetRootsOrderedByDisplayId()[0]
- ->root()
- ->window_tree()
- ->NewWindow(&properties);
+ aura::Window* window = new aura::Window(nullptr);
+ window->Init(ui::LAYER_TEXTURED);
window->SetBounds(bounds);
- window->SetVisible(true);
+ window->Show();
parent->AddChild(window);
return window;
}
void WmTestBase::SetUp() {
setup_called_ = true;
+ // Disable animations during tests.
+ zero_duration_mode_ = base::MakeUnique<ui::ScopedAnimationDurationScaleMode>(
+ ui::ScopedAnimationDurationScaleMode::ZERO_DURATION);
+ // Most tests expect a minimum size of 0x0.
+ minimum_size_lock_ =
+ base::MakeUnique<WmWindowMusTestApi::GlobalMinimumSizeLock>();
test_helper_.reset(new WmTestHelper);
test_helper_->Init();
}
@@ -165,6 +169,8 @@ void WmTestBase::SetUp() {
void WmTestBase::TearDown() {
teardown_called_ = true;
test_helper_.reset();
+ minimum_size_lock_.reset();
+ zero_duration_mode_.reset();
}
} // namespace mus

Powered by Google App Engine
This is Rietveld 408576698