Cache force-installed apps/extensions in device-local accounts

chrome/browser/chromeos/settings/device_settings_test_helper.cc

Index: chrome/browser/chromeos/settings/device_settings_test_helper.cc
diff --git a/chrome/browser/chromeos/settings/device_settings_test_helper.cc b/chrome/browser/chromeos/settings/device_settings_test_helper.cc
index 04a77ea81c1245b2705c8c8cd752645fe4bab3d3..2050364b72064b4c1d7997d7a214be072fc744fc 100644
--- a/chrome/browser/chromeos/settings/device_settings_test_helper.cc
+++ b/chrome/browser/chromeos/settings/device_settings_test_helper.cc
@@ -63,6 +63,7 @@ void DeviceSettingsTestHelper::FlushRetrieve() {
   for (device_local_account_state = device_local_account_policy_.begin();
        device_local_account_state != device_local_account_policy_.end();
        ++device_local_account_state) {
+    std::vector<RetrievePolicyCallback> callbacks;

[Joao da Silva, 2013/10/17 14:57:54]

Another nice catch :-)

Another option here: callbacks.clear()

[bartfab (slow), 2013/10/18 12:58:39]

I prefer creating a new object as it conveys best what this is - a brand new
list, starting out blank.

     callbacks.swap(device_local_account_state->second.retrieve_callbacks_);
     for (std::vector<RetrievePolicyCallback>::iterator cb(callbacks.begin());
          cb != callbacks.end(); ++cb) {