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

Unified Diff: chrome/browser/sync/test/integration/two_client_arc_package_sync_test.cc

Issue 2711033002: Fix Arc integration test. (Closed)
Patch Set: Rebase Created 3 years, 10 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/sync/test/integration/two_client_arc_package_sync_test.cc
diff --git a/chrome/browser/sync/test/integration/two_client_arc_package_sync_test.cc b/chrome/browser/sync/test/integration/two_client_arc_package_sync_test.cc
index ef5a1c8b50b6ad02513e6327358e45955d0f7d3e..9891ec11540519baf7212e34bb8cb6d54b6459a6 100644
--- a/chrome/browser/sync/test/integration/two_client_arc_package_sync_test.cc
+++ b/chrome/browser/sync/test/integration/two_client_arc_package_sync_test.cc
@@ -6,6 +6,7 @@
#include "chrome/browser/sync/test/integration/sync_arc_package_helper.h"
#include "chrome/browser/sync/test/integration/sync_integration_test_util.h"
#include "chrome/browser/sync/test/integration/sync_test.h"
+#include "chrome/browser/ui/app_list/arc/arc_app_list_prefs_factory.h"
namespace arc {
@@ -37,8 +38,13 @@ class TwoClientArcPackageSyncTest : public SyncTest {
return sync_helper_ != nullptr;
}
+ void SetUpOnMainThread() override {
+ // This setting does not affect the profile created by InProcessBrowserTest.
+ // Only sync test profiles are affected.
+ ArcAppListPrefsFactory::SetFactoryForSyncTest();
+ }
+
void TearDownOnMainThread() override {
- sync_helper_->CleanUp();
sync_helper_ = nullptr;
SyncTest::TearDownOnMainThread();
}
@@ -85,7 +91,7 @@ IN_PROC_BROWSER_TEST_F(TwoClientArcPackageSyncTest,
ASSERT_FALSE(AllProfilesHaveSameArcPackageDetails());
ASSERT_TRUE(SetupSync());
-
+ ASSERT_TRUE(AwaitQuiescence());
ASSERT_TRUE(AllProfilesHaveSameArcPackageDetails());
}
@@ -108,7 +114,7 @@ IN_PROC_BROWSER_TEST_F(TwoClientArcPackageSyncTest,
ASSERT_FALSE(AllProfilesHaveSameArcPackageDetails());
ASSERT_TRUE(SetupSync());
-
+ ASSERT_TRUE(AwaitQuiescence());
EXPECT_TRUE(AllProfilesHaveSameArcPackageDetails());
}
@@ -128,7 +134,7 @@ IN_PROC_BROWSER_TEST_F(TwoClientArcPackageSyncTest,
EXPECT_FALSE(AllProfilesHaveSameArcPackageDetails());
ASSERT_TRUE(SetupSync());
-
+ ASSERT_TRUE(AwaitQuiescence());
EXPECT_TRUE(AllProfilesHaveSameArcPackageDetails());
}

Powered by Google App Engine
This is Rietveld 408576698