Remove unused histograms Omnibox.ProgressBar

chrome/android/java/src/org/chromium/chrome/browser/widget/ToolbarProgressBar.java

 // Copyright 2015 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.widget;
 
 import android.animation.TimeAnimator;
 import android.animation.TimeAnimator.TimeListener;
 import android.content.Context;
 import android.graphics.Color;
 import android.os.Build;
 import android.support.v4.view.ViewCompat;
 import android.text.TextUtils;
 import android.view.ViewGroup;
 import android.view.accessibility.AccessibilityEvent;
 import android.widget.FrameLayout.LayoutParams;
 import android.widget.ProgressBar;
 
 import org.chromium.base.ApiCompatibilityUtils;
 import org.chromium.base.CommandLine;
 import org.chromium.base.VisibleForTesting;
-import org.chromium.base.metrics.RecordHistogram;
 import org.chromium.chrome.R;
 import org.chromium.chrome.browser.ChromeSwitches;
 import org.chromium.chrome.browser.util.ColorUtils;
 import org.chromium.components.variations.VariationsAssociatedData;
 import org.chromium.ui.UiUtils;
 import org.chromium.ui.interpolators.BakedBezierInterpolator;
 
 /**
  * Progress bar for use in the Toolbar view.
  */
 public class ToolbarProgressBar extends ClipDrawableProgressBar {
 
     private static final String ANIMATION_FIELD_TRIAL_NAME = "ProgressBarAnimationAndroid";
-    private static final String PROGRESS_BAR_UPDATE_COUNT_HISTOGRAM =
-            "Omnibox.ProgressBarUpdateCount";
-    private static final String PROGRESS_BAR_BREAK_POINT_UPDATE_COUNT_HISTOGRAM =
-            "Omnibox.ProgressBarBreakPointUpdateCount";
 
     /**
      * Interface for progress bar animation interpolation logics.
      */
     interface AnimationLogic {
         /**
          * Resets internal data. It must be called on every loading start.
          * @param startProgress The progress for the animation to start at. This is used when the
          *                      animation logic switches.
          */
@@ skipping 15 matching lines @@
 
     private static final float THEMED_BACKGROUND_WHITE_FRACTION = 0.2f;
     private static final float ANIMATION_WHITE_FRACTION = 0.4f;
 
     private static final long PROGRESS_FRAME_TIME_CAP_MS = 50;
     private long mAlphaAnimationDurationMs = 140;
     private long mHidingDelayMs = 100;
 
     private boolean mIsStarted;
     private float mTargetProgress;
-    private int mTargetProgressUpdateCount;
     private AnimationLogic mAnimationLogic;
     private boolean mAnimationInitialized;
     private int mMarginTop;
     private ViewGroup mProgressBarContainer;
     private int mProgressStartCount;
     private int mThemeColor;
 
     /** Whether or not to use the status bar color as the background of the toolbar. */
     private boolean mUseStatusBarColorAsBackground;
 
@@ skipping 169 matching lines @@
     public void start() {
         mIsStarted = true;
         mProgressStartCount++;
 
         if (mIsUsingSmoothIndeterminate) {
             removeCallbacks(mStartSmoothIndeterminate);
             postDelayed(mStartSmoothIndeterminate, ANIMATION_START_THRESHOLD);
         }
 
         mIsRunningSmoothIndeterminate = false;
-        mTargetProgressUpdateCount = 0;
-        resetProgressUpdateCount();
         super.setProgress(0.0f);
         if (mAnimationLogic != null) mAnimationLogic.reset(0.0f);
         removeCallbacks(mHideRunnable);
         animateAlphaTo(1.0f);
     }
 
     /**
      * @return True if the progress bar is showing and started.
      */
     public boolean isStarted() {
         return mIsStarted;
     }
 
     /**
      * Start hiding progress bar animation.
      * @param delayed Whether a delayed fading out animation should be posted.
      */
     public void finish(boolean delayed) {
         mIsStarted = false;
 
         if (delayed) {
             updateVisibleProgress();
-            RecordHistogram.recordCount1000Histogram(PROGRESS_BAR_UPDATE_COUNT_HISTOGRAM,
-                    getProgressUpdateCount());
-            RecordHistogram.recordCount100Histogram(
-                    PROGRESS_BAR_BREAK_POINT_UPDATE_COUNT_HISTOGRAM,
-                    mTargetProgressUpdateCount);
         } else {
             removeCallbacks(mHideRunnable);
             animate().cancel();
             if (mAnimatingView != null) {
                 removeCallbacks(mStartSmoothIndeterminate);
                 mAnimatingView.cancelAnimation();
                 mTargetProgress = 0;
             }
             mIsRunningSmoothIndeterminate = false;
             setAlpha(0.0f);
@@ skipping 75 matching lines @@
             // If the progress bar was updated, reset the callback that triggers the
             // smooth-indeterminate animation.
             removeCallbacks(mStartSmoothIndeterminate);
             if (progress == 1.0) {
                 if (mAnimatingView != null) mAnimatingView.cancelAnimation();
             } else if (mAnimatingView != null && !mAnimatingView.isRunning()) {
                 postDelayed(mStartSmoothIndeterminate, ANIMATION_START_THRESHOLD);
             }
         }
 
-        mTargetProgressUpdateCount += 1;
         mTargetProgress = progress;
         updateVisibleProgress();
     }
 
     @Override
     public void setVisibility(int visibility) {
         super.setVisibility(visibility);
         if (mAnimatingView != null) mAnimatingView.setVisibility(visibility);
     }
 
@@ skipping 49 matching lines @@
         return ProgressBar.class.getName();
     }
 
     @Override
     public void onInitializeAccessibilityEvent(AccessibilityEvent event) {
         super.onInitializeAccessibilityEvent(event);
         event.setCurrentItemIndex((int) (mTargetProgress * 100));
         event.setItemCount(100);
     }
 }