[SyncFS] Implement getters and counters of FileMetadata and FileTracker

[SyncFS] Implement following four methods.
- CountFileMetadata
- CountFileTracker
- GetAllMeatadaIDs
- GetAllTrackerIDs


BUG=347425
TEST=./unit_tests --gtest_filter="Metadata*"

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

[peria, 2014-07-08 09:30:09]

PTL.

[tzik, 2014-07-08 10:42:37]

lgtm

[peria, 2014-07-09 01:40:36]

The CQ bit was checked by peria@chromium.org

[commit-bot: I haz the power, 2014-07-09 01:41:23]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/peria@chromium.org/372843007/20001

[nhiroki, 2014-07-09 02:13:45]

LGTM with nits (you don't have to stop the CQ)

https://codereview.chromium.org/372843007/diff/20001/chrome/browser/sync_file...
File
chrome/browser/sync_file_system/drive_backend/metadata_database_index_on_disk_unittest.cc
(right):

https://codereview.chromium.org/372843007/diff/20001/chrome/browser/sync_file...
chrome/browser/sync_file_system/drive_backend/metadata_database_index_on_disk_unittest.cc:191:
EXPECT_EQ(3U, file_ids.size());
This could be ASSERT_EQ to avoid out of range errors on line 193-195
(IndexAppRootIDByAppIDTest is using ASSERT_EQ for this purpose).

https://codereview.chromium.org/372843007/diff/20001/chrome/browser/sync_file...
chrome/browser/sync_file_system/drive_backend/metadata_database_index_on_disk_unittest.cc:199:
EXPECT_EQ(4U, tracker_ids.size());
ditto.

[peria, 2014-07-09 02:17:13]

The CQ bit was unchecked by peria@chromium.org

[peria, 2014-07-09 02:18:36]

The CQ bit was checked by peria@chromium.org

[peria, 2014-07-09 02:18:40]

https://codereview.chromium.org/372843007/diff/20001/chrome/browser/sync_file...
File
chrome/browser/sync_file_system/drive_backend/metadata_database_index_on_disk_unittest.cc
(right):

https://codereview.chromium.org/372843007/diff/20001/chrome/browser/sync_file...
chrome/browser/sync_file_system/drive_backend/metadata_database_index_on_disk_unittest.cc:191:
EXPECT_EQ(3U, file_ids.size());
On 2014/07/09 02:13:45, nhiroki wrote:
> This could be ASSERT_EQ to avoid out of range errors on line 193-195
> (IndexAppRootIDByAppIDTest is using ASSERT_EQ for this purpose).

Done.

https://codereview.chromium.org/372843007/diff/20001/chrome/browser/sync_file...
chrome/browser/sync_file_system/drive_backend/metadata_database_index_on_disk_unittest.cc:199:
EXPECT_EQ(4U, tracker_ids.size());
On 2014/07/09 02:13:45, nhiroki wrote:
> ditto.

Done.

[commit-bot: I haz the power, 2014-07-09 02:21:06]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/peria@chromium.org/372843007/40001

[peria, 2014-07-09 02:31:43]

The CQ bit was unchecked by peria@chromium.org

[peria, 2014-07-09 02:33:56]

The CQ bit was checked by peria@chromium.org

[commit-bot: I haz the power, 2014-07-09 02:34:41]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/peria@chromium.org/372843007/60001

[commit-bot: I haz the power, 2014-07-09 04:21:03]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  mac_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/mac_chromium_rel/buil...)

[commit-bot: I haz the power, 2014-07-09 06:42:00]

Change committed as 281980