Fix queued writes in DeviceSettingsProvider.

chrome/browser/chromeos/settings/device_settings_provider_unittest.cc

Index: chrome/browser/chromeos/settings/device_settings_provider_unittest.cc
diff --git a/chrome/browser/chromeos/settings/device_settings_provider_unittest.cc b/chrome/browser/chromeos/settings/device_settings_provider_unittest.cc
index 3835748c2bf5d424dfcabf0d4de1dbdb095cf67f..3571f53535cd1402dd4e248b42554cf2d51758ba 100644
--- a/chrome/browser/chromeos/settings/device_settings_provider_unittest.cc
+++ b/chrome/browser/chromeos/settings/device_settings_provider_unittest.cc
@@ -204,6 +204,30 @@ TEST_F(DeviceSettingsProviderTest, SetPrefSucceed) {
   EXPECT_TRUE(bool_value);
 }
 
+TEST_F(DeviceSettingsProviderTest, SetPrefTwice) {
+  owner_key_util_->SetPrivateKey(policy_.signing_key());
+  device_settings_service_.SetUsername(policy_.policy_data().username());
+  device_settings_test_helper_.Flush();
+
+  EXPECT_CALL(*this, SettingChanged(_)).Times(AnyNumber());
+  EXPECT_CALL(*this, SettingChanged(_)).Times(AnyNumber());

[pastarmovj, 2012/09/20 08:30:07]

If you expect any number why do you have this line twice? I guess this is meant
to be Times(2) in a single line?

[Mattias Nissler (ping if slow), 2012/09/20 09:47:02]

I intended to have only one line :) Fixed.

I don't think we should force exactly two updates, I think either behavior (one
or two or many updates) are fine.

+
+  base::StringValue value1("beta");
+  provider_->Set(kReleaseChannel, value1);
+  base::StringValue value2("dev");
+  provider_->Set(kReleaseChannel, value2);
+
+  // Let the changes propagate through the system.
+  device_settings_test_helper_.set_policy_blob(std::string());
+  device_settings_test_helper_.Flush();
+
+  // Verify the second change has been applied.
+  const base::Value* saved_value = provider_->Get(kReleaseChannel);
+  EXPECT_TRUE(value2.Equals(saved_value));
+
+  Mock::VerifyAndClearExpectations(this);
+}
+
 TEST_F(DeviceSettingsProviderTest, PolicyRetrievalFailedBadSignature) {
   owner_key_util_->SetPublicKeyFromPrivateKey(policy_.signing_key());
   policy_.policy().set_policy_data_signature("bad signature");