diff options
author | akalin@chromium.org <akalin@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-08-12 18:40:29 +0000 |
---|---|---|
committer | akalin@chromium.org <akalin@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-08-12 18:40:29 +0000 |
commit | 8252963a12dc832a040be3877ff9bc4ff6acaa1d (patch) | |
tree | 067a2c2c55f96931bd3d905e6f001aae4313cafa | |
parent | 9fcbae04779503c5717c757c052e0749240f8d06 (diff) | |
download | chromium_src-8252963a12dc832a040be3877ff9bc4ff6acaa1d.zip chromium_src-8252963a12dc832a040be3877ff9bc4ff6acaa1d.tar.gz chromium_src-8252963a12dc832a040be3877ff9bc4ff6acaa1d.tar.bz2 |
[Sync] Add comment for SyncBackendHost::OnBackendInitialized
BUG=
TEST=
Review URL: http://codereview.chromium.org/7614001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@96587 0039d316-1c4b-4281-b951-d872f2087c98
-rw-r--r-- | chrome/browser/sync/engine/syncapi.h | 7 | ||||
-rw-r--r-- | chrome/browser/sync/glue/sync_backend_host.h | 10 |
2 files changed, 11 insertions, 6 deletions
diff --git a/chrome/browser/sync/engine/syncapi.h b/chrome/browser/sync/engine/syncapi.h index e869e5e..eaf9a56 100644 --- a/chrome/browser/sync/engine/syncapi.h +++ b/chrome/browser/sync/engine/syncapi.h @@ -845,7 +845,8 @@ class SyncManager { // WARNING: Calling methods on the SyncManager before receiving this // message, unless otherwise specified, produces undefined behavior. // - // The given backend can emit the following events: + // |js_backend| is what about:sync interacts with. It can emit + // the following events: /** * @param {{ enabled: boolean }} details A dictionary containing: @@ -861,8 +862,8 @@ class SyncManager { */ // function onIncomingNotification(details); - // The given backend responds to the following messages (all other - // messages are ignored): + // Also, it responds to the following messages (all other messages + // are ignored): /** * Gets the current notification state. diff --git a/chrome/browser/sync/glue/sync_backend_host.h b/chrome/browser/sync/glue/sync_backend_host.h index 7e6ee22..9270f75 100644 --- a/chrome/browser/sync/glue/sync_backend_host.h +++ b/chrome/browser/sync/glue/sync_backend_host.h @@ -55,9 +55,13 @@ class SyncFrontend { public: SyncFrontend() {} - // The backend has completed initialization and it is now ready to accept and - // process changes. If success is false, initialization wasn't able to be - // completed and should be retried. + // The backend has completed initialization and it is now ready to + // accept and process changes. If success is false, initialization + // wasn't able to be completed and should be retried. + // + // |js_backend| is what about:sync interacts with; it's different + // from the 'Backend' in 'OnBackendInitialized' (unfortunately). It + // is initialized only if |success| is true. virtual void OnBackendInitialized( const WeakHandle<JsBackend>& js_backend, bool success) = 0; |