network_time: Remove NetworkTimeService

chrome/test/base/testing_browser_process.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/test/base/testing_browser_process.h"
 
 #include "base/prefs/pref_service.h"
 #include "base/strings/string_util.h"
 #include "base/time/default_tick_clock.h"
 #include "build/build_config.h"
@@ skipping 50 matching lines @@
   // g_browser_process must be NULL during its own destruction.
   BrowserProcess* browser_process = g_browser_process;
   g_browser_process = NULL;
   delete browser_process;
 }
 
 TestingBrowserProcess::TestingBrowserProcess()
     : notification_service_(content::NotificationService::Create()),
       module_ref_count_(0),
       app_locale_("en"),
-      network_time_tracker_(new NetworkTimeTracker(
-          scoped_ptr<base::TickClock>(new base::DefaultTickClock))),
       local_state_(NULL),
       io_thread_(NULL),
       system_request_context_(NULL),
       platform_part_(new TestingBrowserProcessPlatformPart()),
       extensions_browser_client_(
           new extensions::ChromeExtensionsBrowserClient) {
   extensions::ExtensionsBrowserClient::Set(extensions_browser_client_.get());
   apps::AppsClient::Set(ChromeAppsClient::GetInstance());
 }
 
@@ skipping 275 matching lines @@
     return (local_state_ != NULL);
 }
 
 #if defined(ENABLE_WEBRTC)
 WebRtcLogUploader* TestingBrowserProcess::webrtc_log_uploader() {
   return NULL;
 }
 #endif
 
 NetworkTimeTracker* TestingBrowserProcess::network_time_tracker() {
+  if (!network_time_tracker_) {
+    DCHECK(local_state_);
+    network_time_tracker_.reset(new NetworkTimeTracker(
+        scoped_ptr<base::TickClock>(new base::DefaultTickClock()),
+        local_state_));
+  }
   return network_time_tracker_.get();
 }
 
 void TestingBrowserProcess::SetSystemRequestContext(
     net::URLRequestContextGetter* context_getter) {
   system_request_context_ = context_getter;
 }
 
 void TestingBrowserProcess::SetLocalState(PrefService* local_state) {
   if (!local_state) {
     // The local_state_ PrefService is owned outside of TestingBrowserProcess,
     // but some of the members of TestingBrowserProcess hold references to it
     // (for example, via PrefNotifier members). But given our test
     // infrastructure which tears down individual tests before freeing the
     // TestingBrowserProcess, there's not a good way to make local_state outlive
     // these dependencies. As a workaround, whenever local_state_ is cleared
     // (assumedly as part of exiting the test and freeing TestingBrowserProcess)
     // any components owned by TestingBrowserProcess that depend on local_state
     // are also freed.
+    network_time_tracker_.reset();
 #if !defined(OS_IOS)
     notification_ui_manager_.reset();
 #endif
 #if defined(ENABLE_CONFIGURATION_POLICY)
     SetBrowserPolicyConnector(NULL);
 #endif
   }
   local_state_ = local_state;
 }
 
@@ skipping 23 matching lines @@
 
 ///////////////////////////////////////////////////////////////////////////////
 
 TestingBrowserProcessInitializer::TestingBrowserProcessInitializer() {
   TestingBrowserProcess::CreateInstance();
 }
 
 TestingBrowserProcessInitializer::~TestingBrowserProcessInitializer() {
   TestingBrowserProcess::DeleteInstance();
 }