Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(398)

Unified Diff: chrome/browser/ui/panels/panel_browsertest.cc

Issue 10825210: Accelerator support for browserless panels on Windows. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix win_aura Created 8 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/ui/panels/panel.cc ('k') | chrome/browser/ui/panels/panel_view.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/panels/panel_browsertest.cc
diff --git a/chrome/browser/ui/panels/panel_browsertest.cc b/chrome/browser/ui/panels/panel_browsertest.cc
index 7fd665e7a9dd51080843246655b495b78eac5960..b7c121e6c71286c2fee53e3b8a03f7c5f5f276ff 100644
--- a/chrome/browser/ui/panels/panel_browsertest.cc
+++ b/chrome/browser/ui/panels/panel_browsertest.cc
@@ -32,6 +32,7 @@
#include "chrome/common/url_constants.h"
#include "chrome/test/base/ui_test_utils.h"
#include "content/public/browser/download_manager.h"
+#include "content/public/browser/native_web_keyboard_event.h"
#include "content/public/browser/notification_service.h"
#include "content/public/browser/web_contents.h"
#include "content/public/common/url_constants.h"
@@ -1545,4 +1546,44 @@ IN_PROC_BROWSER_TEST_F(PanelBrowserTest,
panel->Close();
}
+#if defined(OS_WIN)
+#define MAYBE_Accelerator Accelerator
+#else
+#define MAYBE_Accelerator DISABLED_Accelerator
+#endif
+IN_PROC_BROWSER_TEST_F(PanelBrowserTest, MAYBE_Accelerator) {
+ PanelManager* panel_manager = PanelManager::GetInstance();
+
+ // Create a test panel with web contents loaded.
+ CreatePanelParams params("1", gfx::Rect(), SHOW_AS_ACTIVE);
+ GURL url(ui_test_utils::GetTestUrl(
+ FilePath(kTestDir),
+ FilePath(FILE_PATH_LITERAL("update-preferred-size.html"))));
+ params.url = url;
+ Panel* panel = CreatePanelWithParams(params);
+ EXPECT_EQ(1, panel_manager->num_panels());
+
+ // Close the panel by acclerator.
+ content::WindowedNotificationObserver signal(
+ chrome::NOTIFICATION_PANEL_CLOSED,
+ content::Source<Panel>(panel));
+#if defined(USE_AURA)
+ content::NativeWebKeyboardEvent key_event(
+ ui::ET_KEY_PRESSED,
+ false,
+ ui::VKEY_W,
+ ui::EF_CONTROL_DOWN,
+ base::Time::Now().ToDoubleT());
+#elif defined(OS_WIN)
+ ::MSG key_msg = { NULL, WM_KEYDOWN, ui::VKEY_W, 0 };
+ content::NativeWebKeyboardEvent key_event(key_msg);
+ key_event.modifiers = content::NativeWebKeyboardEvent::ControlKey;
+#else
+ content::NativeWebKeyboardEvent key_event;
+#endif
+ panel->HandleKeyboardEvent(key_event);
+ signal.Wait();
+ EXPECT_EQ(0, panel_manager->num_panels());
+}
+
#endif // OS_WIN || OS_MACOSX
« no previous file with comments | « chrome/browser/ui/panels/panel.cc ('k') | chrome/browser/ui/panels/panel_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698