[Android] Measure AppMenu animation jankiness by frame times UMA.

chrome/android/java/src/org/chromium/chrome/browser/appmenu/AppMenu.java

 // Copyright 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 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;
 import android.graphics.Rect;
 import android.graphics.drawable.Drawable;
 import android.os.Build;
 import android.view.KeyEvent;
 import android.view.LayoutInflater;
 import android.view.Menu;
 import android.view.MenuItem;
 import android.view.Surface;
 import android.view.View;
 import android.view.View.OnKeyListener;
 import android.view.ViewGroup;
 import android.widget.AdapterView;
 import android.widget.AdapterView.OnItemClickListener;
 import android.widget.ImageButton;
 import android.widget.ListPopupWindow;
 import android.widget.ListView;
 import android.widget.PopupWindow;
 import android.widget.PopupWindow.OnDismissListener;
 
+import org.chromium.base.AnimationFrameTimeHistogram;
 import org.chromium.base.SysUtils;
 import org.chromium.chrome.R;
 
 import java.util.ArrayList;
 import java.util.List;
 
 /**
  * Shows a popup of menuitems anchored to a host view. When a item is selected we call
  * Activity.onOptionsItemSelected with the appropriate MenuItem.
  *   - Only visible MenuItems are shown.
  *   - Disabled items are grayed out.
  */
 public class AppMenu implements OnItemClickListener, OnKeyListener {
 
     private static final float LAST_ITEM_SHOW_FRACTION = 0.5f;
 
     private final Menu mMenu;
     private final int mItemRowHeight;
     private final int mItemDividerHeight;
     private final int mVerticalFadeDistance;
     private final int mNegativeSoftwareVerticalOffset;
     private ListPopupWindow mPopup;
     private AppMenuAdapter mAdapter;
     private AppMenuHandler mHandler;
     private int mCurrentScreenRotation = -1;
     private boolean mIsByHardwareButton;
+    private AnimatorSet mMenuItemEnterAnimator;
+    private AnimatorListener mAnimationHistogramRecorder = AnimationFrameTimeHistogram
+            .getAnimatorRecorder("WrenchMenu.OpeningAnimationFrameTimes");
 
     /**
      * Creates and sets up the App Menu.
      * @param menu Original menu created by the framework.
      * @param itemRowHeight Desired height for each app menu row.
      * @param itemDividerHeight Desired height for the divider between app menu items.
      * @param handler AppMenuHandler receives callbacks from AppMenu.
      * @param res Resources object used to get dimensions and style attributes.
      */
     AppMenu(Menu menu, int itemRowHeight, int itemDividerHeight, AppMenuHandler handler,
@@ skipping 67 matching lines @@
         mPopup = new ListPopupWindow(context, null, android.R.attr.popupMenuStyle);
         mPopup.setModal(true);
         mPopup.setAnchorView(anchorView);
         mPopup.setInputMethodMode(PopupWindow.INPUT_METHOD_NOT_NEEDED);
         mPopup.setOnDismissListener(new OnDismissListener() {
             @Override
             public void onDismiss() {
                 if (mPopup.getAnchorView() instanceof ImageButton) {
                     ((ImageButton) mPopup.getAnchorView()).setSelected(false);
                 }
+
+                if (mMenuItemEnterAnimator != null) mMenuItemEnterAnimator.cancel();
+
+                mHandler.appMenuDismissed();
                 mHandler.onMenuVisibilityChanged(false);
             }
         });
 
         // Some OEMs don't actually let us change the background... but they still return the
         // padding of the new background, which breaks the menu height.  If we still have a
         // drawable here even though our style says @null we should use this padding instead...
         Drawable originalBgDrawable = mPopup.getBackground();
 
         // Need to explicitly set the background here.  Relying on it being set in the style caused
@@ skipping 139 matching lines @@
                 }
             }
         }
         return false;
     }
 
     /**
      * Dismisses the app menu and cancels the drag-to-scroll if it is taking place.
      */
     void dismiss() {
-        mHandler.appMenuDismissed();
         if (isShowing()) {
             mPopup.dismiss();
         }
     }
 
     /**
      * @return Whether the app menu is currently showing.
      */
     boolean isShowing() {
         if (mPopup == null) {
@@ skipping 41 matching lines @@
             } else {
                 mPopup.setHeight(spaceForFullItems - mItemRowHeight + spaceForPartialItem
                         + padding.top + padding.bottom);
             }
         } else {
             mPopup.setHeight(ViewGroup.LayoutParams.WRAP_CONTENT);
         }
     }
 
     private void runMenuItemEnterAnimations() {
-        AnimatorSet animation = new AnimatorSet();
+        mMenuItemEnterAnimator = new AnimatorSet();
         AnimatorSet.Builder builder = null;
 
         ViewGroup list = mPopup.getListView();
         for (int i = 0; i < list.getChildCount(); i++) {
             View view = list.getChildAt(i);
             Object animatorObject = view.getTag(R.id.menu_item_enter_anim_id);
             if (animatorObject != null) {
                 if (builder == null) {
-                    builder = animation.play((Animator) animatorObject);
+                    builder = mMenuItemEnterAnimator.play((Animator) animatorObject);
                 } else {
                     builder.with((Animator) animatorObject);
                 }
             }
         }
 
-        animation.start();
+        mMenuItemEnterAnimator.addListener(mAnimationHistogramRecorder);
+        mMenuItemEnterAnimator.start();
     }
 }