Remove unused histograms

Removed Omnibox.ProgressBarUpdateCount and
Omnibox.ProgressBarBreakPointUpdateCount
since collected data is not used anymore

BUG=719801
R=mdjones@chromium.org

Review-Url: https://codereview.chromium.org/2885013004
Cr-Commit-Position: refs/heads/master@{#473413}

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

@@ -39,7 +39,6 @@ public class ClipDrawableProgressBar extends ImageView {
     private final ColorDrawable mForegroundDrawable;
     private int mBackgroundColor = Color.TRANSPARENT;
     private float mProgress;
-    private int mProgressUpdateCount;
     private int mDesiredVisibility;
 
     /**
@@ -84,7 +83,6 @@ public class ClipDrawableProgressBar extends ImageView {
         if (mProgress == progress) return;
 
         mProgress = progress;
-        mProgressUpdateCount += 1;
         getDrawable().setLevel(Math.round(progress * CLIP_DRAWABLE_MAX));
     }
 
@@ -130,20 +128,6 @@ public class ClipDrawableProgressBar extends ImageView {
         }
     }
 
-    /**
-     * Resets progress update count to 0.
-     */
-    public void resetProgressUpdateCount() {
-        mProgressUpdateCount = 0;
-    }
-
-    /**
-     * @return Progress update count since reset.
-     */
-    public int getProgressUpdateCount() {
-        return mProgressUpdateCount;
-    }
-
     private void updateInternalVisibility() {
         int oldVisibility = getVisibility();
         int newVisibility = mDesiredVisibility;

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

@@ -19,7 +19,6 @@ 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;
@@ -33,10 +32,6 @@ import org.chromium.ui.interpolators.BakedBezierInterpolator;
 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.
@@ -72,7 +67,6 @@ public class ToolbarProgressBar extends ClipDrawableProgressBar {
 
     private boolean mIsStarted;
     private float mTargetProgress;
-    private int mTargetProgressUpdateCount;
     private AnimationLogic mAnimationLogic;
     private boolean mAnimationInitialized;
     private int mMarginTop;
@@ -262,8 +256,6 @@ public class ToolbarProgressBar extends ClipDrawableProgressBar {
         }
 
         mIsRunningSmoothIndeterminate = false;
-        mTargetProgressUpdateCount = 0;
-        resetProgressUpdateCount();
         super.setProgress(0.0f);
         if (mAnimationLogic != null) mAnimationLogic.reset(0.0f);
         removeCallbacks(mHideRunnable);
@@ -286,11 +278,6 @@ public class ToolbarProgressBar extends ClipDrawableProgressBar {
 
         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();
@@ -386,7 +373,6 @@ public class ToolbarProgressBar extends ClipDrawableProgressBar {
             }
         }
 
-        mTargetProgressUpdateCount += 1;
         mTargetProgress = progress;
         updateVisibleProgress();
     }

tools/metrics/histograms/histograms.xml

@@ -46385,6 +46385,9 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
 
 <histogram name="Omnibox.ProgressBarBreakPointUpdateCount"
     units="break point updates">
+  <obselete>
+    Obselete 05/16/2017. Data is unused (crbug.com/719801).
+  </obselete>
   <owner>kkimlabs@chromium.org</owner>
   <summary>
     The number of progress bar break point updates from page load started to
@@ -46393,6 +46396,9 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
 </histogram>
 
 <histogram name="Omnibox.ProgressBarUpdateCount" units="frame updates">
+  <obselete>
+    Obselete 05/16/2017. Data is unused (crbug.com/719801).
+  </obselete>
   <owner>kkimlabs@chromium.org</owner>
   <summary>
     The number of progress bar frame updates from page load started to page load