diff options
| author | mazda@chromium.org <mazda@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-01-07 22:34:38 +0000 |
|---|---|---|
| committer | mazda@chromium.org <mazda@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-01-07 22:34:38 +0000 |
| commit | 0570d9bfc8db4e3e1e8e124d0f16869b59e8deed (patch) | |
| tree | 2789039230ec3dc2a117b4175340ca26c544d5d9 /ash | |
| parent | c297d3a13ae36bf2f574e55b3f1f9494b3f98e7b (diff) | |
| download | chromium_src-0570d9bfc8db4e3e1e8e124d0f16869b59e8deed.zip chromium_src-0570d9bfc8db4e3e1e8e124d0f16869b59e8deed.tar.gz chromium_src-0570d9bfc8db4e3e1e8e124d0f16869b59e8deed.tar.bz2 | |
Fix a leak in KeyboardOverlayViewTest.OpenAcceleratorsClose.
BUG=167165
Review URL: https://codereview.chromium.org/11648037
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@175400 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'ash')
| -rw-r--r-- | ash/keyboard_overlay/keyboard_overlay_view_unittest.cc | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/ash/keyboard_overlay/keyboard_overlay_view_unittest.cc b/ash/keyboard_overlay/keyboard_overlay_view_unittest.cc index fcb82ce..87afe05 100644 --- a/ash/keyboard_overlay/keyboard_overlay_view_unittest.cc +++ b/ash/keyboard_overlay/keyboard_overlay_view_unittest.cc @@ -24,9 +24,10 @@ bool operator==(const KeyboardOverlayView::KeyEventData& lhs, // Verifies that the accelerators that open the keyboard overlay close it. TEST_F(KeyboardOverlayViewTest, OpenAcceleratorsClose) { - KeyboardOverlayView* view = new KeyboardOverlayView( + ui::test::TestWebDialogDelegate delegate(GURL("chrome://keyboardoverlay")); + KeyboardOverlayView view( Shell::GetInstance()->browser_context(), - new ui::test::TestWebDialogDelegate(GURL("chrome://keyboardoverlay")), + &delegate, new ui::test::TestWebContentsHandler); for (size_t i = 0; i < kAcceleratorDataLength; ++i) { if (kAcceleratorData[i].action != SHOW_KEYBOARD_OVERLAY) @@ -37,7 +38,7 @@ TEST_F(KeyboardOverlayViewTest, OpenAcceleratorsClose) { open_key_data.keycode, open_key_data.modifiers, false); - EXPECT_TRUE(view->IsCancelingKeyEvent(&open_key)); + EXPECT_TRUE(view.IsCancelingKeyEvent(&open_key)); } } |