ChromeOS: Implement periodic timezone refresh on geolocation data.

chrome/browser/chromeos/login/ui/login_display_host_impl.cc

 // Copyright 2014 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/browser/chromeos/login/ui/login_display_host_impl.h"
 
 #include <vector>
 
 #include "base/bind.h"
 #include "base/command_line.h"
@@ skipping 26 matching lines @@
 #include "chrome/browser/chromeos/login/ui/input_events_blocker.h"
 #include "chrome/browser/chromeos/login/ui/oobe_display.h"
 #include "chrome/browser/chromeos/login/ui/webui_login_display.h"
 #include "chrome/browser/chromeos/login/ui/webui_login_view.h"
 #include "chrome/browser/chromeos/login/wizard_controller.h"
 #include "chrome/browser/chromeos/mobile_config.h"
 #include "chrome/browser/chromeos/net/delay_network_call.h"
 #include "chrome/browser/chromeos/policy/browser_policy_connector_chromeos.h"
 #include "chrome/browser/chromeos/policy/enrollment_config.h"
 #include "chrome/browser/chromeos/system/input_device_settings.h"
+#include "chrome/browser/chromeos/system/timezone_util.h"
 #include "chrome/browser/chromeos/ui/focus_ring_controller.h"
 #include "chrome/browser/lifetime/application_lifetime.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/browser/ui/webui/chromeos/login/oobe_ui.h"
 #include "chrome/common/chrome_constants.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/pref_names.h"
 #include "chrome/grit/browser_resources.h"
 #include "chromeos/audio/chromeos_sounds.h"
 #include "chromeos/chromeos_constants.h"
@@ skipping 1087 matching lines @@
       media::SoundsManager::Get()->GetDuration(SOUND_STARTUP));
 }
 
 void LoginDisplayHostImpl::OnLoginPromptVisible() {
   if (!login_prompt_visible_time_.is_null())
     return;
   login_prompt_visible_time_ = base::TimeTicks::Now();
   TryToPlayStartupSound();
 }
 
+void LoginDisplayHostImpl::StartTimeZoneResolve() {
+  if (!base::CommandLine::ForCurrentProcess()->HasSwitch(
+          chromeos::switches::kEnableTimeZoneTrackingOption)) {
+    return;
+  }
+
+  if (!g_browser_process->local_state()->GetBoolean(
+          prefs::kResolveDeviceTimezoneByGeolocation)) {
+    return;
+  }
+
+  if (system::HasSystemTimezonePolicy())
+    return;
+
+  g_browser_process->platform_part()->timezone_resolver()->Start();
+}
+
 ////////////////////////////////////////////////////////////////////////////////
 // external
 
 // Declared in login_wizard.h so that others don't need to depend on our .h.
 // TODO(nkostylev): Split this into a smaller functions.
 void ShowLoginWizard(const std::string& first_screen_name) {
   if (browser_shutdown::IsTryingToQuit())
     return;
 
   VLOG(1) << "Showing OOBE screen: " << first_screen_name;
@@ skipping 21 matching lines @@
           switches::kNaturalScrollDefault));
 #endif
 
   gfx::Rect screen_bounds(chromeos::CalculateScreenBounds(gfx::Size()));
 
   g_browser_process->platform_part()->SessionManager()->SetSessionState(
       StartupUtils::IsOobeCompleted()
           ? session_manager::SESSION_STATE_LOGIN_PRIMARY
           : session_manager::SESSION_STATE_OOBE);
 
-  LoginDisplayHost* display_host = new LoginDisplayHostImpl(screen_bounds);
+  LoginDisplayHostImpl* display_host_impl =

[Dmitry Polukhin, 2015/01/20 15:32:28]

What happens with lifetime of the object? Even if it is OK, I still have
concerns about creating alias for memory allocated by new. I think it is better
to change display_host type to be LoginDisplayHostImpl.

[Alexander Alekseev, 2015/01/22 18:55:22]

Done.

+      new LoginDisplayHostImpl(screen_bounds);
+  LoginDisplayHost* display_host = display_host_impl;
 
   bool show_app_launch_splash_screen =
       (first_screen_name == WizardController::kAppLaunchSplashScreenName);
   if (show_app_launch_splash_screen) {
     const std::string& auto_launch_app_id =
         KioskAppManager::Get()->GetAutoLaunchApp();
     display_host->StartAppLaunch(auto_launch_app_id,
                                  false /* diagnostic_mode */);
     return;
   }
 
   // Check whether we need to execute OOBE flow.
   const policy::EnrollmentConfig enrollment_config =
       g_browser_process->platform_part()
           ->browser_policy_connector_chromeos()
           ->GetPrescribedEnrollmentConfig();
   if (enrollment_config.should_enroll() && first_screen_name.empty()) {
     // Shows networks screen instead of enrollment screen to resume the
     // interrupted auto start enrollment flow because enrollment screen does
     // not handle flaky network. See http://crbug.com/332572
     display_host->StartWizard(WizardController::kNetworkScreenName);
     return;
   }
 
   if (StartupUtils::IsEulaAccepted()) {
     DelayNetworkCall(
         ServicesCustomizationDocument::GetInstance()
             ->EnsureCustomizationAppliedClosure(),
         base::TimeDelta::FromMilliseconds(kDefaultNetworkRetryDelayMS));
+
+    display_host_impl->StartTimeZoneResolve();
   }
 
   bool show_login_screen =
       (first_screen_name.empty() && StartupUtils::IsOobeCompleted()) ||
       first_screen_name == WizardController::kLoginScreenName;
 
   if (show_login_screen) {
     display_host->StartSignInScreen(LoginScreenContext());
     return;
   }
@@ skipping 40 matching lines @@
 
   locale_util::SwitchLanguageCallback callback(
       base::Bind(&OnLanguageSwitchedCallback, base::Passed(data.Pass())));
 
   // Load locale keyboards here. Hardware layout would be automatically enabled.
   locale_util::SwitchLanguage(
       locale, true, true /* login_layouts_only */, callback);
 }
 
 }  // namespace chromeos