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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/metrics/MainIntentBehaviorMetrics.java

Issue 2779083003: Add additional metrics for main intent behaviors. (Closed)
Patch Set: Address isherman@ comments Created 3 years, 9 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/android/java/src/org/chromium/chrome/browser/metrics/MainIntentBehaviorMetrics.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/metrics/MainIntentBehaviorMetrics.java b/chrome/android/java/src/org/chromium/chrome/browser/metrics/MainIntentBehaviorMetrics.java
index 3b93b4743348e640699ff15d2c0a4e14625d4b33..8feb225fd05542755cb04b32c78f57a3c911bcd1 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/metrics/MainIntentBehaviorMetrics.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/metrics/MainIntentBehaviorMetrics.java
@@ -29,6 +29,11 @@ import java.util.concurrent.TimeUnit;
*/
public class MainIntentBehaviorMetrics implements ApplicationStatus.ActivityStateListener {
+ private static final long BACKGROUND_TIME_24_HOUR_MS = 86400000;
+ private static final long BACKGROUND_TIME_12_HOUR_MS = 43200000;
+ private static final long BACKGROUND_TIME_6_HOUR_MS = 21600000;
+ private static final long BACKGROUND_TIME_1_HOUR_MS = 3600000;
+
private static final long TIMEOUT_DURATION_MS = 10000;
@Retention(RetentionPolicy.SOURCE)
@@ -82,6 +87,16 @@ public class MainIntentBehaviorMetrics implements ApplicationStatus.ActivityStat
public void onMainIntentWithNative(long backgroundDurationMs) {
RecordUserAction.record("MobileStartup.MainIntentReceived");
+ if (backgroundDurationMs >= BACKGROUND_TIME_24_HOUR_MS) {
+ RecordUserAction.record("MobileStartup.MainIntentReceived.After24Hours");
+ } else if (backgroundDurationMs >= BACKGROUND_TIME_12_HOUR_MS) {
+ RecordUserAction.record("MobileStartup.MainIntentReceived.After12Hours");
+ } else if (backgroundDurationMs >= BACKGROUND_TIME_6_HOUR_MS) {
+ RecordUserAction.record("MobileStartup.MainIntentReceived.After6Hours");
+ } else if (backgroundDurationMs >= BACKGROUND_TIME_1_HOUR_MS) {
+ RecordUserAction.record("MobileStartup.MainIntentReceived.After1Hour");
+ }
+
if (mPendingActionRecordForMainIntent) return;
mBackgroundDurationMs = backgroundDurationMs;
« no previous file with comments | « chrome/android/java/src/org/chromium/chrome/browser/ChromeTabbedActivity.java ('k') | tools/metrics/actions/actions.xml » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698