| Index: chrome/browser/ui/browser_navigator_browsertest.cc
|
| ===================================================================
|
| --- chrome/browser/ui/browser_navigator_browsertest.cc (revision 66453)
|
| +++ chrome/browser/ui/browser_navigator_browsertest.cc (working copy)
|
| @@ -7,6 +7,7 @@
|
| #include "chrome/browser/profile.h"
|
| #include "chrome/browser/tab_contents/tab_contents.h"
|
| #include "chrome/browser/tab_contents/tab_contents_view.h"
|
| +#include "chrome/browser/tab_contents_wrapper.h"
|
| #include "chrome/browser/tabs/tab_strip_model.h"
|
| #include "chrome/browser/ui/browser.h"
|
| #include "chrome/browser/ui/browser_navigator.h"
|
| @@ -38,12 +39,13 @@
|
| return browser;
|
| }
|
|
|
| - TabContents* CreateTabContents() {
|
| - return new TabContents(browser()->profile(),
|
| - NULL,
|
| - MSG_ROUTING_NONE,
|
| - browser()->GetSelectedTabContents(),
|
| - NULL);
|
| + TabContentsWrapper* CreateTabContents() {
|
| + return Browser::TabContentsFactory(
|
| + browser()->profile(),
|
| + NULL,
|
| + MSG_ROUTING_NONE,
|
| + browser()->GetSelectedTabContents(),
|
| + NULL);
|
| }
|
|
|
| void RunSuppressTest(WindowOpenDisposition disposition) {
|
| @@ -124,7 +126,7 @@
|
| p.disposition = NEW_FOREGROUND_TAB;
|
| browser::Navigate(&p);
|
| EXPECT_NE(old_contents, browser()->GetSelectedTabContents());
|
| - EXPECT_EQ(browser()->GetSelectedTabContents(), p.target_contents);
|
| + EXPECT_EQ(browser()->GetSelectedTabContentsWrapper(), p.target_contents);
|
| EXPECT_EQ(2, browser()->tab_count());
|
| }
|
|
|
| @@ -340,7 +342,7 @@
|
| // Navigate() should have opened the contents in a new foreground in the
|
| // current Browser.
|
| EXPECT_EQ(browser(), p.browser);
|
| - EXPECT_EQ(browser()->GetSelectedTabContents(), p.target_contents);
|
| + EXPECT_EQ(browser()->GetSelectedTabContentsWrapper(), p.target_contents);
|
|
|
| // We should have one window, with two tabs.
|
| EXPECT_EQ(1u, BrowserList::size());
|
| @@ -375,7 +377,7 @@
|
| // All platforms should respect size however provided width > 400 (Mac has a
|
| // minimum window width of 400).
|
| EXPECT_EQ(p.window_bounds.size(),
|
| - p.target_contents->view()->GetContainerSize());
|
| + p.target_contents->tab_contents()->view()->GetContainerSize());
|
|
|
| // We should have two windows, the new popup and the browser() provided by the
|
| // framework.
|
| @@ -400,7 +402,7 @@
|
| // Navigate() should have inserted a new tab at slot 0 in the tabstrip.
|
| EXPECT_EQ(browser(), p.browser);
|
| EXPECT_EQ(0, browser()->tabstrip_model()->GetIndexOfTabContents(
|
| - static_cast<const TabContents*>(p.target_contents)));
|
| + static_cast<const TabContentsWrapper*>(p.target_contents)));
|
|
|
| // We should have one window - the browser() provided by the framework.
|
| EXPECT_EQ(1u, BrowserList::size());
|
| @@ -421,7 +423,7 @@
|
| // Navigate() should have found browser() and create a new tab.
|
| EXPECT_EQ(browser(), p.browser);
|
| EXPECT_NE(old_contents, browser()->GetSelectedTabContents());
|
| - EXPECT_EQ(browser()->GetSelectedTabContents(), p.target_contents);
|
| + EXPECT_EQ(browser()->GetSelectedTabContentsWrapper(), p.target_contents);
|
| EXPECT_EQ(2, browser()->tab_count());
|
| }
|
|
|
| @@ -440,7 +442,7 @@
|
|
|
| // Navigate() should have found incognito, not browser().
|
| EXPECT_EQ(incognito, p.browser);
|
| - EXPECT_EQ(incognito->GetSelectedTabContents(), p.target_contents);
|
| + EXPECT_EQ(incognito->GetSelectedTabContentsWrapper(), p.target_contents);
|
| EXPECT_EQ(1, incognito->tab_count());
|
| }
|
|
|
|
|