From 969ff790df011552be3e8bf70dacd4bf2bd10aa5 Mon Sep 17 00:00:00 2001 From: "finnur@google.com" Date: Thu, 28 Aug 2008 22:11:13 +0000 Subject: We should not EXPECT to get a browser object or a window object, we should ASSERT that we got it (which ends the test on NULL pointers). Otherwise we continue on and crash when we try to use these pointers, such as when passing it into GetBrowserForWindow. git-svn-id: svn://svn.chromium.org/chrome/trunk/src@1508 0039d316-1c4b-4281-b951-d872f2087c98 --- chrome/browser/browser_focus_uitest.cc | 22 ++++++++++++++-------- 1 file changed, 14 insertions(+), 8 deletions(-) (limited to 'chrome/browser/browser_focus_uitest.cc') diff --git a/chrome/browser/browser_focus_uitest.cc b/chrome/browser/browser_focus_uitest.cc index fe83ed5..4b7b39a 100644 --- a/chrome/browser/browser_focus_uitest.cc +++ b/chrome/browser/browser_focus_uitest.cc @@ -64,10 +64,11 @@ TEST_F(BrowserFocusTest, BrowsersRememberFocus) { // The focus should be on the Tab contents. scoped_ptr window(automation()->GetActiveWindow()); - EXPECT_TRUE(window.get() != NULL); + ASSERT_TRUE(window.get() != NULL); scoped_ptr browser(automation()-> GetBrowserForWindow(window.get())); + ASSERT_TRUE(browser.get() != NULL); int focused_view_id; EXPECT_TRUE(window->GetFocusedViewID(&focused_view_id)); @@ -99,10 +100,10 @@ TEST_F(BrowserFocusTest, BrowsersRememberFocus) { // Open a new browser window. EXPECT_TRUE(automation()->OpenNewBrowserWindow(SW_SHOWNORMAL)); scoped_ptr new_window(automation()->GetActiveWindow()); - EXPECT_TRUE(new_window.get() != NULL); + ASSERT_TRUE(new_window.get() != NULL); scoped_ptr new_browser(automation()-> GetBrowserForWindow(new_window.get())); - EXPECT_TRUE(new_browser.get() != NULL); + ASSERT_TRUE(new_browser.get() != NULL); // Let's make sure we have 2 different browser windows. EXPECT_TRUE(browser->handle() != new_browser->handle()); @@ -132,9 +133,10 @@ TEST_F(BrowserFocusTest, TabsRememberFocus) { TestServer server(kDocRoot); scoped_ptr window(automation()->GetActiveWindow()); - EXPECT_TRUE(window.get() != NULL); + ASSERT_TRUE(window.get() != NULL); scoped_ptr browser( automation()->GetBrowserForWindow(window.get())); + ASSERT_TRUE(browser.get() != NULL); // First we navigate to our test page. GURL url = server.TestServerPageW(kSimplePage); @@ -200,16 +202,18 @@ TEST_F(BrowserFocusTest, BackgroundBrowserDontStealFocus) { tab->NavigateToURL(simple_page_url); scoped_ptr window(automation()->GetActiveWindow()); - EXPECT_TRUE(window.get() != NULL); + ASSERT_TRUE(window.get() != NULL); scoped_ptr browser( automation()->GetBrowserForWindow(window.get())); + ASSERT_TRUE(browser.get() != NULL); // Open a new browser window. EXPECT_TRUE(automation()->OpenNewBrowserWindow(SW_SHOWNORMAL)); scoped_ptr new_window(automation()->GetActiveWindow()); - EXPECT_TRUE(window.get() != NULL); + ASSERT_TRUE(window.get() != NULL); scoped_ptr new_browser( automation()->GetBrowserForWindow(new_window.get())); + ASSERT_TRUE(new_browser.get() != NULL); GURL steal_focus_url = server.TestServerPageW(kStealFocusPage); new_browser->AppendTab(steal_focus_url); @@ -236,9 +240,10 @@ TEST_F(BrowserFocusTest, LocationBarLockFocus) { tab->NavigateToURL(url); scoped_ptr window(automation()->GetActiveWindow()); - EXPECT_TRUE(window.get() != NULL); + ASSERT_TRUE(window.get() != NULL); scoped_ptr browser( automation()->GetBrowserForWindow(window.get())); + ASSERT_TRUE(browser.get() != NULL); // Click on the location bar. gfx::Rect bounds; @@ -267,9 +272,10 @@ TEST_F(BrowserFocusTest, FocusTraversal) { tab->NavigateToURL(url); scoped_ptr window(automation()->GetActiveWindow()); - EXPECT_TRUE(window.get() != NULL); + ASSERT_TRUE(window.get() != NULL); scoped_ptr browser( automation()->GetBrowserForWindow(window.get())); + ASSERT_TRUE(browser.get() != NULL); // Click on the location bar. gfx::Rect bounds; -- cgit v1.1