Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(932)

Issue 15067011: As a part of switch back to GData WAPI, update DB version. (Closed)

Created:
7 years, 7 months ago by hidehiko
Modified:
7 years, 7 months ago
Reviewers:
satorux1, kinaba
CC:
chromium-reviews, tfarina, oshima+watch_chromium.org, stevenjb+watch_chromium.org, nkostylev+watch_chromium.org, davemoore+watch_chromium.org
Visibility:
Public.

Description

As a part of switch back to GData WAPI, update DB version. Because GData WAPI and Drive API v2 has different resource id format, also the root folder id is different. So, to switch back to GData WAPI, it is needed to refresh the user's local metadata. BUG=239729 TEST=Ran unit_tests R=kinaba@chromium.org, satorux@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=199664

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M chrome/browser/chromeos/drive/resource_metadata_storage.h View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 6 (0 generated)
hidehiko
7 years, 7 months ago (2013-05-13 03:53:59 UTC) #1
hidehiko
Thank you for your review in advance, - hidehiko
7 years, 7 months ago (2013-05-13 03:54:14 UTC) #2
kinaba
lgtm
7 years, 7 months ago (2013-05-13 03:54:52 UTC) #3
satorux1
lgtm. thank you for catching this.
7 years, 7 months ago (2013-05-13 04:23:45 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/hidehiko@chromium.org/15067011/1
7 years, 7 months ago (2013-05-13 04:24:19 UTC) #5
hidehiko
7 years, 7 months ago (2013-05-13 05:51:53 UTC) #6
Message was sent while issue was closed.
Committed patchset #1 manually as r199664 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698