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

Unified Diff: chrome/browser/metrics/desktop_session_duration/chrome_visibility_observer.h

Issue 2385103003: [DesktopSessionDurationTracker] Remove visibility-switch timeout from session length. (Closed)
Patch Set: Fix unittests. Created 4 years, 2 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/metrics/desktop_session_duration/chrome_visibility_observer.h
diff --git a/chrome/browser/metrics/desktop_session_duration/chrome_visibility_observer.h b/chrome/browser/metrics/desktop_session_duration/chrome_visibility_observer.h
index 7cf00d26eab5bc28c3392404a7efc4217a1fd00b..492736a9c87c274c8abe22fe61acd55423e55928 100644
--- a/chrome/browser/metrics/desktop_session_duration/chrome_visibility_observer.h
+++ b/chrome/browser/metrics/desktop_session_duration/chrome_visibility_observer.h
@@ -6,6 +6,7 @@
#define CHROME_BROWSER_METRICS_DESKTOP_SESSION_DURATION_CHROME_VISIBILITY_OBSERVER_H_
#include "base/memory/weak_ptr.h"
+#include "base/time/time.h"
#include "chrome/browser/ui/browser_list_observer.h"
namespace metrics {
@@ -18,7 +19,8 @@ class ChromeVisibilityObserver : public chrome::BrowserListObserver {
protected:
// Notifies |DesktopSessionDurationTracker| of visibility changes. Overridden
// by tests.
- virtual void SendVisibilityChangeEvent(bool active);
+ virtual void SendVisibilityChangeEvent(bool active,
+ base::TimeDelta time_ago);
private:
// Cancels visibility change in case when the browser becomes visible after a

Powered by Google App Engine
This is Rietveld 408576698