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

Unified Diff: chrome/browser/ui/ash/launcher/arc_app_window_launcher_controller.h

Issue 2922383002: [Merge M60] arc: Provide metric to monitor Play Store window is shown time. (Closed)
Patch Set: Created 3 years, 6 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/ash/launcher/arc_app_window_launcher_controller.h
diff --git a/chrome/browser/ui/ash/launcher/arc_app_window_launcher_controller.h b/chrome/browser/ui/ash/launcher/arc_app_window_launcher_controller.h
index 7206c2387d15e124f6337addd808a545fa8f3f7e..3a14af5e6926aabcfa35ab33fa1af9fdad5eda90 100644
--- a/chrome/browser/ui/ash/launcher/arc_app_window_launcher_controller.h
+++ b/chrome/browser/ui/ash/launcher/arc_app_window_launcher_controller.h
@@ -11,6 +11,8 @@
#include "ash/shell_observer.h"
#include "base/macros.h"
+#include "base/time/time.h"
+#include "chrome/browser/chromeos/arc/arc_session_manager.h"
#include "chrome/browser/ui/app_list/arc/arc_app_list_prefs.h"
#include "chrome/browser/ui/ash/launcher/app_window_launcher_controller.h"
#include "chrome/browser/ui/ash/launcher/arc_app_shelf_id.h"
@@ -32,7 +34,8 @@ class ArcAppWindowLauncherController : public AppWindowLauncherController,
public aura::EnvObserver,
public aura::WindowObserver,
public ash::ShellObserver,
- public ArcAppListPrefs::Observer {
+ public ArcAppListPrefs::Observer,
+ public arc::ArcSessionManager::Observer {
public:
explicit ArcAppWindowLauncherController(ChromeLauncherController* owner);
~ArcAppWindowLauncherController() override;
@@ -113,6 +116,10 @@ class ArcAppWindowLauncherController : public AppWindowLauncherController,
AppWindowLauncherItemController* ControllerForWindow(
aura::Window* window) override;
+ // arc::ArcSessionManager::Observer:
+ void OnArcOptInManagementCheckStarted() override;
+ void OnArcSessionStopped(arc::ArcStopReason stop_reason) override;
+
int active_task_id_ = -1;
TaskIdToAppWindowInfo task_id_to_app_window_info_;
ShelfGroupToAppControllerMap app_shelf_group_to_controller_map_;
@@ -120,6 +127,12 @@ class ArcAppWindowLauncherController : public AppWindowLauncherController,
Profile* observed_profile_ = nullptr;
bool observing_shell_ = false;
+ // The time when the ARC OptIn management check was started. This happens
+ // right after user agrees the ToS or in some cases for managed user when ARC
+ // starts for the first time. OptIn management check is preceding step before
+ // ARC container is actually started.
+ base::Time opt_in_management_check_start_time_;
+
DISALLOW_COPY_AND_ASSIGN(ArcAppWindowLauncherController);
};
« no previous file with comments | « chrome/browser/chromeos/arc/arc_optin_uma.cc ('k') | chrome/browser/ui/ash/launcher/arc_app_window_launcher_controller.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698