[SyncFS] Introduce counters in LevelDBWrapper

[SyncFS] Introduce counters in LevelDBWrapper to count the number
of pending transactions for PUT/DELETE operations independently.
It means that both counters counts the operation for the entry, if an entry is put and deleted.


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

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

[peria, 2014-08-14 05:44:28]

PTL.

This CL will help http://crrev.com/455263003

[tzik, 2014-08-14 05:52:28]

lgtm

[peria, 2014-08-14 08:50:20]

Hiroki-san,
Would you take a look, if you have time?
It is OK to leave it for few days.

[nhiroki, 2014-08-14 08:59:31]

LGTM!

https://codereview.chromium.org/466373004/diff/20001/chrome/browser/sync_file...
File chrome/browser/sync_file_system/drive_backend/leveldb_wrapper.h (right):

https://codereview.chromium.org/466373004/diff/20001/chrome/browser/sync_file...
chrome/browser/sync_file_system/drive_backend/leveldb_wrapper.h:92: // Return
the number of pending PUT/DELETE operations.
nit: s/Return/Returns/

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

https://codereview.chromium.org/466373004/diff/20001/chrome/browser/sync_file...
chrome/browser/sync_file_system/drive_backend/leveldb_wrapper_unittest.cc:158: 
nit: extra blank line

https://codereview.chromium.org/466373004/diff/20001/chrome/browser/sync_file...
chrome/browser/sync_file_system/drive_backend/leveldb_wrapper_unittest.cc:178: 
nit: ditto.

[peria, 2014-08-14 09:01:53]

Thanks!

https://codereview.chromium.org/466373004/diff/20001/chrome/browser/sync_file...
File chrome/browser/sync_file_system/drive_backend/leveldb_wrapper.h (right):

https://codereview.chromium.org/466373004/diff/20001/chrome/browser/sync_file...
chrome/browser/sync_file_system/drive_backend/leveldb_wrapper.h:92: // Return
the number of pending PUT/DELETE operations.
On 2014/08/14 08:59:30, nhiroki wrote:
> nit: s/Return/Returns/

Done.

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

https://codereview.chromium.org/466373004/diff/20001/chrome/browser/sync_file...
chrome/browser/sync_file_system/drive_backend/leveldb_wrapper_unittest.cc:158: 
On 2014/08/14 08:59:30, nhiroki wrote:
> nit: extra blank line

Done.

https://codereview.chromium.org/466373004/diff/20001/chrome/browser/sync_file...
chrome/browser/sync_file_system/drive_backend/leveldb_wrapper_unittest.cc:178: 
On 2014/08/14 08:59:30, nhiroki wrote:
> nit: ditto.

Done.

[peria, 2014-08-14 09:02:01]

The CQ bit was checked by peria@chromium.org

[commit-bot: I haz the power, 2014-08-14 09:03:48]

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

[commit-bot: I haz the power, 2014-08-14 15:21:46]

Committed patchset #2 (40001) as 289595