diff options
author | estade@chromium.org <estade@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-08-14 01:11:14 +0000 |
---|---|---|
committer | estade@chromium.org <estade@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-08-14 01:11:14 +0000 |
commit | b410bc3a7509a653af93fdc1d4466bfc1b3f9cfd (patch) | |
tree | c5f137fba9fbf234f9bcc2b4eec81937ee904549 /chrome/browser/views/tabs | |
parent | a97cca4ae68dc9f34ece5ff97de5675721cf88a1 (diff) | |
download | chromium_src-b410bc3a7509a653af93fdc1d4466bfc1b3f9cfd.zip chromium_src-b410bc3a7509a653af93fdc1d4466bfc1b3f9cfd.tar.gz chromium_src-b410bc3a7509a653af93fdc1d4466bfc1b3f9cfd.tar.bz2 |
Revert some of my automation provider changes. Maybe it's possible that GetResource() returns NULL when ContainsHandle() returns true? I can't repro locally and there is no try server for this.
Review URL: http://codereview.chromium.org/165510
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23404 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/views/tabs')
-rw-r--r-- | chrome/browser/views/tabs/tab_dragging_test.cc | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/chrome/browser/views/tabs/tab_dragging_test.cc b/chrome/browser/views/tabs/tab_dragging_test.cc index e8baf39..e187e71 100644 --- a/chrome/browser/views/tabs/tab_dragging_test.cc +++ b/chrome/browser/views/tabs/tab_dragging_test.cc @@ -346,8 +346,7 @@ TEST_F(TabDraggingTest, Tab1Tab3Escape) { } // Drag Tab_2 out of the Tab strip. A new window should open with this tab. -// TODO (sidchat, estade): Fix this asap (Aug 13 2009) -TEST_F(TabDraggingTest, DISABLED_Tab2OutOfTabStrip) { +TEST_F(TabDraggingTest, Tab2OutOfTabStrip) { scoped_refptr<BrowserProxy> browser(automation()->GetBrowserWindow(0)); ASSERT_TRUE(browser.get()); scoped_refptr<WindowProxy> window(browser->GetWindow()); |