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

Unified Diff: chrome/browser/network_time/network_time_service.cc

Issue 271853004: Merge NetworkTimeNotifier to NetworkTimeTracker (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 6 years, 7 months 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: chrome/browser/network_time/network_time_service.cc
diff --git a/chrome/browser/network_time/network_time_service.cc b/chrome/browser/network_time/network_time_service.cc
index 86a85d16c34ce2c2087fd00195f61f8be0d0706c..7776ba773d47a33b7634de1a00f1b6805129b798 100644
--- a/chrome/browser/network_time/network_time_service.cc
+++ b/chrome/browser/network_time/network_time_service.cc
@@ -6,6 +6,7 @@
#include "base/command_line.h"
#include "base/prefs/pref_service.h"
+#include "chrome/browser/browser_process.h"
#include "chrome/browser/network_time/network_time_tracker.h"
#include "chrome/browser/profiles/profile.h"
#include "chrome/common/chrome_switches.h"
@@ -28,9 +29,6 @@ NetworkTimeService::NetworkTimeService(Profile* profile)
return;
}
- network_time_tracker_.reset(new NetworkTimeTracker);
- network_time_tracker_->Start();
-
const base::DictionaryValue* time_mapping =
profile_->GetPrefs()->GetDictionary(prefs::kNetworkTimeMapping);
double local_time_js;
@@ -43,7 +41,7 @@ NetworkTimeService::NetworkTimeService(Profile* profile)
// Drop saved mapping if clock skew has changed or the data is too old.
profile_->GetPrefs()->ClearPref(prefs::kNetworkTimeMapping);
} else {
- network_time_tracker_->InitFromSavedTime(
+ g_browser_process->network_time_tracker()->InitFromSavedTime(
NetworkTimeTracker::TimeMapping(
local_time_saved, base::Time::FromJsTime(network_time_js)));
}
@@ -53,10 +51,7 @@ NetworkTimeService::NetworkTimeService(Profile* profile)
NetworkTimeService::~NetworkTimeService() {}
void NetworkTimeService::Shutdown() {
- if (!network_time_tracker_)
- return;
-
- if (network_time_tracker_->received_network_time()) {
+ if (g_browser_process->network_time_tracker()->received_network_time()) {
// Update time mapping if tracker received time update from server, i.e.
// mapping is accurate.
base::Time local_now = base::Time::Now();
@@ -70,13 +65,12 @@ void NetworkTimeService::Shutdown() {
base::Time NetworkTimeService::GetNetworkTime(
const base::Time& local_time) {
- if (!network_time_tracker_.get() || local_time.is_null() ||
- local_time.is_max())
+ if (local_time.is_null() || local_time.is_max())
return local_time;
base::Time network_time_now;
- if (!network_time_tracker_->GetNetworkTime(base::TimeTicks::Now(),
- &network_time_now, NULL)) {
+ if (!g_browser_process->network_time_tracker()->GetNetworkTime(
+ base::TimeTicks::Now(), &network_time_now, NULL)) {
return local_time;
}
return local_time + (network_time_now - base::Time::Now());
« no previous file with comments | « chrome/browser/network_time/network_time_service.h ('k') | chrome/browser/network_time/network_time_tracker.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698