[SyncFS] Drop test only un-thread-safe getters from SyncEngine (4)

[SyncFS] Drop test only un-thread-safe getters from SyncEngine

BUG=347425, 380615
NOTRY=true

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

[tzik, 2014-06-09 13:25:15]

PTL

[peria, 2014-06-10 03:18:15]

https://codereview.chromium.org/322813002/diff/1/chrome/browser/sync_file_sys...
File
chrome/browser/sync_file_system/drive_backend/drive_backend_sync_unittest.cc
(right):

https://codereview.chromium.org/322813002/diff/1/chrome/browser/sync_file_sys...
chrome/browser/sync_file_system/drive_backend/drive_backend_sync_unittest.cc:574:
SyncEngineContext* context() {
This method is used only in metadata_database().
Would you merge them?

https://codereview.chromium.org/322813002/diff/1/chrome/browser/sync_file_sys...
chrome/browser/sync_file_system/drive_backend/drive_backend_sync_unittest.cc:580:
MetadataDatabase* metadata_database() {
This change seems to be independent from the objective of this CL.
Could you make another CL for this?

[tzik, 2014-06-10 03:53:23]

https://codereview.chromium.org/322813002/diff/1/chrome/browser/sync_file_sys...
File
chrome/browser/sync_file_system/drive_backend/drive_backend_sync_unittest.cc
(right):

https://codereview.chromium.org/322813002/diff/1/chrome/browser/sync_file_sys...
chrome/browser/sync_file_system/drive_backend/drive_backend_sync_unittest.cc:574:
SyncEngineContext* context() {
On 2014/06/10 03:18:15, peria wrote:
> This method is used only in metadata_database().
> Would you merge them?

Done.

https://codereview.chromium.org/322813002/diff/1/chrome/browser/sync_file_sys...
chrome/browser/sync_file_system/drive_backend/drive_backend_sync_unittest.cc:580:
MetadataDatabase* metadata_database() {
On 2014/06/10 03:18:15, peria wrote:
> This change seems to be independent from the objective of this CL.
> Could you make another CL for this?

Done.

[peria, 2014-06-10 03:56:02]

lgtm

[tzik, 2014-06-10 06:11:54]

The CQ bit was checked by tzik@chromium.org

[commit-bot: I haz the power, 2014-06-10 06:16:00]

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

[commit-bot: I haz the power, 2014-06-10 06:19:34]

Change committed as 275971