Added a timezone policy and pyauto tests for it.

chrome/browser/policy/browser_policy_connector.cc

Index: chrome/browser/policy/browser_policy_connector.cc
diff --git a/chrome/browser/policy/browser_policy_connector.cc b/chrome/browser/policy/browser_policy_connector.cc
index 2874a911dd55d483b43354fedfc3b51ff9cfd511..4eaadbf3434acd9f27f1d8067f7de6c91226fce9 100644
--- a/chrome/browser/policy/browser_policy_connector.cc
+++ b/chrome/browser/policy/browser_policy_connector.cc
@@ -9,7 +9,11 @@
 #include "base/command_line.h"
 #include "base/file_path.h"
 #include "base/path_service.h"
+#include "base/utf_string_conversions.h"
 #include "chrome/browser/browser_process.h"
+#include "chrome/browser/chromeos/cros_settings.h"
+#include "chrome/browser/chromeos/cros_settings_provider.h"
+#include "chrome/browser/chromeos/system/timezone_settings.h"

[Joao da Silva, 2012/07/27 13:40:57]

Move these to the #if defined(OS_CHROMEOS) below

[pneubeck (no reviews), 2012/07/31 22:21:05]

Done.

 #include "chrome/browser/policy/async_policy_provider.h"
 #include "chrome/browser/policy/cloud_policy_client.h"
 #include "chrome/browser/policy/cloud_policy_provider.h"
@@ -567,9 +571,34 @@ void BrowserPolicyConnector::CompleteInitialization() {
           g_browser_process->local_state(),
           chromeos::system::StatisticsProvider::GetInstance(),
           NULL));
+
+  SetTimezoneIfPolicyAvailable();

[Joao da Silva, 2012/07/27 13:40:57]

Notice that CompleteInitialization() is only executed if device policy is
enabled. Please do this refactorings:

- move the posting of CompleteInitialization from InitializeDevicePolicy to
Init, and make it unconditional (and execute on every platform). It should be
the last thing done in Init.

- move the posting of GetAppPackUpdater from Init to CompleteInitialization

[pneubeck (no reviews), 2012/07/31 22:21:05]

The only thing, I don't like about the "execute on every platform" is, that
GetAppPackUpdater is also ChromeOS only. Thus if !chromeos we post an empty
task.
Therefore I made it conditional on the chromeos platform.

Is that ok?

 #endif
 }
 
+#if defined(OS_CHROMEOS)

[Joao da Silva, 2012/07/27 13:40:57]

Move the #ifdef into the method, and make the method available on every platform
i.e. remove the #ifdef from the header. See InitializeDevicePolicy for an
example.

[pneubeck (no reviews), 2012/07/31 22:21:05]

Done.

+void BrowserPolicyConnector::SetTimezoneIfPolicyAvailable() {
+  typedef chromeos::CrosSettingsProvider Provider;
+  Provider::TrustedStatus result =
+      chromeos::CrosSettings::Get()->PrepareTrustedValues(
+          base::Bind(&BrowserPolicyConnector::SetTimezoneIfPolicyAvailable,
+                     weak_ptr_factory_.GetWeakPtr()));
+
+  if (result == Provider::TEMPORARILY_UNTRUSTED ||
+     result == Provider::PERMANENTLY_UNTRUSTED)

[Joao da Silva, 2012/07/27 13:40:57]

Nit: indent of the 2nd line
Nit: braces when the conditional or the body span more than 1 line

I'd rather write this as

if (result != Provider::TRUSTED)
  return;

[pneubeck (no reviews), 2012/07/31 22:21:05]

Done.

+    return;
+
+  DCHECK_EQ(result, Provider::TRUSTED);
+
+  std::string timezone;
+  if (chromeos::CrosSettings::Get()->GetString(
+          chromeos::kSystemTimezonePolicy, &timezone)) {
+    chromeos::system::TimezoneSettings::GetInstance()->SetTimezoneFromID(
+        UTF8ToUTF16(timezone));
+  }
+}
+#endif
+
 // static
 ConfigurationPolicyProvider* BrowserPolicyConnector::CreatePlatformProvider() {
 #if defined(OS_WIN)