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

Unified Diff: chrome/browser/sync/engine/syncapi.h

Issue 7614001: [Sync] Add comment for SyncBackendHost::OnBackendInitialized (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years, 4 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/sync/engine/syncapi.h
diff --git a/chrome/browser/sync/engine/syncapi.h b/chrome/browser/sync/engine/syncapi.h
index 122bd3476882f7025641c5a4a651ce8e51505e84..a769ace3494ca2a12c03b177571c7370bc42c4ea 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:
tim (not reviewing) 2011/08/11 21:43:19 Is this really the best spot for the full JsBacken
akalin 2011/08/12 18:24:26 I can't think of a better place that won't get out
/**
* @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.
« no previous file with comments | « no previous file | chrome/browser/sync/glue/sync_backend_host.h » ('j') | chrome/browser/sync/glue/sync_backend_host.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698