Fixed the ScreenOrientationController so that it doesn't crash by animating

ash/content/display/screen_orientation_controller_chromeos_unittest.cc

 // Copyright 2014 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 <vector>
+
 #include "ash/ash_switches.h"
 #include "ash/content/display/screen_orientation_controller_chromeos.h"
 #include "ash/display/display_info.h"
 #include "ash/display/display_manager.h"
 #include "ash/shell.h"
 #include "ash/test/ash_test_base.h"
 #include "ash/test/ash_test_helper.h"
 #include "ash/test/test_shell_delegate.h"
 #include "ash/test/test_system_tray_delegate.h"
 #include "ash/wm/maximize_mode/maximize_mode_controller.h"
 #include "base/command_line.h"
 #include "base/memory/scoped_ptr.h"
+#include "base/strings/stringprintf.h"
 #include "chromeos/accelerometer/accelerometer_reader.h"
 #include "chromeos/accelerometer/accelerometer_types.h"
 #include "content/public/browser/browser_context.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/test/test_browser_context.h"
 #include "third_party/WebKit/public/platform/WebScreenOrientationLockType.h"
 #include "ui/aura/window.h"
 #include "ui/gfx/display.h"
 #include "ui/message_center/message_center.h"
 #include "ui/views/test/webview_test_helper.h"
 #include "ui/views/view.h"
 #include "ui/views/views_delegate.h"
 #include "ui/wm/public/activation_client.h"
 
 namespace ash {
 
 namespace {
 
 const float kDegreesToRadians = 3.1415926f / 180.0f;
 const float kMeanGravity = 9.8066f;
 
+DisplayInfo CreateDisplayInfo(int64 id, const gfx::Rect& bounds) {
+  DisplayInfo info(id, "dummy", false);
+  info.SetBounds(bounds);
+  return info;
+}
+
 void EnableMaximizeMode(bool enable) {
   Shell::GetInstance()
       ->maximize_mode_controller()
       ->EnableMaximizeModeWindowManager(enable);
 }
 
 gfx::Display::Rotation GetInternalDisplayRotation() {
   return Shell::GetInstance()
       ->display_manager()
       ->GetDisplayInfo(gfx::Display::InternalDisplayId())
@@ skipping 553 matching lines @@
   // Should not crash, even thought there is no internal display.
   SetInternalDisplayRotation(gfx::Display::ROTATE_180);
   EXPECT_FALSE(RotationLocked());
 
   // With an internal display now available, functionality should resume.
   gfx::Display::SetInternalDisplayId(internal_display_id);
   SetInternalDisplayRotation(gfx::Display::ROTATE_90);
   EXPECT_TRUE(RotationLocked());
 }
 
+// Verifies that rotating an inactive Display does not crash. See
+// www.crbug.com/479503.
+TEST_F(ScreenOrientationControllerTest, RotateInactiveDisplayShouldNotCrash) {
+  const int64 kInternalDisplayId = gfx::Display::InternalDisplayId();
+  const int64 kExternalDisplayId = 10;

[jonross, 2015/04/24 14:18:08]

Can this ever be the id of the internal display? If so you may want to
explicitly set the internal display id as well.

[bruthig, 2015/04/24 16:17:13]

Originally I had the assert to verify that kInternalDisplayId !=
kExternalDisplayId but still it is a potential flaky test.  I've updated to
explicitly set the internal display id as well.

+  const gfx::Display::Rotation kNewRotation = gfx::Display::ROTATE_180;
+
+  ASSERT_NE(kInternalDisplayId, gfx::Display::kInvalidDisplayID);
+  ASSERT_NE(Shell::GetInstance()
+                ->display_manager()
+                ->GetDisplayInfo(kInternalDisplayId)
+                .rotation(),
+            kNewRotation);
+
+  const DisplayInfo internal_display_info =
+      CreateDisplayInfo(kInternalDisplayId, gfx::Rect(0, 0, 500, 500));
+  const DisplayInfo external_display_info =
+      CreateDisplayInfo(kExternalDisplayId, gfx::Rect(1, 1, 500, 500));
+
+  std::vector<DisplayInfo> display_info_list_two_active;
+  display_info_list_two_active.push_back(internal_display_info);
+  display_info_list_two_active.push_back(external_display_info);
+
+  std::vector<DisplayInfo> display_info_list_one_active;
+  display_info_list_one_active.push_back(external_display_info);
+
+  // The DisplayInfo list with two active displays needs to be added first so
+  // that the DisplayManager can track the |internal_display_info| as inactive
+  // instead of non-existent.
+  ash::Shell::GetInstance()->display_manager()->UpdateDisplays(
+      display_info_list_two_active);
+  ash::Shell::GetInstance()->display_manager()->UpdateDisplays(
+      display_info_list_one_active);
+
+  delegate()->SetDisplayRotation(kNewRotation);
+}
+
 }  // namespace ash