| Index: chromeos/display/output_configurator_unittest.cc
|
| diff --git a/chromeos/display/output_configurator_unittest.cc b/chromeos/display/output_configurator_unittest.cc
|
| index 3d4fbfc11e8afb96445c5c8a4cefeb16d292464e..8f69ce7433058fbf1d3a6df6f10046f76f694433 100644
|
| --- a/chromeos/display/output_configurator_unittest.cc
|
| +++ b/chromeos/display/output_configurator_unittest.cc
|
| @@ -208,8 +208,8 @@ class TestStateController : public OutputConfigurator::StateController {
|
| void set_state(OutputState state) { state_ = state; }
|
|
|
| // OutputConfigurator::StateController overrides:
|
| - virtual OutputState GetStateForOutputs(
|
| - const OutputSnapshotList& outputs) const OVERRIDE { return state_; }
|
| + virtual OutputState GetStateForDisplayIds(
|
| + const std::vector<int64>& outputs) const OVERRIDE { return state_; }
|
|
|
| private:
|
| OutputState state_;
|
| @@ -240,6 +240,7 @@ class OutputConfiguratorTest : public testing::Test {
|
| o->is_internal = true;
|
| o->is_aspect_preserving_scaling = true;
|
| o->touch_device_id = 0;
|
| + o->has_display_id = true;
|
|
|
| o = &outputs_[1];
|
| o->output = 2;
|
| @@ -252,6 +253,7 @@ class OutputConfiguratorTest : public testing::Test {
|
| o->is_internal = false;
|
| o->is_aspect_preserving_scaling = true;
|
| o->touch_device_id = 0;
|
| + o->has_display_id = true;
|
|
|
| UpdateOutputs(2);
|
| delegate_->AddMode(kSmallModeId, kSmallModeWidth, kSmallModeHeight, false);
|
| @@ -309,19 +311,6 @@ class OutputConfiguratorTest : public testing::Test {
|
|
|
| } // namespace
|
|
|
| -TEST_F(OutputConfiguratorTest, IsInternalOutputName) {
|
| - EXPECT_TRUE(OutputConfigurator::IsInternalOutputName("LVDS"));
|
| - EXPECT_TRUE(OutputConfigurator::IsInternalOutputName("eDP"));
|
| - EXPECT_TRUE(OutputConfigurator::IsInternalOutputName("LVDSxx"));
|
| - EXPECT_TRUE(OutputConfigurator::IsInternalOutputName("eDPzz"));
|
| -
|
| - EXPECT_FALSE(OutputConfigurator::IsInternalOutputName("xyz"));
|
| - EXPECT_FALSE(OutputConfigurator::IsInternalOutputName("abcLVDS"));
|
| - EXPECT_FALSE(OutputConfigurator::IsInternalOutputName("cdeeDP"));
|
| - EXPECT_FALSE(OutputConfigurator::IsInternalOutputName("LVD"));
|
| - EXPECT_FALSE(OutputConfigurator::IsInternalOutputName("eD"));
|
| -}
|
| -
|
| TEST_F(OutputConfiguratorTest, ConnectSecondOutput) {
|
| InitWithSingleOutput();
|
|
|
| @@ -594,4 +583,21 @@ TEST_F(OutputConfiguratorTest, InvalidOutputStates) {
|
| EXPECT_TRUE(configurator_.SetDisplayMode(STATE_DUAL_EXTENDED));
|
| }
|
|
|
| +TEST_F(OutputConfiguratorTest, GetOutputStateForDisplays) {
|
| + outputs_[0].has_display_id = false;
|
| + UpdateOutputs(2);
|
| +
|
| + configurator_.Init(false, 0);
|
| + configurator_.Start();
|
| +
|
| + state_controller_.set_state(STATE_DUAL_MIRROR);
|
| + test_api_.SendOutputChangeEvents(true);
|
| + EXPECT_EQ(STATE_DUAL_EXTENDED, configurator_.output_state());
|
| +
|
| + outputs_[0].has_display_id = true;
|
| + UpdateOutputs(2);
|
| + test_api_.SendOutputChangeEvents(true);
|
| + EXPECT_EQ(STATE_DUAL_MIRROR, configurator_.output_state());
|
| +}
|
| +
|
| } // namespace chromeos
|
|
|