[Drive] Fetch id field of the parents in FileResource.

[Drive] Fetch id field of the parents in FileResource.

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

[tzik, 2014-06-02 08:08:12]

PTL

[hashimoto, 2014-06-02 08:09:52]

https://codereview.chromium.org/306073007/diff/1/chrome/browser/drive/drive_a...
File chrome/browser/drive/drive_api_service.cc (right):

https://codereview.chromium.org/306073007/diff/1/chrome/browser/drive/drive_a...
chrome/browser/drive/drive_api_service.cc:117:
"parents/parentLink,alternateLink,"
Please add "id" here too.

https://codereview.chromium.org/306073007/diff/1/chrome/browser/drive/drive_a...
chrome/browser/drive/drive_api_service.cc:123:
"parents/parentLink,alternateLink,modifiedDate,lastViewedByMeDate,shared),"
ditto.

[tzik, 2014-06-02 08:13:00]

https://codereview.chromium.org/306073007/diff/1/chrome/browser/drive/drive_a...
File chrome/browser/drive/drive_api_service.cc (right):

https://codereview.chromium.org/306073007/diff/1/chrome/browser/drive/drive_a...
chrome/browser/drive/drive_api_service.cc:117:
"parents/parentLink,alternateLink,"
On 2014/06/02 08:09:53, hashimoto wrote:
> Please add "id" here too.

Done.

https://codereview.chromium.org/306073007/diff/1/chrome/browser/drive/drive_a...
chrome/browser/drive/drive_api_service.cc:123:
"parents/parentLink,alternateLink,modifiedDate,lastViewedByMeDate,shared),"
On 2014/06/02 08:09:53, hashimoto wrote:
> ditto.

Done.

[hashimoto, 2014-06-02 08:13:37]

lgtm

[tzik, 2014-06-02 08:14:33]

The CQ bit was checked by tzik@chromium.org

[commit-bot: I haz the power, 2014-06-02 08:14:55]

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

[tzik, 2014-06-02 08:22:23]

The CQ bit was unchecked by tzik@chromium.org

[tzik, 2014-06-02 08:30:40]

Fixed |fields| parameter syntax.
Could you take another look?

[hashimoto, 2014-06-02 08:32:11]

sorry, I should have noticed it.
lgtm

[tzik, 2014-06-02 08:34:38]

The CQ bit was checked by tzik@chromium.org

[tzik, 2014-06-02 08:34:39]

The CQ bit was unchecked by tzik@chromium.org

[tzik, 2014-06-02 08:35:07]

The CQ bit was checked by tzik@chromium.org

[commit-bot: I haz the power, 2014-06-02 08:35:19]

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

[tzik, 2014-06-02 08:36:03]

The CQ bit was unchecked by tzik@chromium.org

[tzik, 2014-06-02 15:25:26]

The CQ bit was checked by tzik@chromium.org

[commit-bot: I haz the power, 2014-06-02 15:25:47]

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

[commit-bot: I haz the power, 2014-06-02 18:23:23]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  android_chromium_gn_compile_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_chromium_gn_c...)
  android_dbg_triggered_tests on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_dbg_triggered...)
  chromium_presubmit on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/chromium_presubmit/bu...)
  linux_chromium_chromeos_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_chrome...)
  linux_chromium_clang_dbg on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_clang_...)
  linux_chromium_gn_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_gn_rel...)
  linux_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_rel/bu...)

[commit-bot: I haz the power, 2014-06-02 18:27:12]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-06-02 18:27:12]

Try jobs failed on following builders:
  android_dbg_triggered_tests on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_dbg_triggered...)
  chromium_presubmit on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/chromium_presubmit/bu...)
  linux_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_rel/bu...)

[tzik, 2014-06-03 04:27:28]

The CQ bit was checked by tzik@chromium.org

[commit-bot: I haz the power, 2014-06-03 04:30:02]

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

[commit-bot: I haz the power, 2014-06-03 08:55:16]

Change committed as 274437