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

Unified Diff: chrome/browser/ui/app_list/arc/arc_app_test.h

Issue 2106663004: arc: Initial implemetation of Chrome sync for Arc packages. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Wrap bridge state related method in unit_tests. Created 4 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/ui/app_list/arc/arc_app_test.h
diff --git a/chrome/browser/ui/app_list/arc/arc_app_test.h b/chrome/browser/ui/app_list/arc/arc_app_test.h
index ad079d1ac7608699438653871007d943417e0a9f..3d05c097616f54bd7d01f110932a4f1935e25ada 100644
--- a/chrome/browser/ui/app_list/arc/arc_app_test.h
+++ b/chrome/browser/ui/app_list/arc/arc_app_test.h
@@ -43,6 +43,11 @@ class ArcAppTest {
void SetUp(Profile* profile);
void TearDown();
+ // Public methods to set bridge status for unit_tests.
+ void SetBridgeServiceReady();
+ void SetBridgeServiceStopped();
+ void ResetBridgeServiceReady();
+
static std::string GetAppId(const arc::mojom::AppInfo& app_info);
static std::string GetAppId(const arc::mojom::ShortcutInfo& shortcut);
@@ -77,6 +82,10 @@ class ArcAppTest {
const user_manager::User* CreateUserAndLogin();
bool FindPackage(const arc::mojom::ArcPackageInfo& package);
+ // AppInstance is destoryed after bridge stops and InstanceHolder::
+ // CloseChannel(). Reset AppInstance to continue unit_tests.
+ void ResetAppInstance();
+
// Unowned pointer.
Profile* profile_ = nullptr;

Powered by Google App Engine
This is Rietveld 408576698