Propagate more information about ValueStore errors to callers, notably an

chrome/browser/value_store/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/value_store.h"
 
 #include "base/logging.h"
 
+// Implementation of Error.
+
+ValueStore::Error::Error(ErrorCode code,
+                         const std::string& message,
+                         scoped_ptr<std::string> key)
+    : code(code), message(message), key(key.Pass()) {}
+
+ValueStore::Error::~Error() {}
+
 // Implementation of ReadResultType.
 
-ValueStore::ReadResultType::ReadResultType(DictionaryValue* settings)
-    : settings_(settings) {
-  DCHECK(settings);
+ValueStore::ReadResultType::ReadResultType(scoped_ptr<DictionaryValue> settings)
+    : settings_(settings.Pass()) {
+  CHECK(settings_);
 }
 
-ValueStore::ReadResultType::ReadResultType(const std::string& error)
-    : error_(error) {
-  DCHECK(!error.empty());
+ValueStore::ReadResultType::ReadResultType(scoped_ptr<Error> error)
+    : error_(error.Pass()) {
+  CHECK(error_);
 }
 
 ValueStore::ReadResultType::~ReadResultType() {}
 
-bool ValueStore::ReadResultType::HasError() const {
-  return !error_.empty();
+// Implementation of WriteResultType.
+
+ValueStore::WriteResultType::WriteResultType(
+    scoped_ptr<ValueStoreChangeList> changes)
+    : changes_(changes.Pass()) {
+  CHECK(changes_);
 }
 
-scoped_ptr<DictionaryValue>& ValueStore::ReadResultType::settings() {
-  DCHECK(!HasError());
-  return settings_;
-}
-
-const std::string& ValueStore::ReadResultType::error() const {
-  DCHECK(HasError());
-  return error_;
-}
-
-// Implementation of WriteResultType.
-
-ValueStore::WriteResultType::WriteResultType(ValueStoreChangeList* changes)
-    : changes_(changes) {
-  DCHECK(changes);
-}
-
-ValueStore::WriteResultType::WriteResultType(const std::string& error)
-    : error_(error) {
-  DCHECK(!error.empty());
+ValueStore::WriteResultType::WriteResultType(scoped_ptr<Error> error)
+    : error_(error.Pass()) {
+  CHECK(error_);
 }
 
 ValueStore::WriteResultType::~WriteResultType() {}
-
-bool ValueStore::WriteResultType::HasError() const {
-  return !error_.empty();
-}
-
-const ValueStoreChangeList& ValueStore::WriteResultType::changes() const {
-  DCHECK(!HasError());
-  return *changes_;
-}
-
-const std::string& ValueStore::WriteResultType::error() const {
-  DCHECK(HasError());
-  return error_;
-}