Add tests to verify accelerators properly work on constrained window.

ui/views/test/test_widget_observer.h

Index: ui/views/test/test_widget_observer.h
diff --git a/ui/views/test/test_widget_observer.h b/ui/views/test/test_widget_observer.h
new file mode 100644
index 0000000000000000000000000000000000000000..4bed16f0757b9fe87a18f2dbe2e14281243e4de3
--- /dev/null
+++ b/ui/views/test/test_widget_observer.h
@@ -0,0 +1,34 @@
+// 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.
+
+#ifndef UI_VIEWS_TEST_TEST_WIDGET_OBSERVER_H_
+#define UI_VIEWS_TEST_TEST_WIDGET_OBSERVER_H_
+
+#include <stddef.h>

[sky, 2012/12/21 15:46:01]

Is this include really needed?

[mazda, 2013/01/07 22:41:14]

This file cannot be compiled without this because NULL is defined in stddef.h.

+
+#include "base/compiler_specific.h"
+#include "ui/views/widget/widget_observer.h"
+
+namespace views {
+namespace test {
+
+// A Widget observer class used in the tests below to observe bubbles closing.
+class TestWidgetObserver : public WidgetObserver {
+ public:
+  explicit TestWidgetObserver(Widget* widget);
+  virtual ~TestWidgetObserver();
+
+  bool widget_closed() const { return widget_ == NULL; }
+
+ private:
+  // WidgetObserver overrides:
+  virtual void OnWidgetClosing(Widget* widget) OVERRIDE;
+
+  Widget* widget_;
+};

[sky, 2012/12/21 15:46:01]

DISALLOW_...

[mazda, 2013/01/07 22:41:14]

Done.

+
+}  // namespace test
+}  // namespace views
+
+#endif  // UI_VIEWS_TEST_TEST_WIDGET_OBSERVER_H_