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

Unified Diff: chrome/browser/sync/test/integration/themes_helper.h

Issue 395503004: sync: Refactor themes sync integration tests (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: More comments Created 6 years, 5 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/themes_helper.h
diff --git a/chrome/browser/sync/test/integration/themes_helper.h b/chrome/browser/sync/test/integration/themes_helper.h
index d0c2f155dc4dc7b8b747463a88743e131cb048e7..9911de1ba23a80496639fbf33486b6f6245a8535 100644
--- a/chrome/browser/sync/test/integration/themes_helper.h
+++ b/chrome/browser/sync/test/integration/themes_helper.h
@@ -35,11 +35,6 @@ bool UsingSystemTheme(Profile* profile) WARN_UNUSED_RESULT;
bool ThemeIsPendingInstall(
Profile* profile, const std::string& id) WARN_UNUSED_RESULT;
-// Returns true iff |profile|'s current theme is the given
-// custom theme or if the given theme is pending install.
-bool HasOrWillHaveCustomTheme(
- Profile* profile, const std::string& id) WARN_UNUSED_RESULT;
-
// Sets |profile| to use the custom theme with the given index.
void UseCustomTheme(Profile* profile, int index);
@@ -49,6 +44,18 @@ void UseDefaultTheme(Profile* profile);
// Sets |profile| to use the system theme.
void UseSystemTheme(Profile* profile);
+// Waits until |theme| is pending for install on |profile|.
+// Returns false in case of timeout.
+bool AwaitThemeIsPendingInstall(Profile* profile, const std::string& theme);
+
+// Waits until |profile| is using the system theme.
+// Returns false in case of timeout.
+bool AwaitUsingSystemTheme(Profile* profile);
+
+// Waits until |profile| is using the default theme.
+// Returns false in case of timeout.
+bool AwaitUsingDefaultTheme(Profile* profile);
+
} // namespace themes_helper
#endif // CHROME_BROWSER_SYNC_TEST_INTEGRATION_THEMES_HELPER_H_

Powered by Google App Engine
This is Rietveld 408576698