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

Unified Diff: chrome/browser/ui/browser_browsertest.cc

Issue 10917026: Switch Extensions::TabHelper to use WebContents, WebContentsUserData. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase only Created 8 years, 3 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
Index: chrome/browser/ui/browser_browsertest.cc
diff --git a/chrome/browser/ui/browser_browsertest.cc b/chrome/browser/ui/browser_browsertest.cc
index c0f6a9f3f20f756e6ecbd2cd0fd0327c683ef5a5..d44901a9ae65f7e0c13a78d698be9f41dd5c2727 100644
--- a/chrome/browser/ui/browser_browsertest.cc
+++ b/chrome/browser/ui/browser_browsertest.cc
@@ -800,7 +800,9 @@ IN_PROC_BROWSER_TEST_F(BrowserTest, TabClosingWhenRemovingExtension) {
TabContents* app_contents = chrome::TabContentsFactory(
browser()->profile(), NULL, MSG_ROUTING_NONE, NULL);
- app_contents->extension_tab_helper()->SetExtensionApp(extension_app);
+ extensions::TabHelper* extensions_tab_helper =
+ extensions::TabHelper::FromWebContents(app_contents->web_contents());
+ extensions_tab_helper->SetExtensionApp(extension_app);
model->AddTabContents(app_contents, 0, content::PageTransitionFromInt(0),
TabStripModel::ADD_NONE);
@@ -919,7 +921,9 @@ IN_PROC_BROWSER_TEST_F(BrowserTest, RestorePinnedTabs) {
ui_test_utils::NavigateToURL(browser(), url);
TabContents* app_contents = chrome::TabContentsFactory(
browser()->profile(), NULL, MSG_ROUTING_NONE, NULL);
- app_contents->extension_tab_helper()->SetExtensionApp(extension_app);
+ extensions::TabHelper* extensions_tab_helper =
+ extensions::TabHelper::FromWebContents(app_contents->web_contents());
+ extensions_tab_helper->SetExtensionApp(extension_app);
model->AddTabContents(app_contents, 0, content::PageTransitionFromInt(0),
TabStripModel::ADD_NONE);
model->SetTabPinned(0, true);
@@ -975,8 +979,9 @@ IN_PROC_BROWSER_TEST_F(BrowserTest, RestorePinnedTabs) {
new_model->GetTabContentsAt(2)->web_contents()->GetURL());
EXPECT_TRUE(
- new_model->GetTabContentsAt(0)->extension_tab_helper()->extension_app() ==
- extension_app);
+ extensions::TabHelper::FromWebContents(
+ new_model->GetTabContentsAt(0)->web_contents())->
+ extension_app() == extension_app);
}
#endif // !defined(OS_CHROMEOS)
@@ -1007,10 +1012,11 @@ IN_PROC_BROWSER_TEST_F(BrowserTest, OpenAppWindowLikeNtp) {
NEW_WINDOW));
ASSERT_TRUE(app_window);
- // Apps launched in a window from the NTP do not have extension_app set in
- // tab contents.
- TabContents* tab_contents = TabContents::FromWebContents(app_window);
- EXPECT_FALSE(tab_contents->extension_tab_helper()->extension_app());
+ // Apps launched in a window from the NTP have an extensions tab helper but
+ // do not have extension_app set in it.
+ ASSERT_TRUE(extensions::TabHelper::FromWebContents(app_window));
+ EXPECT_FALSE(
+ extensions::TabHelper::FromWebContents(app_window)->extension_app());
EXPECT_EQ(extension_app->GetFullLaunchURL(), app_window->GetURL());
// The launch should have created a new browser.
« no previous file with comments | « chrome/browser/ui/ash/launcher/browser_launcher_item_controller.cc ('k') | chrome/browser/ui/browser_commands.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698