summaryrefslogtreecommitdiffstats
path: root/chrome/browser/sync_file_system/sync_file_system_service.cc
diff options
context:
space:
mode:
authorkinuko@chromium.org <kinuko@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2013-02-07 11:41:02 +0000
committerkinuko@chromium.org <kinuko@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2013-02-07 11:41:02 +0000
commit92151d2ad2a5168c3aae0e3001efbaacf417c584 (patch)
tree4e8d124c6059796086a67aaa17d5ab868bf53e95 /chrome/browser/sync_file_system/sync_file_system_service.cc
parentff9b1c1cfcfe84efeaa663aff7223663942c4546 (diff)
downloadchromium_src-92151d2ad2a5168c3aae0e3001efbaacf417c584.zip
chromium_src-92151d2ad2a5168c3aae0e3001efbaacf417c584.tar.gz
chromium_src-92151d2ad2a5168c3aae0e3001efbaacf417c584.tar.bz2
SyncFS: remove unused conflict methods
We'll need to revisit this but for now I want to remove this annoying code as it's making the code less readable / hackable. BUG=none TEST=none, just deleting tests Review URL: https://codereview.chromium.org/12218048 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@181284 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/sync_file_system/sync_file_system_service.cc')
-rw-r--r--chrome/browser/sync_file_system/sync_file_system_service.cc70
1 files changed, 0 insertions, 70 deletions
diff --git a/chrome/browser/sync_file_system/sync_file_system_service.cc b/chrome/browser/sync_file_system/sync_file_system_service.cc
index 83903c5..55ea7a9 100644
--- a/chrome/browser/sync_file_system/sync_file_system_service.cc
+++ b/chrome/browser/sync_file_system/sync_file_system_service.cc
@@ -179,84 +179,18 @@ void SyncFileSystemService::InitializeForApp(
DVLOG(1) << "InitializeForApp: " << app_origin.spec();
- if (initialized_app_origins_.find(app_origin) !=
- initialized_app_origins_.end()) {
- DVLOG(1) << "The app is already initialized: " << app_origin.spec();
- callback.Run(fileapi::SYNC_STATUS_OK);
- return;
- }
-
local_file_service_->MaybeInitializeFileSystemContext(
app_origin, service_name, file_system_context,
base::Bind(&SyncFileSystemService::DidInitializeFileSystem,
AsWeakPtr(), app_origin, callback));
}
-void SyncFileSystemService::GetConflictFiles(
- const GURL& app_origin,
- const std::string& service_name,
- const fileapi::SyncFileSetCallback& callback) {
- DCHECK(remote_file_service_);
- DCHECK(app_origin == app_origin.GetOrigin());
-
- if (!ContainsKey(initialized_app_origins_, app_origin)) {
- callback.Run(fileapi::SYNC_STATUS_NOT_INITIALIZED,
- fileapi::FileSystemURLSet());
- return;
- }
-
- remote_file_service_->GetConflictFiles(
- app_origin, base::Bind(&VerifyFileSystemURLSetCallback,
- AsWeakPtr(), app_origin, service_name, callback));
-}
-
-void SyncFileSystemService::GetConflictFileInfo(
- const GURL& app_origin,
- const std::string& service_name,
- const FileSystemURL& url,
- const ConflictFileInfoCallback& callback) {
- DCHECK(local_file_service_);
- DCHECK(remote_file_service_);
- DCHECK(app_origin == app_origin.GetOrigin());
-
- if (!ContainsKey(initialized_app_origins_, app_origin)) {
- callback.Run(fileapi::SYNC_STATUS_NOT_INITIALIZED,
- fileapi::ConflictFileInfo());
- return;
- }
-
- // Call DidGetConflictFileInfo when both remote and local service's
- // GetFileMetadata calls are done.
- SyncFileMetadata* remote_metadata = new SyncFileMetadata;
- SyncFileMetadata* local_metadata = new SyncFileMetadata;
- SyncStatusCallback completion_callback =
- base::Bind(&SyncFileSystemService::DidGetConflictFileInfo,
- AsWeakPtr(), callback, url,
- base::Owned(local_metadata),
- base::Owned(remote_metadata));
- scoped_refptr<SharedCallbackRunner> callback_runner(
- new SharedCallbackRunner(completion_callback));
- local_file_service_->GetLocalFileMetadata(
- url, callback_runner->CreateAssignAndRunCallback(local_metadata));
- remote_file_service_->GetRemoteFileMetadata(
- url, callback_runner->CreateAssignAndRunCallback(remote_metadata));
-}
-
void SyncFileSystemService::GetFileSyncStatus(
const fileapi::FileSystemURL& url,
const fileapi::SyncFileStatusCallback& callback) {
DCHECK(local_file_service_);
DCHECK(remote_file_service_);
- if (!ContainsKey(initialized_app_origins_, url.origin())) {
- base::MessageLoopProxy::current()->PostTask(
- FROM_HERE,
- base::Bind(callback,
- fileapi::SYNC_STATUS_NOT_INITIALIZED,
- fileapi::SYNC_FILE_STATUS_UNKNOWN));
- return;
- }
-
if (remote_file_service_->IsConflicting(url)) {
base::MessageLoopProxy::current()->PostTask(
FROM_HERE,
@@ -359,9 +293,6 @@ void SyncFileSystemService::DidRegisterOrigin(
fileapi::SyncStatusCode status) {
DVLOG(1) << "DidRegisterOrigin: " << app_origin.spec() << " " << status;
- if (status == fileapi::SYNC_STATUS_OK)
- initialized_app_origins_.insert(app_origin);
-
callback.Run(status);
}
@@ -560,7 +491,6 @@ void SyncFileSystemService::Observe(
remote_file_service_->UnregisterOriginForTrackingChanges(
app_origin, base::Bind(&DidHandleOriginForExtensionEvent,
type, app_origin));
- initialized_app_origins_.erase(app_origin);
local_file_service_->SetOriginEnabled(app_origin, false);
} else if (chrome::NOTIFICATION_EXTENSION_LOADED == type) {
std::string extension_id =