google_apis: Clean up gdata_wapi_operations.h

google_apis: Clean up gdata_wapi_operations.h

- Add and fix comments
- Make some members private
- Make some members const
- Give better names to parameters
  (in particular document_url -> content_url/edit_url)

BUG=163544, 163558
TEST=none



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

[kinaba, 2012-11-30 09:57:25]

This is great. LGTM.

https://codereview.chromium.org/11419253/diff/2001/chrome/browser/google_apis...
File chrome/browser/google_apis/gdata_wapi_operations.h (right):

https://codereview.chromium.org/11419253/diff/2001/chrome/browser/google_apis...
chrome/browser/google_apis/gdata_wapi_operations.h:499: // If upload is
complete, |code| is set to HTTP_CREATED for a file file, or
a file file => a new file

[kinaba, 2012-11-30 10:01:15]

...LG except the build break :p,

[satorux1, 2012-11-30 10:04:08]

oops, let me fix and re-upload...

[kinaba, 2012-11-30 10:09:00]

lgtm

[satorux1, 2012-11-30 10:09:10]

fixed the build too.

https://codereview.chromium.org/11419253/diff/2001/chrome/browser/google_apis...
File chrome/browser/google_apis/gdata_wapi_operations.h (right):

https://codereview.chromium.org/11419253/diff/2001/chrome/browser/google_apis...
chrome/browser/google_apis/gdata_wapi_operations.h:499: // If upload is
complete, |code| is set to HTTP_CREATED for a file file, or
On 2012/11/30 09:57:26, kinaba wrote:
> a file file => a new file

Done.

[commit-bot: I haz the power, 2012-11-30 13:30:10]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/satorux@chromium.org/11419253/8001

[commit-bot: I haz the power, 2012-11-30 17:05:01]

Change committed as 170466