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

Issue 252563011: drive: Remove root_feed.json and others (Closed)

Created:
6 years, 8 months ago by hashimoto
Modified:
6 years, 7 months ago
Reviewers:
kinaba
CC:
chromium-reviews, extensions-reviews_chromium.org, nkostylev+watch_chromium.org, tzik, tfarina, yoshiki+watch_chromium.org, nhiroki, rginda+watch_chromium.org, oshima+watch_chromium.org, chromium-apps-reviews_chromium.org, kinuko+watch, stevenjb+watch_chromium.org, davemoore+watch_chromium.org
Visibility:
Public.

Description

drive: Remove root_feed.json and others Remove root_feed.json Remove other JSON files used in ChangeListProcessorTest BUG=357038 TEST=unit_tests Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=266567

Patch Set 1 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+199 lines, -3643 lines) Patch
M chrome/browser/chromeos/drive/change_list_processor_unittest.cc View 14 chunks +184 lines, -233 lines 0 comments Download
M chrome/browser/chromeos/drive/file_system/copy_operation_unittest.cc View 2 chunks +2 lines, -2 lines 0 comments Download
M chrome/browser/chromeos/drive/file_system_unittest.cc View 1 chunk +12 lines, -10 lines 0 comments Download
M chrome/browser/drive/fake_drive_service_unittest.cc View 1 chunk +0 lines, -1 line 0 comments Download
M chrome/test/data/extensions/api_test/file_browser/drive_search_test/test.js View 1 chunk +1 line, -2 lines 0 comments Download
D chrome/test/data/gdata/delta_dir_moved_from_root_to_directory.json View 1 chunk +0 lines, -230 lines 0 comments Download
D chrome/test/data/gdata/delta_file_added_in_directory.json View 1 chunk +0 lines, -249 lines 0 comments Download
D chrome/test/data/gdata/delta_file_added_in_new_but_deleted_directory.json View 1 chunk +0 lines, -256 lines 0 comments Download
D chrome/test/data/gdata/delta_file_added_in_new_directory.json View 1 chunk +0 lines, -249 lines 0 comments Download
D chrome/test/data/gdata/delta_file_added_in_root.json View 1 chunk +0 lines, -163 lines 0 comments Download
D chrome/test/data/gdata/delta_file_deleted_in_directory.json View 1 chunk +0 lines, -168 lines 0 comments Download
D chrome/test/data/gdata/delta_file_deleted_in_root.json View 1 chunk +0 lines, -166 lines 0 comments Download
D chrome/test/data/gdata/delta_file_moved_from_directory_to_root.json View 1 chunk +0 lines, -238 lines 0 comments Download
D chrome/test/data/gdata/delta_file_renamed_in_directory.json View 1 chunk +0 lines, -238 lines 0 comments Download
D chrome/test/data/gdata/root_feed.json View 1 chunk +0 lines, -1438 lines 0 comments Download

Messages

Total messages: 13 (0 generated)
hashimoto
6 years, 8 months ago (2014-04-28 06:59:08 UTC) #1
kinaba
lgtm
6 years, 8 months ago (2014-04-28 07:04:10 UTC) #2
hashimoto
The CQ bit was checked by hashimoto@chromium.org
6 years, 7 months ago (2014-04-28 08:42:19 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/hashimoto@chromium.org/252563011/40001
6 years, 7 months ago (2014-04-28 08:43:22 UTC) #4
commit-bot: I haz the power
The CQ bit was unchecked by commit-bot@chromium.org
6 years, 7 months ago (2014-04-28 10:10:47 UTC) #5
commit-bot: I haz the power
Try jobs failed on following builders: tryserver.chromium on chromium_presubmit
6 years, 7 months ago (2014-04-28 10:10:47 UTC) #6
hashimoto
The CQ bit was checked by hashimoto@chromium.org
6 years, 7 months ago (2014-04-28 11:21:25 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/hashimoto@chromium.org/252563011/40001
6 years, 7 months ago (2014-04-28 11:21:53 UTC) #8
commit-bot: I haz the power
The CQ bit was unchecked by commit-bot@chromium.org
6 years, 7 months ago (2014-04-28 11:40:24 UTC) #9
commit-bot: I haz the power
Try jobs failed on following builders: tryserver.chromium on chromium_presubmit
6 years, 7 months ago (2014-04-28 11:40:25 UTC) #10
hashimoto
The CQ bit was checked by hashimoto@chromium.org
6 years, 7 months ago (2014-04-28 14:46:18 UTC) #11
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/hashimoto@chromium.org/252563011/40001
6 years, 7 months ago (2014-04-28 14:46:45 UTC) #12
commit-bot: I haz the power
6 years, 7 months ago (2014-04-28 14:52:42 UTC) #13
Message was sent while issue was closed.
Change committed as 266567

Powered by Google App Engine
This is Rietveld 408576698