ABANDONED: chromeos: Resume displays when suspend is cancelled.

ash/system/chromeos/power/power_event_observer_unittest.cc

 // Copyright 2013 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 "ash/system/chromeos/power/power_event_observer.h"
 
 #include <memory>
 
 #include "ash/shell.h"
 #include "ash/test/ash_test_base.h"
+#include "base/run_loop.h"
 #include "base/time/time.h"
 #include "chromeos/dbus/dbus_thread_manager.h"
-#include "chromeos/dbus/power_manager_client.h"
+#include "chromeos/dbus/fake_power_manager_client.h"
 #include "ui/aura/window.h"
 #include "ui/aura/window_tree_host.h"
 #include "ui/compositor/compositor.h"
+#include "ui/display/chromeos/display_configurator.h"
+#include "ui/display/chromeos/test/action_logger.h"
+#include "ui/display/chromeos/test/test_native_display_delegate.h"
 
 namespace ash {
 
 class PowerEventObserverTest : public test::AshTestBase {
  public:
   PowerEventObserverTest() {}
   ~PowerEventObserverTest() override {}
 
   // test::AshTestBase::SetUp() overrides:
   void SetUp() override {
     test::AshTestBase::SetUp();
+
+    // Reset any state left in chromeos::FakePowerManagerClient by previous
+    // tests.
+    power_manager_client_ = static_cast<chromeos::FakePowerManagerClient*>(
+        chromeos::DBusThreadManager::Get()->GetPowerManagerClient());
+    power_manager_client_->Reset();
+
     observer_.reset(new PowerEventObserver());
   }
 
   void TearDown() override {
     observer_.reset();
     test::AshTestBase::TearDown();
   }
 
  protected:
   int GetNumVisibleCompositors() {
     int result = 0;
     for (auto* window : Shell::GetAllRootWindows()) {
       if (window->GetHost()->compositor()->IsVisible())
         ++result;
     }
 
     return result;
   }
 
+  chromeos::FakePowerManagerClient* power_manager_client_;  // Non-owned.
   std::unique_ptr<PowerEventObserver> observer_;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(PowerEventObserverTest);
 };
 
 TEST_F(PowerEventObserverTest, LockBeforeSuspend) {
-  chromeos::PowerManagerClient* client =
-      chromeos::DBusThreadManager::Get()->GetPowerManagerClient();
-  ASSERT_EQ(0, client->GetNumPendingSuspendReadinessCallbacks());
+  ASSERT_EQ(0, power_manager_client_->GetNumPendingSuspendReadinessCallbacks());
 
   // Check that the observer requests a suspend-readiness callback when it hears
   // that the system is about to suspend.
   SetCanLockScreen(true);
   SetShouldLockScreenBeforeSuspending(true);
   observer_->SuspendImminent();
-  EXPECT_EQ(1, client->GetNumPendingSuspendReadinessCallbacks());
+  EXPECT_EQ(1, power_manager_client_->GetNumPendingSuspendReadinessCallbacks());
 
   // It should run the callback when it hears that the screen is locked and the
   // lock screen animations have completed.
   observer_->ScreenIsLocked();
   observer_->OnLockAnimationsComplete();
-  EXPECT_EQ(0, client->GetNumPendingSuspendReadinessCallbacks());
+  EXPECT_EQ(0, power_manager_client_->GetNumPendingSuspendReadinessCallbacks());
 
   // If the system is already locked, no callback should be requested.
   observer_->SuspendDone(base::TimeDelta());
   observer_->ScreenIsUnlocked();
   observer_->ScreenIsLocked();
   observer_->OnLockAnimationsComplete();
   observer_->SuspendImminent();
-  EXPECT_EQ(0, client->GetNumPendingSuspendReadinessCallbacks());
+  EXPECT_EQ(0, power_manager_client_->GetNumPendingSuspendReadinessCallbacks());
 
   // It also shouldn't request a callback if it isn't instructed to lock the
   // screen.
   observer_->SuspendDone(base::TimeDelta());
   SetShouldLockScreenBeforeSuspending(false);
   observer_->SuspendImminent();
-  EXPECT_EQ(0, client->GetNumPendingSuspendReadinessCallbacks());
+  EXPECT_EQ(0, power_manager_client_->GetNumPendingSuspendReadinessCallbacks());
 }
 
 TEST_F(PowerEventObserverTest, SetInvisibleBeforeSuspend) {
   // Tests that all the Compositors are marked invisible before a suspend
   // request when the screen is not supposed to be locked before a suspend.
   EXPECT_EQ(1, GetNumVisibleCompositors());
 
   observer_->SuspendImminent();
   EXPECT_EQ(0, GetNumVisibleCompositors());
   observer_->SuspendDone(base::TimeDelta());
@@ skipping 37 matching lines @@
   // - The screen is locked.
   // - The suspend request is canceled.
   // - Another suspend request is started.
   // - The screen lock animations complete.
   //
   // In this case, the observer should block the second suspend request until
   // the animations have completed.
   SetCanLockScreen(true);
   SetShouldLockScreenBeforeSuspending(true);
 
-  chromeos::PowerManagerClient* client =
-      chromeos::DBusThreadManager::Get()->GetPowerManagerClient();
   observer_->SuspendImminent();
-  EXPECT_EQ(1, client->GetNumPendingSuspendReadinessCallbacks());
+  EXPECT_EQ(1, power_manager_client_->GetNumPendingSuspendReadinessCallbacks());
 
   observer_->ScreenIsLocked();
   observer_->SuspendDone(base::TimeDelta());
   observer_->SuspendImminent();
 
   // The expected number of suspend readiness callbacks is 2 because the
   // observer has not run the callback that it got from the first suspend
   // request.  The real PowerManagerClient would reset its internal counter in
   // this situation but the stub client is not that smart.
-  EXPECT_EQ(2, client->GetNumPendingSuspendReadinessCallbacks());
+  EXPECT_EQ(2, power_manager_client_->GetNumPendingSuspendReadinessCallbacks());
 
   observer_->OnLockAnimationsComplete();
-  EXPECT_EQ(1, client->GetNumPendingSuspendReadinessCallbacks());
+  EXPECT_EQ(1, power_manager_client_->GetNumPendingSuspendReadinessCallbacks());
   EXPECT_EQ(0, GetNumVisibleCompositors());
 }
 
+TEST_F(PowerEventObserverTest, SuspendAndResumeDisplays) {
+  // Configure ui::DisplayConfigurator to do asynchronous configuration.
+  ui::test::ActionLogger logger;
+  ui::test::TestNativeDisplayDelegate* delegate =
+      new ui::test::TestNativeDisplayDelegate(&logger);
+  delegate->set_run_async(true);
+  ui::DisplayConfigurator* configurator =
+      Shell::GetInstance()->display_configurator();
+  configurator->SetDelegateForTesting(
+      std::unique_ptr<ui::NativeDisplayDelegate>(delegate));
+
+  // Start a suspend attempt and let the async display-suspending task run.
+  observer_->SuspendImminent();
+  EXPECT_TRUE(configurator->displays_suspended_for_test());
+  base::RunLoop().RunUntilIdle();
+  EXPECT_TRUE(configurator->displays_suspended_for_test());
+
+  // When the system resumes, the displays should be resumed.
+  observer_->SuspendDone(base::TimeDelta());
+  base::RunLoop().RunUntilIdle();
+  EXPECT_FALSE(configurator->displays_suspended_for_test());
+
+  // Now, start another suspend attempt but abort it before the
+  // display-suspending task runs (http://crbug.com/620208). The displays should
+  // still be in the suspended state, since the async task is running and might
+  // undo any resuming that's done at this time.
+  observer_->SuspendImminent();
+  EXPECT_TRUE(configurator->displays_suspended_for_test());
+  observer_->SuspendDone(base::TimeDelta());
+  EXPECT_TRUE(configurator->displays_suspended_for_test());
+
+  // When the task finally completes, the displays should be resumed.
+  base::RunLoop().RunUntilIdle();
+  EXPECT_FALSE(configurator->displays_suspended_for_test());
+
+  // Start suspending and then cancel immediately again.
+  observer_->SuspendImminent();
+  observer_->SuspendDone(base::TimeDelta());
+  EXPECT_TRUE(configurator->displays_suspended_for_test());
+
+  // This time, suspend again before the display-resuming task has a chance to
+  // run. The displays should remain suspended.
+  observer_->SuspendImminent();
+  base::RunLoop().RunUntilIdle();
+  EXPECT_TRUE(configurator->displays_suspended_for_test());
+}
+
 }  // namespace ash