| OLD | NEW |
| 1 // Copyright 2014 The Chromium Authors. All rights reserved. | 1 // Copyright 2014 The Chromium Authors. All rights reserved. |
| 2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
| 3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
| 4 | 4 |
| 5 #include "components/metrics/metrics_service.h" | 5 #include "components/metrics/metrics_service.h" |
| 6 | 6 |
| 7 #include <string> | 7 #include <string> |
| 8 | 8 |
| 9 #include "base/bind.h" | 9 #include "base/bind.h" |
| 10 #include "base/command_line.h" |
| 10 #include "base/memory/scoped_ptr.h" | 11 #include "base/memory/scoped_ptr.h" |
| 11 #include "base/message_loop/message_loop.h" | 12 #include "base/message_loop/message_loop.h" |
| 12 #include "base/metrics/statistics_recorder.h" | 13 #include "base/metrics/statistics_recorder.h" |
| 13 #include "base/prefs/testing_pref_service.h" | 14 #include "base/prefs/testing_pref_service.h" |
| 15 #include "base/strings/string16.h" |
| 14 #include "base/threading/platform_thread.h" | 16 #include "base/threading/platform_thread.h" |
| 15 #include "components/metrics/client_info.h" | 17 #include "components/metrics/client_info.h" |
| 16 #include "components/metrics/compression_utils.h" | 18 #include "components/metrics/compression_utils.h" |
| 17 #include "components/metrics/metrics_hashes.h" | 19 #include "components/metrics/metrics_hashes.h" |
| 18 #include "components/metrics/metrics_log.h" | 20 #include "components/metrics/metrics_log.h" |
| 21 #include "components/metrics/metrics_log_uploader.h" |
| 19 #include "components/metrics/metrics_pref_names.h" | 22 #include "components/metrics/metrics_pref_names.h" |
| 20 #include "components/metrics/metrics_state_manager.h" | 23 #include "components/metrics/metrics_state_manager.h" |
| 24 #include "components/metrics/metrics_switches.h" |
| 25 #include "components/metrics/mock_metrics_service_client.h" |
| 26 #include "components/metrics/proto/system_profile.pb.h" |
| 21 #include "components/metrics/test_metrics_service_client.h" | 27 #include "components/metrics/test_metrics_service_client.h" |
| 22 #include "components/variations/metrics_util.h" | 28 #include "components/variations/metrics_util.h" |
| 29 #include "testing/gmock/include/gmock/gmock.h" |
| 23 #include "testing/gtest/include/gtest/gtest.h" | 30 #include "testing/gtest/include/gtest/gtest.h" |
| 24 | 31 |
| 32 using ::testing::Return; |
| 33 using ::testing::_; |
| 34 |
| 25 namespace metrics { | 35 namespace metrics { |
| 26 | 36 |
| 27 namespace { | 37 namespace { |
| 28 | 38 |
| 29 void StoreNoClientInfoBackup(const ClientInfo& /* client_info */) { | 39 void StoreNoClientInfoBackup(const ClientInfo& /* client_info */) { |
| 30 } | 40 } |
| 31 | 41 |
| 32 scoped_ptr<ClientInfo> ReturnNoBackup() { | 42 scoped_ptr<ClientInfo> ReturnNoBackup() { |
| 33 return scoped_ptr<ClientInfo>(); | 43 return scoped_ptr<ClientInfo>(); |
| 34 } | 44 } |
| (...skipping 48 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 83 MetricsLog::ONGOING_LOG, | 93 MetricsLog::ONGOING_LOG, |
| 84 client, | 94 client, |
| 85 local_state) {} | 95 local_state) {} |
| 86 | 96 |
| 87 ~TestMetricsLog() override {} | 97 ~TestMetricsLog() override {} |
| 88 | 98 |
| 89 private: | 99 private: |
| 90 DISALLOW_COPY_AND_ASSIGN(TestMetricsLog); | 100 DISALLOW_COPY_AND_ASSIGN(TestMetricsLog); |
| 91 }; | 101 }; |
| 92 | 102 |
| 103 class TestMetricsLogUploader : public MetricsLogUploader { |
| 104 public: |
| 105 TestMetricsLogUploader( |
| 106 const std::string& server_url, |
| 107 const std::string& mime_type, |
| 108 const base::Callback<void(int)>& on_upload_complete): |
| 109 MetricsLogUploader(server_url, mime_type, on_upload_complete) {} |
| 110 |
| 111 ~TestMetricsLogUploader() override {} |
| 112 |
| 113 bool UploadLog(const std::string& compressed_log_data, |
| 114 const std::string& log_hash) override { |
| 115 return true; |
| 116 } |
| 117 |
| 118 private: |
| 119 DISALLOW_COPY_AND_ASSIGN(TestMetricsLogUploader); |
| 120 }; |
| 121 |
| 93 class MetricsServiceTest : public testing::Test { | 122 class MetricsServiceTest : public testing::Test { |
| 94 public: | 123 public: |
| 95 MetricsServiceTest() : is_metrics_reporting_enabled_(false) { | 124 MetricsServiceTest() : is_metrics_reporting_enabled_(false) { |
| 96 MetricsService::RegisterPrefs(testing_local_state_.registry()); | 125 MetricsService::RegisterPrefs(testing_local_state_.registry()); |
| 97 metrics_state_manager_ = MetricsStateManager::Create( | 126 metrics_state_manager_ = MetricsStateManager::Create( |
| 98 GetLocalState(), | 127 GetLocalState(), |
| 99 base::Bind(&MetricsServiceTest::is_metrics_reporting_enabled, | 128 base::Bind(&MetricsServiceTest::is_metrics_reporting_enabled, |
| 100 base::Unretained(this)), | 129 base::Unretained(this)), |
| 101 base::Bind(&StoreNoClientInfoBackup), | 130 base::Bind(&StoreNoClientInfoBackup), |
| 102 base::Bind(&ReturnNoBackup)); | 131 base::Bind(&ReturnNoBackup)); |
| 103 } | 132 } |
| 104 | 133 |
| 105 ~MetricsServiceTest() override { | 134 ~MetricsServiceTest() override { |
| 106 MetricsService::SetExecutionPhase(MetricsService::UNINITIALIZED_PHASE, | 135 MetricsService::SetExecutionPhase(MetricsService::UNINITIALIZED_PHASE, |
| 107 GetLocalState()); | 136 GetLocalState()); |
| 108 } | 137 } |
| 109 | 138 |
| 110 MetricsStateManager* GetMetricsStateManager() { | 139 MetricsStateManager* GetMetricsStateManager() { |
| 111 return metrics_state_manager_.get(); | 140 return metrics_state_manager_.get(); |
| 112 } | 141 } |
| 113 | 142 |
| 114 PrefService* GetLocalState() { return &testing_local_state_; } | 143 PrefService* GetLocalState() { return &testing_local_state_; } |
| 115 | 144 |
| 116 // Sets metrics reporting as enabled for testing. | 145 // Sets metrics reporting as enabled for testing. |
| 117 void EnableMetricsReporting() { | 146 void EnableMetricsReporting() { |
| 118 is_metrics_reporting_enabled_ = true; | 147 is_metrics_reporting_enabled_ = true; |
| 119 } | 148 } |
| 120 | 149 |
| 150 void PopulateStabilityLog(MetricsServiceClient& client, |
| 151 bool exited_cleanly) { |
| 152 // Record stability build time and version from previous session, so that |
| 153 // stability metrics (including exited cleanly flag) won't be cleared. |
| 154 testing_local_state_.SetInt64(prefs::kStabilityStatsBuildTime, |
| 155 MetricsLog::GetBuildTime()); |
| 156 testing_local_state_.SetString(prefs::kStabilityStatsVersion, |
| 157 client.GetVersionString()); |
| 158 |
| 159 // Set the clean exit flag, as that will otherwise cause a stabilty |
| 160 // log to be produced, irrespective provider requests. |
| 161 testing_local_state_.SetBoolean( |
| 162 prefs::kStabilityExitedCleanly, exited_cleanly); |
| 163 } |
| 164 |
| 121 // Waits until base::TimeTicks::Now() no longer equals |value|. This should | 165 // Waits until base::TimeTicks::Now() no longer equals |value|. This should |
| 122 // take between 1-15ms per the documented resolution of base::TimeTicks. | 166 // take between 1-15ms per the documented resolution of base::TimeTicks. |
| 123 void WaitUntilTimeChanges(const base::TimeTicks& value) { | 167 void WaitUntilTimeChanges(const base::TimeTicks& value) { |
| 124 while (base::TimeTicks::Now() == value) { | 168 while (base::TimeTicks::Now() == value) { |
| 125 base::PlatformThread::Sleep(base::TimeDelta::FromMilliseconds(1)); | 169 base::PlatformThread::Sleep(base::TimeDelta::FromMilliseconds(1)); |
| 126 } | 170 } |
| 127 } | 171 } |
| 128 | 172 |
| 129 // Returns true if there is a synthetic trial in the given vector that matches | 173 // Returns true if there is a synthetic trial in the given vector that matches |
| 130 // the given trial name and trial group; returns false otherwise. | 174 // the given trial name and trial group; returns false otherwise. |
| (...skipping 79 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 210 EnableMetricsReporting(); | 254 EnableMetricsReporting(); |
| 211 | 255 |
| 212 // Save an existing system profile to prefs, to correspond to what would be | 256 // Save an existing system profile to prefs, to correspond to what would be |
| 213 // saved from a previous session. | 257 // saved from a previous session. |
| 214 TestMetricsServiceClient client; | 258 TestMetricsServiceClient client; |
| 215 TestMetricsLog log("client", 1, &client, GetLocalState()); | 259 TestMetricsLog log("client", 1, &client, GetLocalState()); |
| 216 log.RecordEnvironment(std::vector<MetricsProvider*>(), | 260 log.RecordEnvironment(std::vector<MetricsProvider*>(), |
| 217 std::vector<variations::ActiveGroupId>(), | 261 std::vector<variations::ActiveGroupId>(), |
| 218 0); | 262 0); |
| 219 | 263 |
| 220 // Record stability build time and version from previous session, so that | 264 PopulateStabilityLog(client, true); |
| 221 // stability metrics (including exited cleanly flag) won't be cleared. | |
| 222 GetLocalState()->SetInt64(prefs::kStabilityStatsBuildTime, | |
| 223 MetricsLog::GetBuildTime()); | |
| 224 GetLocalState()->SetString(prefs::kStabilityStatsVersion, | |
| 225 client.GetVersionString()); | |
| 226 | |
| 227 // Set the clean exit flag, as that will otherwise cause a stabilty | |
| 228 // log to be produced, irrespective provider requests. | |
| 229 GetLocalState()->SetBoolean(prefs::kStabilityExitedCleanly, true); | |
| 230 | 265 |
| 231 TestMetricsService service( | 266 TestMetricsService service( |
| 232 GetMetricsStateManager(), &client, GetLocalState()); | 267 GetMetricsStateManager(), &client, GetLocalState()); |
| 233 // Add a metrics provider that requests a stability log. | 268 // Add a metrics provider that requests a stability log. |
| 234 TestMetricsProvider* test_provider = new TestMetricsProvider(true); | 269 TestMetricsProvider* test_provider = new TestMetricsProvider(true); |
| 235 service.RegisterMetricsProvider( | 270 service.RegisterMetricsProvider( |
| 236 scoped_ptr<MetricsProvider>(test_provider)); | 271 scoped_ptr<MetricsProvider>(test_provider)); |
| 237 | 272 |
| 238 service.InitializeMetricsRecordingState(); | 273 service.InitializeMetricsRecordingState(); |
| 239 | 274 |
| (...skipping 35 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 275 | 310 |
| 276 // Set up prefs to simulate restarting after a crash. | 311 // Set up prefs to simulate restarting after a crash. |
| 277 | 312 |
| 278 // Save an existing system profile to prefs, to correspond to what would be | 313 // Save an existing system profile to prefs, to correspond to what would be |
| 279 // saved from a previous session. | 314 // saved from a previous session. |
| 280 TestMetricsServiceClient client; | 315 TestMetricsServiceClient client; |
| 281 TestMetricsLog log("client", 1, &client, GetLocalState()); | 316 TestMetricsLog log("client", 1, &client, GetLocalState()); |
| 282 log.RecordEnvironment(std::vector<MetricsProvider*>(), | 317 log.RecordEnvironment(std::vector<MetricsProvider*>(), |
| 283 std::vector<variations::ActiveGroupId>(), | 318 std::vector<variations::ActiveGroupId>(), |
| 284 0); | 319 0); |
| 285 | 320 PopulateStabilityLog(client, false); |
| 286 // Record stability build time and version from previous session, so that | |
| 287 // stability metrics (including exited cleanly flag) won't be cleared. | |
| 288 GetLocalState()->SetInt64(prefs::kStabilityStatsBuildTime, | |
| 289 MetricsLog::GetBuildTime()); | |
| 290 GetLocalState()->SetString(prefs::kStabilityStatsVersion, | |
| 291 client.GetVersionString()); | |
| 292 | |
| 293 GetLocalState()->SetBoolean(prefs::kStabilityExitedCleanly, false); | |
| 294 | 321 |
| 295 TestMetricsService service( | 322 TestMetricsService service( |
| 296 GetMetricsStateManager(), &client, GetLocalState()); | 323 GetMetricsStateManager(), &client, GetLocalState()); |
| 297 service.InitializeMetricsRecordingState(); | 324 service.InitializeMetricsRecordingState(); |
| 298 | 325 |
| 299 // The initial stability log should be generated and persisted in unsent logs. | 326 // The initial stability log should be generated and persisted in unsent logs. |
| 300 MetricsLogManager* log_manager = service.log_manager(); | 327 MetricsLogManager* log_manager = service.log_manager(); |
| 301 EXPECT_TRUE(log_manager->has_unsent_logs()); | 328 EXPECT_TRUE(log_manager->has_unsent_logs()); |
| 302 EXPECT_FALSE(log_manager->has_staged_log()); | 329 EXPECT_FALSE(log_manager->has_staged_log()); |
| 303 | 330 |
| (...skipping 75 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 379 scoped_ptr<MetricsLog>(new MetricsLog( | 406 scoped_ptr<MetricsLog>(new MetricsLog( |
| 380 "clientID", 1, MetricsLog::ONGOING_LOG, &client, GetLocalState()))); | 407 "clientID", 1, MetricsLog::ONGOING_LOG, &client, GetLocalState()))); |
| 381 service.GetCurrentSyntheticFieldTrials(&synthetic_trials); | 408 service.GetCurrentSyntheticFieldTrials(&synthetic_trials); |
| 382 EXPECT_EQ(3U, synthetic_trials.size()); | 409 EXPECT_EQ(3U, synthetic_trials.size()); |
| 383 EXPECT_TRUE(HasSyntheticTrial(synthetic_trials, "TestTrial1", "Group2")); | 410 EXPECT_TRUE(HasSyntheticTrial(synthetic_trials, "TestTrial1", "Group2")); |
| 384 EXPECT_TRUE(HasSyntheticTrial(synthetic_trials, "TestTrial2", "Group2")); | 411 EXPECT_TRUE(HasSyntheticTrial(synthetic_trials, "TestTrial2", "Group2")); |
| 385 EXPECT_TRUE(HasSyntheticTrial(synthetic_trials, "TestTrial3", "Group3")); | 412 EXPECT_TRUE(HasSyntheticTrial(synthetic_trials, "TestTrial3", "Group3")); |
| 386 service.log_manager_.FinishCurrentLog(); | 413 service.log_manager_.FinishCurrentLog(); |
| 387 } | 414 } |
| 388 | 415 |
| 416 TEST_F(MetricsServiceTest, ServerUrlOverrideSet) { |
| 417 EnableMetricsReporting(); |
| 418 |
| 419 std::string server_url = "https://127.0.0.1/v2"; |
| 420 base::CommandLine* command_line = base::CommandLine::ForCurrentProcess(); |
| 421 // This is the switch that we want to test. |
| 422 command_line->AppendSwitchASCII(switches::kOverrideMetricsUploadUrl, |
| 423 server_url); |
| 424 |
| 425 MockMetricsServiceClient client; |
| 426 EXPECT_CALL(client, GetRegistryBackupKey()) |
| 427 .WillOnce(Return(base::string16())); |
| 428 EXPECT_CALL(client, GetChannel()) |
| 429 .WillRepeatedly(Return(SystemProfileProto::CHANNEL_UNKNOWN)); |
| 430 // The main expectation of this test: ensures that when the command line |
| 431 // switch is set, the uploader is created with the correct url. |
| 432 EXPECT_CALL(client, MockCreateUploader(server_url, _, _)) |
| 433 .WillOnce(Return(new TestMetricsLogUploader(server_url, "mime_type", |
| 434 base::Bind(&MockMetricsServiceClient::DoNothing, |
| 435 base::Unretained(&client))))); |
| 436 |
| 437 // Generate a set of logs for upload |
| 438 TestMetricsLog log("client", 1, &client, GetLocalState()); |
| 439 log.RecordEnvironment(std::vector<MetricsProvider*>(), |
| 440 std::vector<variations::ActiveGroupId>(), |
| 441 0); |
| 442 PopulateStabilityLog(client, true); |
| 443 |
| 444 MetricsService service(GetMetricsStateManager(), &client, GetLocalState()); |
| 445 TestMetricsProvider* test_provider = new TestMetricsProvider(true); |
| 446 service.RegisterMetricsProvider( |
| 447 scoped_ptr<MetricsProvider>(test_provider)); |
| 448 service.InitializeMetricsRecordingState(); |
| 449 EXPECT_TRUE(service.log_manager()->has_unsent_logs()); |
| 450 |
| 451 service.log_manager()->StageNextLogForUpload(); |
| 452 EXPECT_TRUE(service.log_manager()->has_staged_log()); |
| 453 |
| 454 service.SendStagedLog(); |
| 455 } |
| 456 |
| 389 } // namespace metrics | 457 } // namespace metrics |
| OLD | NEW |