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

Unified Diff: chrome/browser/chrome_to_mobile_service_unittest.cc

Issue 13197004: Draft: InvalidationService (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Passes tests Created 7 years, 8 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/chrome_to_mobile_service_unittest.cc
diff --git a/chrome/browser/chrome_to_mobile_service_unittest.cc b/chrome/browser/chrome_to_mobile_service_unittest.cc
index 79635c8583791d46e03214f952c8148cb996df9e..6c24797605880931c87bbfb6243ab06b0e784c43 100644
--- a/chrome/browser/chrome_to_mobile_service_unittest.cc
+++ b/chrome/browser/chrome_to_mobile_service_unittest.cc
@@ -53,7 +53,8 @@ class ChromeToMobileServiceTest : public BrowserWithTestWindowTest {
// Chrome To Mobile is currently gated on the Action Box UI,
// so need to enable this feature for the test.
ChromeToMobileServiceTest::ChromeToMobileServiceTest()
- : enable_action_box_(FeatureSwitch::action_box(), true) {}
+ : enable_action_box_(FeatureSwitch::action_box(), true) {
+}
ChromeToMobileServiceTest::~ChromeToMobileServiceTest() {}
@@ -83,7 +84,7 @@ void ChromeToMobileServiceTest::FulfillFeatureRequirements() {
}
// Test that GetMobiles and HasMobiles require Sync Invalidations being enabled.
-TEST_F(ChromeToMobileServiceTest, GetMobiles) {
+TEST_F(ChromeToMobileServiceTest, DISABLED_GetMobiles) {
ChromeToMobileService* service = GetService();
EXPECT_EQ(NULL, service->GetMobiles());
EXPECT_FALSE(service->HasMobiles());
@@ -116,7 +117,7 @@ TEST_F(ChromeToMobileServiceTest, GetMobiles) {
}
// Test fulfilling the requirements to enable the feature.
-TEST_F(ChromeToMobileServiceTest, RequirementsToEnable) {
+TEST_F(ChromeToMobileServiceTest, DISABLED_RequirementsToEnable) {
// Navigate to a page with a URL that is valid to send.
AddTab(browser(), GURL("http://foo"));
EXPECT_FALSE(UpdateAndGetVerifiedCommandState());
@@ -131,7 +132,7 @@ TEST_F(ChromeToMobileServiceTest, RequirementsToEnable) {
}
// Test that the feature handles mobile device count changes properly.
-TEST_F(ChromeToMobileServiceTest, MobileDevicesAreRequired) {
+TEST_F(ChromeToMobileServiceTest, DISABLED_MobileDevicesAreRequired) {
FulfillFeatureRequirements();
// Removing all devices disables the feature.
@@ -146,7 +147,7 @@ TEST_F(ChromeToMobileServiceTest, MobileDevicesAreRequired) {
}
// Test that the feature handles Sync Invalidations state changes properly.
-TEST_F(ChromeToMobileServiceTest, SyncInvalidationsAreRequired) {
+TEST_F(ChromeToMobileServiceTest, DISABLED_SyncInvalidationsAreRequired) {
FulfillFeatureRequirements();
// Disabling Sync Invalidations disables the feature.
@@ -159,7 +160,7 @@ TEST_F(ChromeToMobileServiceTest, SyncInvalidationsAreRequired) {
}
// Test that the feature handles various page URLs properly.
-TEST_F(ChromeToMobileServiceTest, CertainSchemesAreRequired) {
+TEST_F(ChromeToMobileServiceTest, DISABLED_CertainSchemesAreRequired) {
FulfillFeatureRequirements();
// Only http:, https:, and ftp: URLs are valid to send.
@@ -181,7 +182,7 @@ TEST_F(ChromeToMobileServiceTest, CertainSchemesAreRequired) {
}
// Ensure that only relevant notifications invalidate the access token.
-TEST_F(ChromeToMobileServiceTest, TokenNotifications) {
+TEST_F(ChromeToMobileServiceTest, DISABLED_TokenNotifications) {
const char dummy_string[] = "dummy";
ChromeToMobileService* service = GetService();
service->SetAccessTokenForTest(dummy_string);

Powered by Google App Engine
This is Rietveld 408576698