Delay the metrics policy migration call to make sure ownership has been taken.

chrome/browser/chromeos/user_cros_settings_provider.cc

 // Copyright (c) 2011 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/user_cros_settings_provider.h"
 
 #include <map>
 #include <set>
 
 #include "base/bind.h"
@@ skipping 45 matching lines @@
 
 const char* kListSettings[] = {
   kAccountsPrefUsers
 };
 
 // Only write the property if the owner is the current logged on user.
 void StartStorePropertyOpIfOwner(const std::string& name,
                                  const std::string& value,
                                  SignedSettingsHelper::Callback* callback) {
   if (OwnershipService::GetSharedInstance()->CurrentUserIsOwner()) {
-    BrowserThread::PostTask(BrowserThread::UI,
+    // Delay this task by 30 seconds to ensure the ownership has been taken.

[Mattias Nissler (ping if slow), 2011/08/29 10:55:26]

If you really want to do this, then please put the delay into the other PostTask
call that schedules StartStorePropertyOpIfOwner().

+    BrowserThread::PostDelayedTask(BrowserThread::UI,
                             FROM_HERE,
                             base::Bind(
                                 &SignedSettingsHelper::StartStorePropertyOp,
                                 base::Unretained(SignedSettingsHelper::Get()),
                                 name,
                                 value,
-                                callback));
+                                callback),
+                            1000 * 30);

[Mattias Nissler (ping if slow), 2011/08/29 10:55:26]

declare a constant?

   }
 }
 
 bool IsControlledBooleanSetting(const std::string& pref_path) {
   // TODO(nkostylev): Using std::find for 4 value array generates this warning
   // in chroot stl_algo.h:231: error: array subscript is above array bounds.
   // GCC 4.4.3
   return (pref_path == kAccountsPrefAllowNewUser) ||
          (pref_path == kAccountsPrefAllowGuest) ||
          (pref_path == kAccountsPrefShowUserNamesOnSignIn) ||
@@ skipping 614 matching lines @@
   if (cached_whitelist_update->Remove(email_value, NULL))
     prefs->ScheduleSavePersistentPrefs();
 }
 
 // static
 void UserCrosSettingsProvider::UpdateCachedOwner(const std::string& email) {
   UpdateCacheString(kDeviceOwner, email, USE_VALUE_SUPPLIED);
 }
 
 }  // namespace chromeos