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

Unified Diff: chrome/common/extensions/api/sync_file_system.idl

Issue 11316133: Added implementation of SyncEventObserver to route events to Javascript Extension. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix destructor for compile Created 8 years, 1 month 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
« no previous file with comments | « chrome/chrome_browser_extensions.gypi ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/common/extensions/api/sync_file_system.idl
diff --git a/chrome/common/extensions/api/sync_file_system.idl b/chrome/common/extensions/api/sync_file_system.idl
index 5971efe9dedaa2760ee32ba98bcfa63f03ff5e0d..43e65075bf82fc036ab33970989668b285aa6776 100644
--- a/chrome/common/extensions/api/sync_file_system.idl
+++ b/chrome/common/extensions/api/sync_file_system.idl
@@ -14,30 +14,34 @@ namespace syncFileSystem {
DOMString description;
};
+ enum SyncOperationType {
+ added, updated, deleted
+ };
+
enum SyncStateStatus {
// The sync service is being initialized (e.g. restoring data from the
// database, checking connectivity and authenticating to the service etc).
- INITIALIZING,
+ initializing,
// The sync service is up and running.
- RUNNING,
+ running,
// The sync service is not synchronizing files because the remote service
// needs to be authenticated by the user to proceed.
- AUTHENTICATION_REQUIRED,
+ authentication_required,
// The sync service is not synchronizing files because the remote service
// is (temporarily) unavailable due to some recoverable errors, e.g.
// network is offline, the remote service is down or not
// reachable etc. More details should be given by |description| parameter
// in OnSyncStateUpdated (which could contain service-specific details).
- TEMPORARY_UNAVAILABLE,
+ temporary_unavailable,
// The sync service is disabled and the content will never sync.
// (E.g. this could happen when the user has no account on
// the remote service or the sync service has had an unrecoverable
// error.)
- DISABLED
+ disabled
};
// [nodoc] A callback type for requestFileSystem.
@@ -71,5 +75,10 @@ namespace syncFileSystem {
// sync backend. (E.g. the sync is temporarily disabled due to
// network or authentication error etc)
static void onSyncStateChanged(SyncState detail);
+
+ // Fired when a file has been updated by the background sync service.
+ // TODO(calvinlo): Convert |file_entry_path| from to Webkit FileEntry.
+ static void onFileSynced(DOMString file_entry_path,
+ SyncOperationType operation);
};
};
« no previous file with comments | « chrome/chrome_browser_extensions.gypi ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698