[SyncFS] Move FakeDriveServiceHelper from V1 directory to V2.

[SyncFS] Move FakeDriveServiceHelper from V1 directory to V2.

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

[tzik, 2014-01-07 05:28:16]

PTL.

[nhiroki, 2014-01-07 05:31:45]

lgtm

[kinuko, 2014-01-07 05:45:50]

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

https://codereview.chromium.org/108343009/diff/1/chrome/browser/sync_file_sys...
chrome/browser/sync_file_system/drive_backend/fake_drive_service_helper.cc:167:
APIUtil::GetSyncRootDirectoryName(), std::string(),
Can you make this not rely on v1 APIUtil?  (It's just a thin static wrapper
method, should be easily moved out of api_util.{h,cc})

[tzik, 2014-01-07 06:38:16]

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

https://codereview.chromium.org/108343009/diff/1/chrome/browser/sync_file_sys...
chrome/browser/sync_file_system/drive_backend/fake_drive_service_helper.cc:167:
APIUtil::GetSyncRootDirectoryName(), std::string(),
On 2014/01/07 05:45:51, kinuko wrote:
> Can you make this not rely on v1 APIUtil?  (It's just a thin static wrapper
> method, should be easily moved out of api_util.{h,cc})

I think it's worth separating to CL since the implementation of the function is
simpler in V2:
https://codereview.chromium.org/103013003/

[tzik, 2014-01-08 14:43:01]

PTAL?

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

https://codereview.chromium.org/108343009/diff/1/chrome/browser/sync_file_sys...
chrome/browser/sync_file_system/drive_backend/fake_drive_service_helper.cc:167:
APIUtil::GetSyncRootDirectoryName(), std::string(),
On 2014/01/07 06:38:18, tzik wrote:
> On 2014/01/07 05:45:51, kinuko wrote:
> > Can you make this not rely on v1 APIUtil?  (It's just a thin static wrapper
> > method, should be easily moved out of api_util.{h,cc})
> 
> I think it's worth separating to CL since the implementation of the function
is
> simpler in V2:
> https://codereview.chromium.org/103013003/

Done in another CL.

[kinuko, 2014-01-08 23:50:20]

On 2014/01/08 14:43:01, tzik wrote:
> PTAL?
> 
>
https://codereview.chromium.org/108343009/diff/1/chrome/browser/sync_file_sys...
> File
chrome/browser/sync_file_system/drive_backend/fake_drive_service_helper.cc
> (right):
> 
>
https://codereview.chromium.org/108343009/diff/1/chrome/browser/sync_file_sys...
>
chrome/browser/sync_file_system/drive_backend/fake_drive_service_helper.cc:167:
> APIUtil::GetSyncRootDirectoryName(), std::string(),
> On 2014/01/07 06:38:18, tzik wrote:
> > On 2014/01/07 05:45:51, kinuko wrote:
> > > Can you make this not rely on v1 APIUtil?  (It's just a thin static
wrapper
> > > method, should be easily moved out of api_util.{h,cc})
> > 
> > I think it's worth separating to CL since the implementation of the function
> is
> > simpler in V2:
> > https://codereview.chromium.org/103013003/
> 
> Done in another CL.

lgtm

[commit-bot: I haz the power, 2014-01-09 01:39:46]

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

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

Change committed as 243752