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

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

Issue 968943004: Move Android metrics files into a /metrics/ subfolder/package. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 5 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/android/java/src/org/chromium/chrome/browser/UmaUtils.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/UmaUtils.java b/chrome/android/java/src/org/chromium/chrome/browser/UmaUtils.java
index ff2e9d5c4fe5955b3709824c26a6a667705a356e..e4e2d1d398312459614751ee628b0227ada04aa3 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/UmaUtils.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/UmaUtils.java
@@ -4,51 +4,9 @@
package org.chromium.chrome.browser;
-import org.chromium.base.CalledByNative;
/**
- * Utilities to support startup metrics - Android version.
+ * TODO(yfriedman): Remove this after rolling downstream.
*/
-public class UmaUtils {
-
- private static long sApplicationStartWallClockMs;
-
- private static boolean sRunningApplicationStart;
-
- /**
- * Record the time at which the activity started. This should be called asap after
- * the start of the activity's onCreate function.
- */
- public static void recordMainEntryPointTime() {
- // We can't simply pass this down through a JNI call, since the JNI for chrome
- // isn't initialized until we start the native content browser component, and we
- // then need the start time in the C++ side before we return to Java. As such we
- // save it in a static that the C++ can fetch once it has initialized the JNI.
- sApplicationStartWallClockMs = System.currentTimeMillis();
- }
-
- /**
- * Whether the application is in the early stage since the browser process start. The
- * "application start" ends right after the last histogram related to browser startup is
- * recorded. Currently, the very first navigation commit in the lifetime of the process ends the
- * "application start".
- * Must only be called on the UI thread.
- */
- public static boolean isRunningApplicationStart() {
- return sRunningApplicationStart;
- }
-
- /**
- * Marks/unmarks the "application start" stage of the browser process lifetime.
- * Must only be called on the UI thread.
- */
- public static void setRunningApplicationStart(boolean isAppStart) {
- sRunningApplicationStart = isAppStart;
- }
-
- @CalledByNative
- private static long getMainEntryPointTime() {
- return sApplicationStartWallClockMs;
- }
-
+public class UmaUtils extends org.chromium.chrome.browser.metrics.UmaUtils {
}

Powered by Google App Engine
This is Rietveld 408576698