Do not register invalid layout

ash/display/display_manager_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 "ash/display/display_manager.h"
 
 #include "ash/accelerators/accelerator_commands.h"
 #include "ash/ash_switches.h"
 #include "ash/display/display_info.h"
 #include "ash/display/display_layout_builder.h"
@@ skipping 312 matching lines @@
   UpdateDisplay("640x480,320x200,400x300");
 
   EXPECT_EQ("0,0 640x480",
             display_manager()->GetDisplayAt(0).bounds().ToString());
   EXPECT_EQ("10,480 320x200",
             display_manager()->GetDisplayAt(1).bounds().ToString());
   EXPECT_EQ("20,680 400x300",
             display_manager()->GetDisplayAt(2).bounds().ToString());
 }
 
+TEST_F(DisplayManagerTest, LayoutMorethanThreeDisplaysCrashTest) {
+  if (!SupportsMultipleDisplays())
+    return;
+
+  int64_t primary_id = gfx::Screen::GetScreen()->GetPrimaryDisplay().id();
+  DisplayIdList list =
+      ash::test::CreateDisplayIdList2(primary_id, primary_id + 1);
+
+  DisplayLayoutBuilder builder(primary_id);
+  builder.AddDisplayPlacement(list[1], primary_id, DisplayPlacement::LEFT, 10);
+  builder.AddDisplayPlacement(primary_id + 2, list[1], DisplayPlacement::TOP,
+                              10);
+  display_manager()->layout_store()->RegisterLayoutForDisplayIdList(
+      list, builder.Build());
+  UpdateDisplay("640x480, 500x500");
+}
+
 TEST_F(DisplayManagerTest, LayoutMorethanThreeDisplaysTest) {
   if (!SupportsMultipleDisplays())
     return;
 
   int64_t primary_id = gfx::Screen::GetScreen()->GetPrimaryDisplay().id();
   DisplayIdList list = ash::test::CreateDisplayIdListN(
       3, primary_id, primary_id + 1, primary_id + 2);
   {
     // Layout: [2]
     //         [1][P]
@@ skipping 1869 matching lines @@
   layout_store->RegisterLayoutForDisplayIdList(list, std::move(old_layout));
   const DisplayLayout& stored = layout_store->GetRegisteredDisplayLayout(list);
 
   EXPECT_EQ(id1, stored.placement_list[0]->parent_display_id);
   EXPECT_EQ(id2, stored.placement_list[0]->display_id);
 }
 
 #endif  // OS_CHROMEOS
 
 }  // namespace ash