drive: Stop using DictionaryValue to store data in FakeDriveService

drive: Stop using DictionaryValue to store data in FakeDriveService


Replace the giant DictionaryValue with std::map.
No behavior change is intended.

Remove unused FakeDriveService::FindEntryByContentUrl().
Fix util::ConvertResourceEntryToFileResource() to set EntryKind of hosted documents correctly.
Stop depending on the results' order in SearchOperationTest.

BUG=334544
TEST=unit_tests

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

[hashimoto, 2014-01-15 06:53:32]

Sorry for sending this large CL.

This change does not intend to introduce any behavior change.
Most changes are just indent fix or rewording.

[hashimoto, 2014-01-16 08:25:05]

Splitted SearchOperationTest part to https://codereview.chromium.org/137533009/
which also includes a fix for browser_tests.

Also, https://codereview.chromium.org/139153003/ should fix the test failure in
LocalToRemoteSyncerTest.

[kinaba, 2014-01-17 04:52:13]

lgtm assuming all the tests indeed pass.

[commit-bot: I haz the power, 2014-01-17 07:39:50]

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

[commit-bot: I haz the power, 2014-01-17 10:27:47]

Change committed as 245487