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

Unified Diff: chrome/browser/chromeos/arc/arc_util.h

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/chromeos/arc/arc_util.h
diff --git a/chrome/browser/chromeos/arc/arc_util.h b/chrome/browser/chromeos/arc/arc_util.h
index bde8bbcccb9790e32ca1b3d2d55dd37595f61e2b..deaf62966fbdef31b57ed55ec95ae0e2bd4ab7f2 100644
--- a/chrome/browser/chromeos/arc/arc_util.h
+++ b/chrome/browser/chromeos/arc/arc_util.h
@@ -31,6 +31,12 @@ bool IsArcAllowedForProfile(const Profile* profile);
// case.
void DisallowArcForTesting();
+// Sets ARC as allowed for sync test.
+// In sync test, mutilpe ArcAppListPrefs are created and can confuse
+// ArcSessionManager. Let IsArcAllowedForProfile() return true to stop confusing
+// ArcSessionManager.
+void SetArcAllowedForSyncTesting();
hidehiko 2017/02/24 18:22:26 I commented alternative approach, which this may n
lgcheng 2017/02/25 01:40:38 Done.
+
// Returns whether the user has opted in (or is opting in now) to use Google
// Play Store on ARC.
// This is almost equivalent to the value of "arc.enabled" preference. However,
« no previous file with comments | « no previous file | chrome/browser/chromeos/arc/arc_util.cc » ('j') | chrome/browser/sync/test/integration/sync_test.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698