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

Unified Diff: chrome/browser/extensions/chrome_app_api_browsertest.cc

Issue 12066003: Remove TabStripModel wrapper use. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: gtk Created 7 years, 11 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/extensions/chrome_app_api_browsertest.cc
diff --git a/chrome/browser/extensions/chrome_app_api_browsertest.cc b/chrome/browser/extensions/chrome_app_api_browsertest.cc
index 1b4b00850b1e3b984b21a2b91e58fea3d19f5d5b..9e27d3b197d573caeea93e62224d22afdd467894 100644
--- a/chrome/browser/extensions/chrome_app_api_browsertest.cc
+++ b/chrome/browser/extensions/chrome_app_api_browsertest.cc
@@ -13,7 +13,7 @@
#include "chrome/browser/extensions/extension_service.h"
#include "chrome/browser/profiles/profile.h"
#include "chrome/browser/ui/browser.h"
-#include "chrome/browser/ui/browser_tabstrip.h"
+#include "chrome/browser/ui/tabs/tab_strip_model.h"
#include "chrome/common/chrome_switches.h"
#include "chrome/common/extensions/extension.h"
#include "chrome/common/extensions/manifest.h"
@@ -34,7 +34,7 @@ class ChromeAppAPITest : public ExtensionBrowserTest {
bool result;
CHECK(
content::ExecuteScriptInFrameAndExtractBool(
- chrome::GetActiveWebContents(browser()),
+ browser()->tab_strip_model()->GetActiveWebContents(),
frame_xpath,
kGetAppIsInstalled,
&result));
@@ -49,7 +49,7 @@ class ChromeAppAPITest : public ExtensionBrowserTest {
std::string result;
CHECK(
content::ExecuteScriptInFrameAndExtractString(
- chrome::GetActiveWebContents(browser()),
+ browser()->tab_strip_model()->GetActiveWebContents(),
frame_xpath,
kGetAppInstallState,
&result));
@@ -64,7 +64,7 @@ class ChromeAppAPITest : public ExtensionBrowserTest {
std::string result;
CHECK(
content::ExecuteScriptInFrameAndExtractString(
- chrome::GetActiveWebContents(browser()),
+ browser()->tab_strip_model()->GetActiveWebContents(),
frame_xpath,
kGetAppRunningState,
&result));
@@ -120,7 +120,7 @@ IN_PROC_BROWSER_TEST_F(ChromeAppAPITest, IsInstalled) {
std::string result;
ASSERT_TRUE(
content::ExecuteScriptAndExtractString(
- chrome::GetActiveWebContents(browser()),
+ browser()->tab_strip_model()->GetActiveWebContents(),
kGetAppDetails,
&result));
EXPECT_EQ("null", result);
@@ -134,7 +134,7 @@ IN_PROC_BROWSER_TEST_F(ChromeAppAPITest, IsInstalled) {
ui_test_utils::NavigateToURL(browser(), app_url);
ASSERT_TRUE(
content::ExecuteScriptAndExtractString(
- chrome::GetActiveWebContents(browser()),
+ browser()->tab_strip_model()->GetActiveWebContents(),
kGetAppDetails,
&result));
scoped_ptr<DictionaryValue> app_details(
@@ -148,7 +148,7 @@ IN_PROC_BROWSER_TEST_F(ChromeAppAPITest, IsInstalled) {
// that isInstalled should have the initial value.
ASSERT_TRUE(
content::ExecuteScriptAndExtractString(
- chrome::GetActiveWebContents(browser()),
+ browser()->tab_strip_model()->GetActiveWebContents(),
"window.domAutomationController.send("
" function() {"
" var value = window.chrome.app.isInstalled;"
@@ -198,7 +198,7 @@ IN_PROC_BROWSER_TEST_F(ChromeAppAPITest, GetDetailsForFrame) {
bool result = false;
ASSERT_TRUE(
content::ExecuteScriptAndExtractBool(
- chrome::GetActiveWebContents(browser()),
+ browser()->tab_strip_model()->GetActiveWebContents(),
kTestUnsuccessfulAccess,
&result));
EXPECT_TRUE(result);
@@ -212,7 +212,7 @@ IN_PROC_BROWSER_TEST_F(ChromeAppAPITest, GetDetailsForFrame) {
std::string json;
ASSERT_TRUE(
content::ExecuteScriptAndExtractString(
- chrome::GetActiveWebContents(browser()),
+ browser()->tab_strip_model()->GetActiveWebContents(),
kGetDetailsForFrame,
&json));

Powered by Google App Engine
This is Rietveld 408576698