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

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

Issue 949613002: [Android] Add animation frame time histogram class. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Cache listener, off-load some initializing from startRecording to constructor 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/appmenu/AppMenu.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/appmenu/AppMenu.java b/chrome/android/java/src/org/chromium/chrome/browser/appmenu/AppMenu.java
index e4aa4b7417ea1a223ad2e0dc90dbd5b6de175f21..029bc97693080329ecb9ed915e6d5dd228ae5dbc 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/appmenu/AppMenu.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/appmenu/AppMenu.java
@@ -5,6 +5,7 @@
package org.chromium.chrome.browser.appmenu;
import android.animation.Animator;
+import android.animation.Animator.AnimatorListener;
import android.animation.AnimatorSet;
import android.content.Context;
import android.content.res.Resources;
@@ -25,6 +26,7 @@ import android.widget.ListPopupWindow;
import android.widget.PopupWindow;
import android.widget.PopupWindow.OnDismissListener;
+import org.chromium.base.AnimationFrameTimeHistogram;
import org.chromium.base.SysUtils;
import org.chromium.chrome.R;
@@ -51,6 +53,8 @@ public class AppMenu implements OnItemClickListener, OnKeyListener {
private AppMenuHandler mHandler;
private int mCurrentScreenRotation = -1;
private boolean mIsByHardwareButton;
+ private AnimatorListener mAnimationHistogramRecorder = AnimationFrameTimeHistogram
+ .getAnimatorRecorder("WrenchMenu.OpeningAnimationFrameTimes");
/**
* Creates and sets up the App Menu.
@@ -336,6 +340,7 @@ public class AppMenu implements OnItemClickListener, OnKeyListener {
}
}
+ animation.addListener(mAnimationHistogramRecorder);
animation.start();
}
}

Powered by Google App Engine
This is Rietveld 408576698