Automatically set the overscan insets if the output has the flag and no preference is set.

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/display/display_controller.h"
 #include "ash/screen_ash.h"
 #include "ash/shell.h"
 #include "ash/test/ash_test_base.h"
@@ skipping 496 matching lines @@
   EXPECT_EQ("0,0 100x100", FindDisplayForId(10).bounds().ToString());
 
   // OnNativeDisplaysChanged may change the display bounds.  Here makes sure
   // nothing changed if the exactly same displays are specified.
   display_manager()->OnNativeDisplaysChanged(displays);
   EXPECT_EQ("-500,0 500x500",
             FindDisplayForId(internal_display_id).bounds().ToString());
   EXPECT_EQ("0,0 100x100", FindDisplayForId(10).bounds().ToString());
 }
 
+TEST_F(DisplayManagerTest, AutomaticOverscanInsets) {
+  UpdateDisplay("200x200,400x400");
+
+  std::vector<gfx::Display> displays;
+  displays.push_back(*display_manager()->GetDisplayAt(0));
+  displays.push_back(*display_manager()->GetDisplayAt(1));
+  int64 id = displays[1].id();
+  display_manager()->SetHasOverscanFlagForTest(id, true);
+
+  display_manager()->OnNativeDisplaysChanged(displays);
+  // Didn't specify OverscanInsets but it has.

[oshima, 2013/01/15 23:26:51]

is this comment incomplete?

[Jun Mukai, 2013/01/15 23:43:57]

Done.

+  EXPECT_EQ("11,211 380x380",
+            display_manager()->GetDisplayAt(1)->bounds_in_pixel().ToString());
+
+  // If custom overscan insets is specified, the specified value is used.
+  display_manager()->SetOverscanInsets(id, gfx::Insets(5, 6, 7, 8));
+  display_manager()->OnNativeDisplaysChanged(displays);
+  EXPECT_EQ("7,206 386x388",
+            display_manager()->GetDisplayAt(1)->bounds_in_pixel().ToString());
+
+  // Do not overscan even though it has 'has_overscan' flag, if the custom
+  // insets is empty.
+  display_manager()->SetOverscanInsets(id, gfx::Insets());
+  display_manager()->OnNativeDisplaysChanged(displays);
+  EXPECT_EQ("1,201 400x400",
+            display_manager()->GetDisplayAt(1)->bounds_in_pixel().ToString());
+}
+
 }  // namespace internal
 }  // namespace ash