gdata: Fix a bug in GDataLevelDB where errors are not handled right

chrome/browser/chromeos/gdata/gdata_leveldb.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/chromeos/gdata/gdata_leveldb.h"
 
 #include <string>
 
 #include "base/logging.h"
 #include "base/threading/thread_restrictions.h"
@@ skipping 121 matching lines @@
   const std::string resource_id_key = ResourceIdToKey(resource_id);
   const leveldb::Status db_status = level_db_->Get(leveldb::ReadOptions(),
       leveldb::Slice(resource_id_key), &serialized_proto);
 
   if (db_status.IsNotFound())
     return DB_KEY_NOT_FOUND;
 
   if (db_status.ok()) {
     DCHECK(!serialized_proto.empty());
     *entry = GDataEntry::FromProtoString(serialized_proto);
-    DCHECK(entry->get());
-    return DB_OK;
+    return entry->get() ? DB_OK : DB_CORRUPTION;
   }
   return GetStatus(db_status);
 }
 
 GDataDB::Status GDataLevelDB::GetByPath(const FilePath& path,
                                         scoped_ptr<GDataEntry>* entry) {
   base::ThreadRestrictions::AssertIOAllowed();
 
   entry->reset();
   std::string resource_id;
@@ skipping 15 matching lines @@
 }
 
 scoped_ptr<GDataDBIter> GDataLevelDB::CreateIterator(const FilePath& path) {
   return scoped_ptr<GDataDBIter>(new GDataLevelDBIter(
       scoped_ptr<leveldb::Iterator>(
           level_db_->NewIterator(leveldb::ReadOptions())),
       this,
       path));
 }
 
+GDataDB::Status GDataLevelDB::PutRawForTesting(
+    const std::string& resource_id,
+    const std::string& raw_value) {
+  const std::string resource_id_key = ResourceIdToKey(resource_id);
+  leveldb::Status db_status = level_db_->Put(
+      leveldb::WriteOptions(),
+      leveldb::Slice(resource_id_key),
+      leveldb::Slice(raw_value));
+
+  return GetStatus(db_status);
+}
+
 GDataLevelDBIter::GDataLevelDBIter(scoped_ptr<leveldb::Iterator> level_db_iter,
                                    GDataDB* db,
                                    const FilePath& path)
     : level_db_iter_(level_db_iter.Pass()),
       db_(db),
       path_(path) {
   base::ThreadRestrictions::AssertIOAllowed();
 
   const std::string path_key = PathToKey(path);
   level_db_iter_->Seek(leveldb::Slice(path_key));
@@ skipping 15 matching lines @@
     return false;
 
   // Only consider keys under |path|.
   const std::string path_key = PathToKey(path_);
   leveldb::Slice key_slice(level_db_iter_->key());
   if (!key_slice.starts_with(path_key))
     return false;
 
   GDataDB::Status status =
       db_->GetByResourceId(level_db_iter_->value().ToString(), entry);
-  DCHECK_EQ(GDataDB::DB_OK, status);
+  if (status != GDataDB::DB_OK)
+    return false;
 
   key_slice.remove_prefix(sizeof(kPathPrefix) - 1);
   path->assign(key_slice.ToString());
 
   level_db_iter_->Next();
   return true;
 }
 
 }  // namespace gdata