Rename GDataSystemService to DriveSystemService

Rename GDataSystemService to DriveSystemService

In this CL, the following changes are included:
* Rename GDataSystemService class to DriveSystemService
* gdata_system_service.{h,cc} have been renamed to drive_system_service.{h,cc}

This applies on top of
https://chromiumcodereview.appspot.com/10874028/ and
https://chromiumcodereview.appspot.com/10877006/

BUG=137151
TEST=pass all existing tests.
TBR=satorux@chromium.org, ben@chromium.org, estade@chromium.org, ahendrickson@chromium.org

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

[kochi, 2012-08-23 09:42:47]

Hi Satoru, Daniel,

Could you review this?
Thanks,

As GDataContactsService is now dependent on DriveSystemService, but I think we
should have a separate
ProfilKeyedService for it.
What do you think?

[kochi, 2012-08-23 09:56:28]

Kazuhiro, could you take a look at this CL for Satoru's absence?
Thanks,

On 2012/08/23 09:42:47, Takayoshi Kochi wrote:
> Hi Satoru, Daniel,
> 
> Could you review this?
> Thanks,
> 
> As GDataContactsService is now dependent on DriveSystemService, but I think we
> should have a separate
> ProfilKeyedService for it.
> What do you think?

[kinaba, 2012-08-23 10:57:39]

LGTM with a nit for renaming, leaving GDataContactsService's dependency stuff to
other reviewers.

http://codereview.chromium.org/10873026/diff/1/chrome/browser/chromeos/gdata/...
File chrome/browser/chromeos/gdata/drive_file_system_proxy.cc (right):

http://codereview.chromium.org/10873026/diff/1/chrome/browser/chromeos/gdata/...
chrome/browser/chromeos/gdata/drive_file_system_proxy.cc:17: #include
"chrome/browser/chromeos/gdata/drive_system_service.h"
I suspect this is not used in this file. Could you check it?

[Daniel Erat, 2012-08-23 16:27:18]

I agree that GDataContactsService shouldn't be owned by a class named
DriveSystemService. :-)

I think that GDataContactsService can just be instantiated and owned by
GoogleContactStore.  I'll try making this change now.

[Daniel Erat, 2012-08-23 17:57:01]

Seems like it works.  I'm okay with GDataContactsService temporarily being owned
by a class named DriveSystemService (so you won't need to deal with a bunch of
merge conflicts).

[kochi, 2012-08-27 10:48:37]

Merged https://chromiumcodereview.appspot.com/10882017/,
added TBR=.

Satoru, could you check c/b/extensions, c/b/chromeos
Ben, could you check .gypi changes?
Evan, could you check c/b/ui/webui/?
Andy, could you check c/b/download/?

[Randy Smith (Not in Mondays), 2012-08-27 13:54:49]

c/b/download LGTM.

[Ben Goodger (Google), 2012-08-27 15:51:07]

gyp lgtm