Remove NetworkTimeTracker.UpdateTimeFetchAttempted histogram

Remove NetworkTimeTracker.UpdateTimeFetchAttempted histogram

As mab@ predicted in
https://codereview.chromium.org/2176373003/diff/1/components/network_time/network_time_tracker.cc#newcode391,
NetworkTimeTracker.UpdateTimeFetchAttempted does not show anything
interesting happening on Canary, so the UpdateTimeFetchValid and
UpdateTimeFetchFailed histograms should be enough for our purposes.

BUG=632419
Committed: https://crrev.com/f9a6bfb3670adb37cb84b14d13a1b4e46fa3e44e
Cr-Commit-Position: refs/heads/master@{#412034}

[estark, 2016-08-10 00:31:00]

estark@chromium.org changed reviewers:
+ mab@google.com

[estark, 2016-08-10 00:31:01]

mab PTAL?

[mab, 2016-08-10 00:35:16]

lgtm

[estark, 2016-08-10 00:38:13]

estark@chromium.org changed reviewers:
+ jwd@chromium.org, zea@chromium.org

[estark, 2016-08-10 00:38:13]

zea: can you please review components/network_time_tracker?
jwd: can you please review histograms.xml?

[jwd, 2016-08-10 17:42:27]

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

https://codereview.chromium.org/2232663002/diff/1/tools/metrics/histograms/hi...
tools/metrics/histograms/histograms.xml:34840: -<histogram
name="NetworkTimeTracker.UpdateTimeFetchAttempted">
Instead of removing, can you set this to obsolete, with mm/yyyy of when, and a
brief reason why please.

[estark, 2016-08-10 17:46:19]

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

https://codereview.chromium.org/2232663002/diff/1/tools/metrics/histograms/hi...
tools/metrics/histograms/histograms.xml:34840: -<histogram
name="NetworkTimeTracker.UpdateTimeFetchAttempted">
On 2016/08/10 17:42:26, jwd wrote:
> Instead of removing, can you set this to obsolete, with mm/yyyy of when, and a
> brief reason why please.

Done.

[jwd, 2016-08-10 18:41:43]

histograms LGTM

[Nicolas Zea, 2016-08-15 18:55:00]

lgtm

[estark, 2016-08-15 18:56:16]

The CQ bit was checked by estark@chromium.org

[estark, 2016-08-15 18:56:16]

The patchset sent to the CQ was uploaded after l-g-t-m from mab@google.com
Link to the patchset: https://codereview.chromium.org/2232663002/#ps20001
(title: "Mark histogram as obsolete instead of deleting")

[commit-bot: I haz the power, 2016-08-15 18:56:37]

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, 2016-08-15 20:24:43]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-08-15 20:45:53]

Description was changed from

==========
Remove NetworkTimeTracker.UpdateTimeFetchAttempted histogram

As mab@ predicted in
https://codereview.chromium.org/2176373003/diff/1/components/network_time/net...,
NetworkTimeTracker.UpdateTimeFetchAttempted does not show anything
interesting happening on Canary, so the UpdateTimeFetchValid and
UpdateTimeFetchFailed histograms should be enough for our purposes.

BUG=632419
==========

to

==========
Remove NetworkTimeTracker.UpdateTimeFetchAttempted histogram

As mab@ predicted in
https://codereview.chromium.org/2176373003/diff/1/components/network_time/net...,
NetworkTimeTracker.UpdateTimeFetchAttempted does not show anything
interesting happening on Canary, so the UpdateTimeFetchValid and
UpdateTimeFetchFailed histograms should be enough for our purposes.

BUG=632419

Committed: https://crrev.com/f9a6bfb3670adb37cb84b14d13a1b4e46fa3e44e
Cr-Commit-Position: refs/heads/master@{#412034}
==========

[commit-bot: I haz the power, 2016-08-15 20:45:54]

Patchset 2 (id:??) landed as
https://crrev.com/f9a6bfb3670adb37cb84b14d13a1b4e46fa3e44e
Cr-Commit-Position: refs/heads/master@{#412034}