Fix "modal isn't modal in multi displays" issue

ash/wm/system_modal_container_layout_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/wm/system_modal_container_layout_manager.h"
 
+#include <memory>
+
 #include "ash/common/session/session_state_delegate.h"
 #include "ash/common/shell_window_ids.h"
 #include "ash/root_window_controller.h"
 #include "ash/shell.h"
 #include "ash/test/ash_test_base.h"
 #include "ash/wm/window_util.h"
 #include "base/command_line.h"
 #include "base/compiler_specific.h"
 #include "base/run_loop.h"
+#include "ui/aura/client/aura_constants.h"
+#include "ui/aura/test/test_window_delegate.h"
 #include "ui/aura/window.h"
 #include "ui/aura/window_event_dispatcher.h"
 #include "ui/compositor/layer.h"
 #include "ui/compositor/scoped_animation_duration_scale_mode.h"
 #include "ui/compositor/test/layer_animator_test_controller.h"
 #include "ui/events/test/event_generator.h"
 #include "ui/keyboard/keyboard_controller.h"
 #include "ui/keyboard/keyboard_switches.h"
 #include "ui/keyboard/keyboard_ui.h"
 #include "ui/keyboard/keyboard_util.h"
@@ skipping 615 matching lines @@
   // The keyboard gets shown and the dialog should get pushed up, but not get
   // cropped (and aligned to the top).
   ShowKeyboard(true);
   EXPECT_EQ(modal_size.ToString(), modal_window->bounds().size().ToString());
   EXPECT_EQ(modal_origin.x(), modal_window->bounds().x());
   EXPECT_EQ(0, modal_window->bounds().y());
 
   ShowKeyboard(false);
 }
 
+namespace {
+
+class InputTestDelegate : public aura::test::TestWindowDelegate {
+ public:
+  InputTestDelegate() {}
+  ~InputTestDelegate() override {}
+
+  // ui::EventHandler:
+  void OnMouseEvent(ui::MouseEvent* event) override { mouse_event_count_++; }
+
+  int mouse_event_count() const { return mouse_event_count_; }
+
+ private:
+  int mouse_event_count_ = 0;
+
+  DISALLOW_COPY_AND_ASSIGN(InputTestDelegate);
+};
+
+}  // namespace
+
+// Make sure that events are properly blocked in multi displays environment.
+TEST_F(SystemModalContainerLayoutManagerTest, BlockEvent) {
+  UpdateDisplay("500x500, 500x500");
+  InputTestDelegate delegate;
+  std::unique_ptr<aura::Window> window(CreateTestWindowInShellWithDelegate(
+      &delegate, 0, gfx::Rect(0, 0, 100, 100)));
+  window->Show();
+
+  ui::test::EventGenerator e1(Shell::GetPrimaryRootWindow(), window.get());

[James Cook, 2016/06/16 20:31:36]

nit: e1 -> generator, or event_generator, or something longer

[oshima, 2016/06/17 00:06:12]

Done.

+  e1.ClickLeftButton();

[James Cook, 2016/06/16 20:31:36]

optional: Do you need to do this click? It might be clearer just to go to the
"events should be blocked" case and have the mouse_event_count() be 0.

[oshima, 2016/06/17 00:06:12]

I want to make sure I can receive the event first, but I can remove if you want.
let me know.

[James Cook, 2016/06/17 16:04:43]

No, this is fine.

+  EXPECT_EQ(2, delegate.mouse_event_count());
+
+  views::Widget* widget = views::Widget::CreateWindowWithContextAndBounds(
+      new TestWindow(true), Shell::GetPrimaryRootWindow(),
+      gfx::Rect(200, 200, 100, 100));
+  widget->Show();
+  EXPECT_TRUE(Shell::GetInstance()->IsSystemModalWindowOpen());
+
+  // Events should be blocked.
+  e1.ClickLeftButton();
+  EXPECT_EQ(2, delegate.mouse_event_count());
+  widget->Close();
+
+  EXPECT_FALSE(Shell::GetInstance()->IsSystemModalWindowOpen());
+  e1.ClickLeftButton();
+  EXPECT_EQ(4, delegate.mouse_event_count());
+
+  // We need to delete window here because delegate is on the stack.
+  window.reset();
+}
+
 }  // namespace test
 }  // namespace ash