| Index: ash/wm/lock_state_controller_unittest.cc
|
| diff --git a/ash/wm/lock_state_controller_unittest.cc b/ash/wm/lock_state_controller_unittest.cc
|
| index e8f7b38017eb410f317d35be417ab5a1a11049a5..3f1501f6fe357b1769b170fb6958f270049408d9 100644
|
| --- a/ash/wm/lock_state_controller_unittest.cc
|
| +++ b/ash/wm/lock_state_controller_unittest.cc
|
| @@ -24,7 +24,7 @@
|
| #include "chromeos/dbus/dbus_thread_manager.h"
|
| #include "chromeos/dbus/fake_session_manager_client.h"
|
| #include "ui/display/chromeos/display_configurator.h"
|
| -#include "ui/display/chromeos/test/test_display_snapshot.h"
|
| +#include "ui/display/fake_display_snapshot.h"
|
| #include "ui/display/types/display_constants.h"
|
| #include "ui/events/test/event_generator.h"
|
| #include "ui/gfx/geometry/size.h"
|
| @@ -887,30 +887,30 @@ TEST_F(LockStateControllerTest, IgnorePowerButtonIfScreenIsOff) {
|
| }
|
|
|
| TEST_F(LockStateControllerTest, HonorPowerButtonInDockedMode) {
|
| - std::vector<std::unique_ptr<const ui::DisplayMode>> modes;
|
| - modes.push_back(
|
| - base::MakeUnique<ui::DisplayMode>(gfx::Size(1, 1), false, 60.0f));
|
| -
|
| // Create two outputs, the first internal and the second external.
|
| ui::DisplayConfigurator::DisplayStateList outputs;
|
| - ui::TestDisplaySnapshot internal_display;
|
| - internal_display.set_type(ui::DISPLAY_CONNECTION_TYPE_INTERNAL);
|
| - internal_display.set_modes(std::move(modes));
|
| - outputs.push_back(&internal_display);
|
| -
|
| - modes.clear();
|
| - modes.push_back(
|
| - base::MakeUnique<ui::DisplayMode>(gfx::Size(1, 1), false, 60.0f));
|
| - ui::TestDisplaySnapshot external_display;
|
| - external_display.set_type(ui::DISPLAY_CONNECTION_TYPE_HDMI);
|
| - external_display.set_modes(std::move(modes));
|
| - outputs.push_back(&external_display);
|
| +
|
| + std::unique_ptr<ui::DisplaySnapshot> internal_display =
|
| + display::FakeDisplaySnapshot::Builder()
|
| + .SetId(123)
|
| + .SetNativeMode(gfx::Size(1, 1))
|
| + .SetType(ui::DISPLAY_CONNECTION_TYPE_INTERNAL)
|
| + .Build();
|
| + outputs.push_back(internal_display.get());
|
| +
|
| + std::unique_ptr<ui::DisplaySnapshot> external_display =
|
| + display::FakeDisplaySnapshot::Builder()
|
| + .SetId(456)
|
| + .SetNativeMode(gfx::Size(1, 1))
|
| + .SetType(ui::DISPLAY_CONNECTION_TYPE_HDMI)
|
| + .Build();
|
| + outputs.push_back(external_display.get());
|
|
|
| // When all of the displays are turned off (e.g. due to user inactivity), the
|
| // power button should be ignored.
|
| power_button_controller_->OnScreenBrightnessChanged(0.0);
|
| - internal_display.set_current_mode(nullptr);
|
| - external_display.set_current_mode(nullptr);
|
| + internal_display->set_current_mode(nullptr);
|
| + external_display->set_current_mode(nullptr);
|
| power_button_controller_->OnDisplayModeChanged(outputs);
|
| PressPowerButton();
|
| EXPECT_FALSE(test_api_->is_animating_lock());
|
| @@ -919,7 +919,7 @@ TEST_F(LockStateControllerTest, HonorPowerButtonInDockedMode) {
|
| // When the screen brightness is 0% but the external display is still turned
|
| // on (indicating either docked mode or the user having manually decreased the
|
| // brightness to 0%), the power button should still be handled.
|
| - external_display.set_current_mode(external_display.modes().back().get());
|
| + external_display->set_current_mode(external_display->modes().back().get());
|
| power_button_controller_->OnDisplayModeChanged(outputs);
|
| PressPowerButton();
|
| EXPECT_TRUE(test_api_->is_animating_lock());
|
|
|