Add Drive API specific operations.

Add Drive API specific operations.

Future CLs will use these operations.

BUG=chromium:127728
TEST=none
TBR=ben@chromium.org

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

[kochi, 2012-07-27 06:34:22]

Ryo, could you review?

[hashimoto, 2012-07-27 06:48:37]

http://codereview.chromium.org/10821065/diff/2001/chrome/browser/chromeos/gda...
File chrome/browser/chromeos/gdata/drive_api_operations.cc (right):

http://codereview.chromium.org/10821065/diff/2001/chrome/browser/chromeos/gda...
chrome/browser/chromeos/gdata/drive_api_operations.cc:38: :
GetDataOperation(registry, profile, callback) {}
nit: Please insert a newline here, or remove the new line in
GetAboutOperation::GetAboutOperation.
Please be consistent.

http://codereview.chromium.org/10821065/diff/2001/chrome/browser/chromeos/gda...
File chrome/browser/chromeos/gdata/operations_base.cc (right):

http://codereview.chromium.org/10821065/diff/2001/chrome/browser/chromeos/gda...
chrome/browser/chromeos/gdata/operations_base.cc:50: const char
kDriveFileScope[] = "https://www.googleapis.com/auth/drive.file";
What is this for?

[kochi, 2012-07-27 06:56:58]

http://codereview.chromium.org/10821065/diff/2001/chrome/browser/chromeos/gda...
File chrome/browser/chromeos/gdata/drive_api_operations.cc (right):

http://codereview.chromium.org/10821065/diff/2001/chrome/browser/chromeos/gda...
chrome/browser/chromeos/gdata/drive_api_operations.cc:38: :
GetDataOperation(registry, profile, callback) {}
On 2012/07/27 06:48:37, hashimoto wrote:
> nit: Please insert a newline here, or remove the new line in
> GetAboutOperation::GetAboutOperation.
> Please be consistent.

Done.

http://codereview.chromium.org/10821065/diff/2001/chrome/browser/chromeos/gda...
File chrome/browser/chromeos/gdata/operations_base.cc (right):

http://codereview.chromium.org/10821065/diff/2001/chrome/browser/chromeos/gda...
chrome/browser/chromeos/gdata/operations_base.cc:50: const char
kDriveFileScope[] = "https://www.googleapis.com/auth/drive.file";
This will be used for file access for Drive v2 API.
So this is not used in this change.  Removed.

Currently https://docs.google.com/feeds/ will have the same
authority as drive.file scope, so it is not necessary anyway
for a while.

[hashimoto, 2012-07-27 06:59:53]

lgtm

[kochi, 2012-07-27 07:05:55]

On 2012/07/27 06:59:53, hashimoto wrote:
> lgtm

Thanks, added TBR=ben@chromium.org for chrome/chrome_browser.gypi

[Ben Goodger (Google), 2012-07-27 17:12:53]

lgtm