Fixed check return warnings from Coverity.

chrome/browser/value_store/testing_value_store.cc

 // Copyright (c) 2012 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/value_store/testing_value_store.h"
 
 #include "base/logging.h"
 
 namespace {
 
@@ skipping 80 matching lines @@
 
 ValueStore::WriteResult TestingValueStore::Set(
     WriteOptions options, const DictionaryValue& settings) {
   if (fail_all_requests_) {
     return WriteResultError();
   }
 
   scoped_ptr<ValueStoreChangeList> changes(new ValueStoreChangeList());
   for (DictionaryValue::Iterator it(settings); it.HasNext(); it.Advance()) {
     Value* old_value = NULL;
-    storage_.GetWithoutPathExpansion(it.key(), &old_value);
-    if (!old_value || !old_value->Equals(&it.value())) {
+    if (!storage_.GetWithoutPathExpansion(it.key(), &old_value) ||
+        !old_value->Equals(&it.value())) {
       changes->push_back(
           ValueStoreChange(
               it.key(),
               old_value ? old_value->DeepCopy() : old_value,
               it.value().DeepCopy()));
       storage_.SetWithoutPathExpansion(it.key(), it.value().DeepCopy());
     }
   }
   return MakeWriteResult(changes.release());
 }
@@ skipping 24 matching lines @@
   if (fail_all_requests_) {
     return WriteResultError();
   }
 
   std::vector<std::string> keys;
   for (DictionaryValue::Iterator it(storage_); it.HasNext(); it.Advance()) {
     keys.push_back(it.key());
   }
   return Remove(keys);
 }