Chrome OS: avoid suspending on lid close when casting is active.

chromeos/display/output_configurator_unittest.cc

 // Copyright (c) 2012 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 "chromeos/display/output_configurator.h"
 
 #include <cmath>
 #include <cstdarg>
 #include <map>
 #include <string>
@@ skipping 769 matching lines @@
                         GetCrtcAction(outputs_[1].crtc, 0,
                             kSmallModeHeight + OutputConfigurator::kVerticalGap,
                             kBigModeId, outputs_[1].output).c_str(),
                         kForceDPMS, kUngrab, NULL),
             delegate_->GetActionsAndClear());
   EXPECT_EQ(STATE_DUAL_EXTENDED, configurator_.output_state());
   EXPECT_TRUE(mirroring_controller_.software_mirroring_enabled());
   EXPECT_EQ(1, observer_.num_changes());
 }
 
+TEST_F(OutputConfiguratorTest, Casting) {
+  InitWithSingleOutput();
+
+  // Notify configurator that casting session is started.
+  configurator_.OnCastingStateChanged(true);
+  EXPECT_EQ(JoinActions(kProjectingOn, NULL), delegate_->GetActionsAndClear());

[Daniel Erat, 2014/01/17 19:13:51]

nit: after this code, maybe add:

  configurator_.OnCastingStateChanged(true);
  EXPECT_EQ(kProjectingOn, delegate_->GetActionsAndClear());
  configurator_.OnCastingStateChanged(false);
  EXPECT_EQ(kProjectingOn, delegate_->GetActionsAndClear());

to verify that the configurator keeps a count of the number of active sessions
instead of treating it as a single global state.

(at least, it looks to me like each StateChanged() call with result in another
projecting message being sent. if it doesn't, since the state didn't change,
kNoActions is fine too) :-)

[hshi1, 2014/01/17 19:31:28]

Done.

+
+  // Turn all displays off and check that projecting is not turned off.
+  configurator_.SetDisplayPower(DISPLAY_POWER_ALL_OFF,
+                                OutputConfigurator::kSetDisplayPowerNoFlags);
+  EXPECT_EQ(JoinActions(kGrab,
+                        GetFramebufferAction(kSmallModeWidth, kSmallModeHeight,
+                            outputs_[0].crtc, 0).c_str(),
+                        GetCrtcAction(outputs_[0].crtc, 0, 0, 0,
+                            outputs_[0].output).c_str(),
+                        kUngrab, NULL),
+            delegate_->GetActionsAndClear());
+
+  // Turn all displays back on.
+  configurator_.SetDisplayPower(DISPLAY_POWER_ALL_ON,
+                                OutputConfigurator::kSetDisplayPowerNoFlags);
+  EXPECT_EQ(JoinActions(kGrab,
+                        GetFramebufferAction(kSmallModeWidth, kSmallModeHeight,
+                            outputs_[0].crtc, 0).c_str(),
+                        GetCrtcAction(outputs_[0].crtc, 0, 0, kSmallModeId,
+                            outputs_[0].output).c_str(),
+                        kForceDPMS, kUngrab, NULL),
+            delegate_->GetActionsAndClear());
+
+  // Notify configurator that casting session is ended.
+  configurator_.OnCastingStateChanged(false);
+  EXPECT_EQ(JoinActions(kProjectingOff, NULL), delegate_->GetActionsAndClear());
+}
+
 TEST_F(OutputConfiguratorTest, SuspendAndResume) {
   InitWithSingleOutput();
 
   // No preparation is needed before suspending when the display is already
   // on.  The configurator should still reprobe on resume in case a display
   // was connected while suspended.
   configurator_.SuspendDisplays();
   EXPECT_EQ(kNoActions, delegate_->GetActionsAndClear());
   configurator_.ResumeDisplays();
   EXPECT_EQ(JoinActions(kGrab,
@@ skipping 598 matching lines @@
                         GetCrtcAction(outputs_[1].crtc, 0,
                             outputs_[1].mode_infos[kFirstMode].height +
                               OutputConfigurator::kVerticalGap, kFirstMode + 3,
                             outputs_[1].output).c_str(),
                         kUngrab, kProjectingOn, NULL),
             delegate_->GetActionsAndClear());
 
 }
 
 }  // namespace chromeos