Remove time ticks metrics and MetricServiceClient::OnLogUploadComplete

Remove time ticks metrics and MetricServiceClient::OnLogUploadComplete

BUG=

Review-Url: https://codereview.chromium.org/2615753002
Cr-Commit-Position: refs/heads/master@{#443421}
Committed: https://chromium.googlesource.com/chromium/src/+/b710b9ba8df2a9b9d89bd338c487ed1eaa0d4a20

[Steven Holte, 2017-01-04 23:15:04]

holte@chromium.org changed reviewers:
+ rkaplow@chromium.org, wez@chromium.org

[Steven Holte, 2017-01-05 00:52:27]

holte@chromium.org changed reviewers:
+ olivierrobin@chromium.org

[Steven Holte, 2017-01-05 00:52:28]

+olivierrobin for ios/chrome/browser/metrics/
+wez for blimp/engine/app
+rkaplow for everything else

[Wez, 2017-01-05 01:57:27]

blimp/ LGTM

[Olivier, 2017-01-05 07:40:58]

ios lgtm

[rkaplow, 2017-01-05 15:31:38]

lgtm

cool thanks for cleanup

[Steven Holte, 2017-01-05 21:58:26]

holte@chromium.org changed reviewers:
+ asvitkine@chromium.org

[Steven Holte, 2017-01-05 22:00:11]

holte@chromium.org changed reviewers:
+ boliu@chromium.org

[Steven Holte, 2017-01-05 22:00:11]

+boliu for android_webview
+asvitkine for chromecast/browser/metrics

[boliu, 2017-01-05 22:17:11]

On 2017/01/05 22:00:11, Steven Holte wrote:
> +boliu for android_webview

rs lgtm

> +asvitkine for chromecast/browser/metrics

[Alexei Svitkine (slow), 2017-01-05 23:31:25]

LGTM - sweet!

[Steven Holte, 2017-01-11 00:36:50]

The CQ bit was checked by holte@chromium.org

[Steven Holte, 2017-01-11 00:36:51]

The patchset sent to the CQ was uploaded after l-g-t-m from
rkaplow@chromium.org, olivierrobin@chromium.org, wez@chromium.org,
asvitkine@chromium.org, boliu@chromium.org
Link to the patchset: https://codereview.chromium.org/2615753002/#ps60001
(title: "Rebase")

[commit-bot: I haz the power, 2017-01-11 00:37:29]

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-01-11 02:16:09]

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

[commit-bot: I haz the power, 2017-01-11 02:16:10]

Try jobs failed on following builders:
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)

[Steven Holte, 2017-01-11 20:30:07]

The CQ bit was checked by holte@chromium.org

[commit-bot: I haz the power, 2017-01-11 20:30:39]

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-01-11 21:29:37]

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

[commit-bot: I haz the power, 2017-01-11 21:29:39]

Try jobs failed on following builders:
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)

[Steven Holte, 2017-01-12 21:12:50]

The CQ bit was checked by holte@chromium.org

[commit-bot: I haz the power, 2017-01-12 21:13:41]

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-01-12 22:27:47]

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

[commit-bot: I haz the power, 2017-01-12 22:27:49]

Failed to apply patch for
chrome/browser/metrics/chrome_metrics_service_client.cc:
While running git apply --index -p1;
  error: patch failed:
chrome/browser/metrics/chrome_metrics_service_client.cc:34
  error: chrome/browser/metrics/chrome_metrics_service_client.cc: patch does not
apply

Patch:       chrome/browser/metrics/chrome_metrics_service_client.cc
Index: chrome/browser/metrics/chrome_metrics_service_client.cc
diff --git a/chrome/browser/metrics/chrome_metrics_service_client.cc
b/chrome/browser/metrics/chrome_metrics_service_client.cc
index
f736507830c695ea3f8c6bf15275834f2fc8eeba..e36c05bf7947f13b8c5f5e21129644e2bdc2fcb0
100644
--- a/chrome/browser/metrics/chrome_metrics_service_client.cc
+++ b/chrome/browser/metrics/chrome_metrics_service_client.cc
@@ -34,7 +34,6 @@
 #include "chrome/browser/metrics/metrics_reporting_state.h"
 #include "chrome/browser/metrics/sampling_metrics_provider.h"
 #include "chrome/browser/metrics/subprocess_metrics_provider.h"
-#include "chrome/browser/metrics/time_ticks_experiment_win.h"
 #include
"chrome/browser/safe_browsing/certificate_reporting_metrics_provider.h"
 #include "chrome/browser/sync/chrome_sync_client.h"
 #include "chrome/browser/ui/browser_otr_state.h"
@@ -387,13 +386,6 @@ void
ChromeMetricsServiceClient::OnEnvironmentUpdate(std::string* environment) {
 #endif  // OS_WIN || OS_MACOSX
 }
 
-void ChromeMetricsServiceClient::OnLogUploadComplete() {
-  // Collect time ticks stats after each UMA upload.
-#if defined(OS_WIN)
-  chrome::CollectTimeTicksStats();
-#endif
-}
-
 void ChromeMetricsServiceClient::OnLogCleanShutdown() {
 #if defined(OS_WIN)
   base::FilePath user_data_dir;

[Steven Holte, 2017-01-12 22:32:16]

The CQ bit was checked by holte@chromium.org

[Steven Holte, 2017-01-12 22:32:16]

The patchset sent to the CQ was uploaded after l-g-t-m from
rkaplow@chromium.org, asvitkine@chromium.org, olivierrobin@chromium.org,
wez@chromium.org, boliu@chromium.org
Link to the patchset: https://codereview.chromium.org/2615753002/#ps80001
(title: "Rebase")

[commit-bot: I haz the power, 2017-01-12 22:33:34]

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-01-13 00:50:29]

CQ is committing da patch.

Bot data: {"patchset_id": 80001, "attempt_start_ts": 1484260336277200,
"parent_rev": "a27abc1cedbf0f585c5e2830beadebf1cc2b52cb", "commit_rev":
"b710b9ba8df2a9b9d89bd338c487ed1eaa0d4a20"}

[commit-bot: I haz the power, 2017-01-13 00:51:01]

Description was changed from

==========
Remove time ticks metrics and MetricServiceClient::OnLogUploadComplete

BUG=
==========

to

==========
Remove time ticks metrics and MetricServiceClient::OnLogUploadComplete

BUG=

Review-Url: https://codereview.chromium.org/2615753002
Cr-Commit-Position: refs/heads/master@{#443421}
Committed:
https://chromium.googlesource.com/chromium/src/+/b710b9ba8df2a9b9d89bd338c487...
==========

[commit-bot: I haz the power, 2017-01-13 00:51:03]

Committed patchset #5 (id:80001) as
https://chromium.googlesource.com/chromium/src/+/b710b9ba8df2a9b9d89bd338c487...