gdata: Introduce GDataWapiFeedLoader class

gdata: Introduce GDataWapiFeedLoader class

This class is introduced to separate code related to WAPI
(codename of Documents List API) feed loading.

To minimize diffs, the the class is defined in gdata_file_system.cc
The code will be moved to a set of new files in a separate patch.

BUG=130669
TEST=out/Release/unit_tests --gtest_filter=GData* && out/Release/browser_tests --gtest_filter=GData*

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

[satorux1, 2012-08-02 00:55:43]

Here's another one.

[achuithb, 2012-08-02 01:38:25]

Ya, I do like it.

It's going to make my rebase difficult though :/

lgtm

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

http://codereview.chromium.org/10828126/diff/1/chrome/browser/chromeos/gdata/...
chrome/browser/chromeos/gdata/gdata_file_system.cc:801:
weak_ptr_factory_.GetWeakPtr(),
why not ui_weak_ptr_ here?

http://codereview.chromium.org/10828126/diff/1/chrome/browser/chromeos/gdata/...
chrome/browser/chromeos/gdata/gdata_file_system.cc:976: local_changestamp + 1,
0,
nit: indentation

[satorux1, 2012-08-02 07:22:51]

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

http://codereview.chromium.org/10828126/diff/1/chrome/browser/chromeos/gdata/...
chrome/browser/chromeos/gdata/gdata_file_system.cc:801:
weak_ptr_factory_.GetWeakPtr(),
On 2012/08/02 01:38:25, achuith.bhandarkar wrote:
> why not ui_weak_ptr_ here?

Changed back to ui_weak_ptr_. I tried to delete ui_weak_ptr_ in a separate
patch, but was a bad idea.

http://codereview.chromium.org/10828126/diff/1/chrome/browser/chromeos/gdata/...
chrome/browser/chromeos/gdata/gdata_file_system.cc:976: local_changestamp + 1,
0,
On 2012/08/02 01:38:25, achuith.bhandarkar wrote:
> nit: indentation

Done.