chromeos: Use WriteBatch to update DriveResourceMetadata DB

chromeos: Use WriteBatch to update DriveResourceMetadata DB

DriveResourceMetadataStorage::PutEntry/RemoveEntry is responsible to manage parent-child relationship.
Rename DriveResourceMetadata::RemoveDirectoryChild to RemoveEntryRecursively.
Return error from DriveResourceMetadata::AddEntryToDirectory/RemoveEntryRecursively.
DriveResourceMetadata::AddEntryToDirectory is responsible to remove an entry from its old parent.

BUG=234110
TEST=unit_tests

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=196119

[hashimoto, 2013-04-24 04:01:54]

Please review.
This change depends on https://chromiumcodereview.appspot.com/14091012/
("Rewrite DriveResourceMetadata::MoveEntryToDirectory with RefreshEntry").

[hidehiko, 2013-04-24 07:29:45]

https://chromiumcodereview.appspot.com/14108009/diff/6001/chrome/browser/chro...
File chrome/browser/chromeos/drive/drive_resource_metadata.cc (right):

https://chromiumcodereview.appspot.com/14108009/diff/6001/chrome/browser/chro...
chrome/browser/chromeos/drive/drive_resource_metadata.cc:911: bool
DriveResourceMetadata::AddEntryToDirectory(const DriveEntryProto& entry) {
AddEntryToDirectory sounds weird because this doesn't just add an entry to its
parent directory, but it also removes the entry from its old parent now.

[hashimoto, 2013-04-24 08:05:56]

https://codereview.chromium.org/14108009/diff/6001/chrome/browser/chromeos/dr...
File chrome/browser/chromeos/drive/drive_resource_metadata.cc (right):

https://codereview.chromium.org/14108009/diff/6001/chrome/browser/chromeos/dr...
chrome/browser/chromeos/drive/drive_resource_metadata.cc:911: bool
DriveResourceMetadata::AddEntryToDirectory(const DriveEntryProto& entry) {
On 2013/04/24 07:29:45, hidehiko wrote:
> AddEntryToDirectory sounds weird because this doesn't just add an entry to its
> parent directory, but it also removes the entry from its old parent now.

Renamed to PutEntryUnderDirectory.

[hidehiko, 2013-04-24 08:18:53]

lgtm

[commit-bot: I haz the power, 2013-04-24 09:17:32]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/hashimoto@chromium.org/14108009/15001

[commit-bot: I haz the power, 2013-04-24 11:41:59]

Change committed as 196119