Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(262)

Unified Diff: components/metrics/metrics_service.cc

Issue 1474483004: WebView Metrics client implementation (Chromium part) (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: move client back to browser layer, make it a Leaky global, and some cleanup Created 5 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/metrics/metrics_service.cc
diff --git a/components/metrics/metrics_service.cc b/components/metrics/metrics_service.cc
index 55aa9f10c8c0daaf3e0a764020602c5e3d7c5d1b..4f02e3591e410e0fbda4535070047311242d29dd 100644
--- a/components/metrics/metrics_service.cc
+++ b/components/metrics/metrics_service.cc
@@ -318,7 +318,7 @@ MetricsService::~MetricsService() {
void MetricsService::InitializeMetricsRecordingState() {
InitializeMetricsState();
-
+ LOG(WARNING) << "metrix InitializeMetricsRecordingState";
base::Closure upload_callback =
base::Bind(&MetricsService::StartScheduledUpload,
self_ptr_factory_.GetWeakPtr());
@@ -332,6 +332,7 @@ void MetricsService::InitializeMetricsRecordingState() {
}
void MetricsService::Start() {
+ LOG(WARNING) << "metrix starting metrics service";
HandleIdleSinceLastTransmission(false);
EnableRecording();
EnableReporting();
@@ -344,12 +345,14 @@ void MetricsService::StartRecordingForTests() {
}
void MetricsService::Stop() {
+ LOG(WARNING) << "metrix stopping metrics service";
HandleIdleSinceLastTransmission(false);
DisableReporting();
DisableRecording();
}
void MetricsService::EnableReporting() {
+ LOG(WARNING) << "metrix enablereporting";
if (reporting_active_)
return;
reporting_active_ = true;
@@ -385,7 +388,7 @@ MetricsService::CreateEntropyProvider() {
void MetricsService::EnableRecording() {
DCHECK(IsSingleThreaded());
-
+ LOG(WARNING) << "metrix EnableRecording " << recording_state_;
if (recording_state_ == ACTIVE)
return;
recording_state_ = ACTIVE;
@@ -640,6 +643,8 @@ void MetricsService::InitializeMetricsState() {
}
void MetricsService::OnUserAction(const std::string& action) {
+ LOG(INFO) << "metrix metrics::MetricsService::OnUserAction action=" << action;
+
if (!ShouldLogEvents())
return;
@@ -648,6 +653,7 @@ void MetricsService::OnUserAction(const std::string& action) {
}
void MetricsService::FinishedInitTask() {
+ LOG(WARNING) << "metrix FinishedInitTask";
DCHECK_EQ(INIT_TASK_SCHEDULED, state_);
state_ = INIT_TASK_DONE;
@@ -728,6 +734,7 @@ void MetricsService::OpenNewLog() {
}
void MetricsService::StartInitTask() {
+ LOG(WARNING) << "metrix StartInitTask";
client_->InitializeSystemProfileMetrics(
base::Bind(&MetricsService::FinishedInitTask,
self_ptr_factory_.GetWeakPtr()));
@@ -777,6 +784,7 @@ void MetricsService::PushPendingLogsToPersistentStorage() {
// Transmission of logs methods
void MetricsService::StartSchedulerIfNecessary() {
+ LOG(WARNING) << "metrix StartSchedulerIfNecessary";
// Never schedule cutting or uploading of logs in test mode.
if (test_mode_active_)
return;
@@ -792,6 +800,8 @@ void MetricsService::StartSchedulerIfNecessary() {
void MetricsService::StartScheduledUpload() {
DCHECK(state_ >= INIT_TASK_DONE);
+ LOG(WARNING) << " metrix StartScheduledUpload " << idle_since_last_transmission_
+ << " " << recording_active() << " " << state_;
// If we're getting no notifications, then the log won't have much in it, and
// it's possible the computer is about to go to sleep, so don't upload and
// stop the scheduler.
@@ -847,6 +857,7 @@ void MetricsService::OnFinalLogInfoCollectionDone() {
void MetricsService::SendNextLog() {
DCHECK_EQ(SENDING_LOGS, state_);
+ LOG(WARNING) << "metrix SendNextLog";
if (!reporting_active()) {
scheduler_->Stop();
scheduler_->UploadCancelled();
@@ -940,6 +951,7 @@ void MetricsService::PrepareInitialMetricsLog() {
}
void MetricsService::SendStagedLog() {
+ LOG(WARNING) << "metrix SendStagedLog";
DCHECK(log_manager_.has_staged_log());
if (!log_manager_.has_staged_log())
return;

Powered by Google App Engine
This is Rietveld 408576698