| OLD | NEW |
| 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. | 1 // Copyright (c) 2012 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 "chrome/browser/metrics/variations/variations_service.h" | 5 #include "chrome/browser/metrics/variations/variations_service.h" |
| 6 | 6 |
| 7 #include <set> | 7 #include <set> |
| 8 | 8 |
| 9 #include "base/build_time.h" | 9 #include "base/build_time.h" |
| 10 #include "base/command_line.h" | 10 #include "base/command_line.h" |
| (...skipping 239 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 250 // Note that the act of instantiating the scheduler will start the fetch, if | 250 // Note that the act of instantiating the scheduler will start the fetch, if |
| 251 // the scheduler deems appropriate. Using Unretained is fine here since the | 251 // the scheduler deems appropriate. Using Unretained is fine here since the |
| 252 // lifespan of request_scheduler_ is guaranteed to be shorter than that of | 252 // lifespan of request_scheduler_ is guaranteed to be shorter than that of |
| 253 // this service. | 253 // this service. |
| 254 request_scheduler_.reset(VariationsRequestScheduler::Create( | 254 request_scheduler_.reset(VariationsRequestScheduler::Create( |
| 255 base::Bind(&VariationsService::FetchVariationsSeed, | 255 base::Bind(&VariationsService::FetchVariationsSeed, |
| 256 base::Unretained(this)), local_state_)); | 256 base::Unretained(this)), local_state_)); |
| 257 request_scheduler_->Start(); | 257 request_scheduler_->Start(); |
| 258 } | 258 } |
| 259 | 259 |
| 260 // TODO(rkaplow): Handle this and the similar event in metrics_service by |
| 261 // observing an 'OnAppEnterForeground' event in RequestScheduler instead of |
| 262 // requiring the frontend code to notify each service individually. Since the |
| 263 // scheduler will handle it directly the VariationService shouldn't need to |
| 264 // know details of this anymore. |
| 265 void VariationsService::OnAppEnterForeground() { |
| 266 request_scheduler_->OnAppEnterForeground(); |
| 267 } |
| 268 |
| 260 // static | 269 // static |
| 261 GURL VariationsService::GetVariationsServerURL(PrefService* local_state) { | 270 GURL VariationsService::GetVariationsServerURL(PrefService* local_state) { |
| 262 std::string server_url_string(CommandLine::ForCurrentProcess()-> | 271 std::string server_url_string(CommandLine::ForCurrentProcess()-> |
| 263 GetSwitchValueASCII(switches::kVariationsServerURL)); | 272 GetSwitchValueASCII(switches::kVariationsServerURL)); |
| 264 if (server_url_string.empty()) | 273 if (server_url_string.empty()) |
| 265 server_url_string = kDefaultVariationsServerURL; | 274 server_url_string = kDefaultVariationsServerURL; |
| 266 GURL server_url = GURL(server_url_string); | 275 GURL server_url = GURL(server_url_string); |
| 267 | 276 |
| 268 const std::string restrict_param = GetRestrictParameterPref(local_state); | 277 const std::string restrict_param = GetRestrictParameterPref(local_state); |
| 269 if (!restrict_param.empty()) { | 278 if (!restrict_param.empty()) { |
| (...skipping 178 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 448 | 457 |
| 449 void VariationsService::RecordLastFetchTime() { | 458 void VariationsService::RecordLastFetchTime() { |
| 450 // local_state_ is NULL in tests, so check it first. | 459 // local_state_ is NULL in tests, so check it first. |
| 451 if (local_state_) { | 460 if (local_state_) { |
| 452 local_state_->SetInt64(prefs::kVariationsLastFetchTime, | 461 local_state_->SetInt64(prefs::kVariationsLastFetchTime, |
| 453 base::Time::Now().ToInternalValue()); | 462 base::Time::Now().ToInternalValue()); |
| 454 } | 463 } |
| 455 } | 464 } |
| 456 | 465 |
| 457 } // namespace chrome_variations | 466 } // namespace chrome_variations |
| OLD | NEW |