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

Issue 6609009: Remove FileSystemQuotaManager class (Closed)

Created:
9 years, 9 months ago by kinuko
Modified:
9 years, 7 months ago
Reviewers:
michaeln, ericu
CC:
chromium-reviews, pam+watch_chromium.org, darin-cc_chromium.org, kinuko+watch, jam, Paweł Hajdan Jr., Dai Mikurube (google.com)
Visibility:
Public.

Description

Remove FileSystemQuotaManager class Now that we have SpecialStoragePolicy class and are going to have QuotaFileSystemUtil, I think we're ready to get rid of this class. BUG=none TEST=none Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=76858

Patch Set 1 : test fix #

Unified diffs Side-by-side diffs Delta from patch set Stats (+126 lines, -227 lines) Patch
M content/browser/file_system/file_system_dispatcher_host.cc View 1 chunk +0 lines, -2 lines 0 comments Download
M webkit/fileapi/file_system_context.h View 3 chunks +12 lines, -3 lines 0 comments Download
M webkit/fileapi/file_system_context.cc View 3 chunks +14 lines, -3 lines 0 comments Download
A webkit/fileapi/file_system_context_unittest.cc View 1 chunk +85 lines, -0 lines 0 comments Download
M webkit/fileapi/file_system_operation.h View 2 chunks +3 lines, -5 lines 0 comments Download
M webkit/fileapi/file_system_operation.cc View 11 chunks +11 lines, -17 lines 0 comments Download
D webkit/fileapi/file_system_quota_manager.h View 1 chunk +0 lines, -54 lines 0 comments Download
D webkit/fileapi/file_system_quota_manager.cc View 1 chunk +0 lines, -38 lines 0 comments Download
D webkit/fileapi/file_system_quota_manager_unittest.cc View 1 chunk +0 lines, -102 lines 0 comments Download
M webkit/fileapi/webkit_fileapi.gypi View 1 chunk +0 lines, -2 lines 0 comments Download
M webkit/tools/test_shell/test_shell.gypi View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 3 (0 generated)
kinuko
9 years, 9 months ago (2011-03-03 03:56:00 UTC) #1
ericu
LGTM
9 years, 9 months ago (2011-03-03 04:18:32 UTC) #2
michaeln
9 years, 9 months ago (2011-03-03 22:29:28 UTC) #3
LGTM2!

Powered by Google App Engine
This is Rietveld 408576698