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

Unified Diff: chrome/browser/extensions/api/sync_file_system/sync_file_system_api.cc

Issue 16190005: [SyncFileSystem] Stop referring to DriveFileSyncService from extension/ (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/extensions/api/sync_file_system/sync_file_system_api.cc
diff --git a/chrome/browser/extensions/api/sync_file_system/sync_file_system_api.cc b/chrome/browser/extensions/api/sync_file_system/sync_file_system_api.cc
index 64e229572f62b53310ab40f67d04f101f6cf2ae4..6b30975d56c9772cadfecd8efe76cc1e3437fbb5 100644
--- a/chrome/browser/extensions/api/sync_file_system/sync_file_system_api.cc
+++ b/chrome/browser/extensions/api/sync_file_system/sync_file_system_api.cc
@@ -14,7 +14,6 @@
#include "chrome/browser/extensions/api/sync_file_system/extension_sync_event_observer_factory.h"
#include "chrome/browser/extensions/api/sync_file_system/sync_file_system_api_helpers.h"
#include "chrome/browser/profiles/profile.h"
-#include "chrome/browser/sync_file_system/drive_file_sync_service.h"
#include "chrome/browser/sync_file_system/sync_file_system_service.h"
#include "chrome/browser/sync_file_system/sync_file_system_service_factory.h"
#include "chrome/common/extensions/api/sync_file_system.h"
@@ -40,10 +39,6 @@ namespace extensions {
namespace {
-// This is the only supported cloud backend service for now.
-const char* const kDriveCloudService =
- sync_file_system::DriveFileSyncService::kServiceName;
-
// Error messages.
const char kFileError[] = "File error %d.";
const char kQuotaError[] = "Quota error %d.";
@@ -58,7 +53,9 @@ sync_file_system::SyncFileSystemService* GetSyncFileSystemService(
ExtensionSyncEventObserver* observer =
ExtensionSyncEventObserverFactory::GetForProfile(profile);
DCHECK(observer);
- observer->InitializeForService(service, kDriveCloudService);
+ observer->InitializeForService(
+ service,
+ sync_file_system::SyncFileSystemService::GetDefaultServiceName());
kinuko 2013/05/29 14:12:20 I think this parameter can be dropped. (Not used b
return service;
}
@@ -113,12 +110,13 @@ void SyncFileSystemDeleteFileSystemFunction::DidDeleteFileSystem(
}
bool SyncFileSystemRequestFileSystemFunction::RunImpl() {
- // Please note that Google Drive is the only supported cloud backend at this
- // time. However other functions which have already been written to
- // accommodate different service names are being left as is to allow easier
- // future support for other backend services. (http://crbug.com/172562).
- const std::string service_name = sync_file_system::DriveFileSyncService::
- kServiceName;
+ // Please note that the default backing service is the only supported cloud
+ // backend at this time. However other functions which have already been
+ // written to accommodate different service names are being left as is to
+ // allow easier future support for other backend services.
+ // (http://crbug.com/172562).
+ const std::string service_name =
+ sync_file_system::SyncFileSystemService::GetDefaultServiceName();
// Initializes sync context for this extension and continue to open
// a new file system.
GetSyncFileSystemService(profile())->

Powered by Google App Engine
This is Rietveld 408576698