Remove unused histograms Omnibox.ProgressBar

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}
Committed: https://chromium.googlesource.com/chromium/src/+/b3b711c92dfc7033662f8ee048dd8a8bd5e88b16

[huayinz, 2017-05-17 21:27:25]

Description was changed from

==========
remove unused histograms:  Omnibox.ProgressBarUpdateCount 
Omnibox.ProgressBarBreakPointUpdateCount

BUG=719801
R=mdjones@chromium.org
==========

to

==========
remove unused histograms
  
Removed Omnibox.ProgressBarUpdateCount and 
Omnibox.ProgressBarBreakPointUpdateCount since collected data is not used
anymore

BUG=719801
R=mdjones@chromium.org
==========

[huayinz, 2017-05-17 21:41:08]

Description was changed from

==========
remove unused histograms
  
Removed Omnibox.ProgressBarUpdateCount and 
Omnibox.ProgressBarBreakPointUpdateCount since collected data is not used
anymore

BUG=719801
R=mdjones@chromium.org
==========

to

==========
remove unused histograms
  
Removed Omnibox.ProgressBarUpdateCount and 
Omnibox.ProgressBarBreakPointUpdateCount 
since collected data is not used anymore

BUG=719801
R=mdjones@chromium.org
==========

[huayinz, 2017-05-17 22:10:39]

Description was changed from

==========
remove unused histograms
  
Removed Omnibox.ProgressBarUpdateCount and 
Omnibox.ProgressBarBreakPointUpdateCount 
since collected data is not used anymore

BUG=719801
R=mdjones@chromium.org
==========

to

==========
Remove unused histograms
  
Removed Omnibox.ProgressBarUpdateCount and 
Omnibox.ProgressBarBreakPointUpdateCount 
since collected data is not used anymore

BUG=719801
R=mdjones@chromium.org
==========

[mdjones, 2017-05-17 22:31:20]

lgtm % suggestion

https://codereview.chromium.org/2885013004/diff/1/tools/metrics/histograms/hi...
File tools/metrics/histograms/histograms.xml (right):

https://codereview.chromium.org/2885013004/diff/1/tools/metrics/histograms/hi...
tools/metrics/histograms/histograms.xml:46331: +    Obselete 05/16/2017. Data is
not needed anymore.
nit: "Data is unused (crbug.com/719801)."

[huayinz, 2017-05-17 23:01:49]

https://codereview.chromium.org/2885013004/diff/1/tools/metrics/histograms/hi...
File tools/metrics/histograms/histograms.xml (right):

https://codereview.chromium.org/2885013004/diff/1/tools/metrics/histograms/hi...
tools/metrics/histograms/histograms.xml:46331: +    Obselete 05/16/2017. Data is
not needed anymore.
On 2017/05/17 22:31:20, mdjones wrote:
> nit: "Data is unused (crbug.com/719801)."

Done.

[huayinz, 2017-05-17 23:08:50]

huayinz@chromium.org changed reviewers:
+ mpearson@chromium.org, tedchoc@chromium.org

[huayinz, 2017-05-17 23:08:50]

tedchoc@chromium.org: ptal

mpearson@chromium.org: Please look at histograms.xml

Thanks,
Becky

[huayinz, 2017-05-17 23:09:57]

huayinz@chromium.org changed reviewers:
- mpearson@chromium.org

[huayinz, 2017-05-17 23:10:53]

huayinz@chromium.org changed reviewers:
+ jwd@chromium.org

[huayinz, 2017-05-17 23:10:54]

jwd@chromium.org: Please take a look at histograms.xml

Thanks,
Becky

[Ted C, 2017-05-17 23:27:36]

lgtm

[jwd, 2017-05-19 19:44:58]

lgtm

[huayinz, 2017-05-19 23:30:26]

The CQ bit was checked by huayinz@chromium.org

[huayinz, 2017-05-19 23:30:26]

The patchset sent to the CQ was uploaded after l-g-t-m from mdjones@chromium.org
Link to the patchset: https://codereview.chromium.org/2885013004/#ps20001
(title: "add bug link to obselete tag")

[commit-bot: I haz the power, 2017-05-19 23:30:57]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-19 23:40:18]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-05-19 23:40:19]

Try jobs failed on following builders:
  android_arm64_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_arm6...)
  android_clang_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_clan...)
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[huayinz, 2017-05-20 01:45:15]

The CQ bit was checked by huayinz@chromium.org

[commit-bot: I haz the power, 2017-05-20 01:45:42]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-20 04:41:23]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1495244715935770,
"parent_rev": "0e3e423a26aafb66f0f10bfbdddc3ad7d2b3e6ad", "commit_rev":
"b3b711c92dfc7033662f8ee048dd8a8bd5e88b16"}

[commit-bot: I haz the power, 2017-05-20 04:41:38]

Description was changed from

==========
Remove unused histograms
  
Removed Omnibox.ProgressBarUpdateCount and 
Omnibox.ProgressBarBreakPointUpdateCount 
since collected data is not used anymore

BUG=719801
R=mdjones@chromium.org
==========

to

==========
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}
Committed:
https://chromium.googlesource.com/chromium/src/+/b3b711c92dfc7033662f8ee048dd...
==========

[commit-bot: I haz the power, 2017-05-20 04:41:41]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/b3b711c92dfc7033662f8ee048dd...