[Sync] Speed up sync node browser/search in about:sync

chrome/browser/sync/engine/syncapi.h

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // This file defines the "sync API", an interface to the syncer
 // backend that exposes (1) the core functionality of maintaining a consistent
 // local snapshot of a hierarchical object set; (2) a means to transactionally
 // access and modify those objects; (3) a means to control client/server
 // synchronization tasks, namely: pushing local object modifications to a
 // server, pulling nonlocal object modifications from a server to this client,
@@ skipping 257 matching lines @@
   virtual int64 GetSuccessorId() const;
 
   // Return the ID of the first child of this node.  If this node has no
   // children, return 0.
   virtual int64 GetFirstChildId() const;
 
   // These virtual accessors provide access to data members of derived classes.
   virtual const syncable::Entry* GetEntry() const = 0;
   virtual const BaseTransaction* GetTransaction() const = 0;
 
-  // Dumps all node info into a DictionaryValue and returns it.
+  // Dumps a summary of node info into a DictionaryValue and returns it.
   // Transfers ownership of the DictionaryValue to the caller.
-  DictionaryValue* ToValue() const;
+  DictionaryValue* GetSummaryAsValue() const;
+
+  // Dumps all node details into a DictionaryValue and returns it.
+  // Transfers ownership of the DictionaryValue to the caller.
+  DictionaryValue* GetDetailsAsValue() const;
 
   // Does a case in-sensitive search for a given string, which must be
   // lower case.
   bool ContainsString(const std::string& lowercase_query) const;
 
  protected:
   BaseNode();
   virtual ~BaseNode();
   // The server has a size limit on client tags, so we generate a fixed length
   // hash locally. This also ensures that ModelTypes have unique namespaces.
@@ skipping 673 matching lines @@
   // NOTE: It is OK (in fact, it's probably a good idea) to call this before
   // having received OnInitializationCompleted.
   void AddObserver(Observer* observer);
 
   // Remove the given observer.  Make sure to call this if the
   // Observer is being destroyed so the SyncManager doesn't
   // potentially dereference garbage.
   void RemoveObserver(Observer* observer);
 
   // Returns a pointer to the JsBackend (which is owned by the sync
-  // manager).  Never returns NULL.  The following events are sent by
-  // the returned backend:
+  // manager).  Never returns NULL.  jsDocs for raised events are below:
+
+  /**
+   * @param {{ enabled: boolean }} details enabled is set to whether
+   *     or not notifications are enabled.
+   */
+  // function onNotificationStateChange(details);
+
+  /**
+   * @param {{ changedTypes: Array.<string> }} details changedTypes is
+   *     a list of types (as strings) for which there are new updates.
+   */
+  // function onIncomingNotification(details);
+
+  // jsDocs for handled messages are below (all other messages are
+  // ignored).
+
+  /**
+   * Gets the current notification state.
+   *
+   * @param {function(boolean)} callback Called with whether or not
+   *     notifications are enabled.
+   */
+  // function getNotificationState(callback);
+
+  /**
+   * Gets details about the root node.
+   *
+   * @param {function(!Object)} callback Called with details about the
+   *     root node.
+   */
+  // TODO(akalin): Change this to getRootNodeId or eliminate it
+  // entirely.
   //
-  // onNotificationStateChange({ enabled: (boolean) }):
-  //   Sent when notifications are enabled or disabled.
-  //
-  // onIncomingNotification({ changedTypes: (array) }):
-  //   Sent when an incoming notification arrives.  |changedTypes|
-  //   is a list of sync types (strings) which have changed.
-  //
-  // The following messages are processed by the returned backend:
-  //
-  // getNotificationState():
-  //   callback(boolean notificationsEnabled):
-  //     notificationsEnabled: whether or not notifications are
-  //     enabled.
-  //
-  // getRootNode():
-  //   callback(dictionary nodeInfo):
-  //     nodeInfo: Information on the root node.
-  //
-  // getNodesById(array idList):
-  //   idList: A list of IDs as strings.
-  //   callback(array nodeList):
-  //     nodeList: Information on each node for each valid id in
-  //     idList.  Not guaranteed to be in any order.
-  //
-  // getChildNodeIds(string id):
-  //   id: The id of the node for which to return the child node ids.
-  //   callback(array idList):
-  //     idList: The child node IDs of the node with the given id.
-  //
-  // All other messages are dropped.
+  // function getRootNodeDetails(callback);
+
+  /**
+   * Gets summary information for a list of ids.
+   *
+   * @param {Array.<string>} idList List of 64-bit ids in decimal
+   *     string form.
+   * @param {Array.<{id: string, title: string, isFolder: boolean}>}
+   * callback Called with summaries for the nodes in idList that
+   *     exist.
+   */
+  // function getNodeSummariesById(idList, callback);
+
+  /**
+   * Gets detailed information for a list of ids.
+   *
+   * @param {Array.<string>} idList List of 64-bit ids in decimal
+   *     string form.
+   * @param {Array.<!Object>} callback Called with detailed
+   *     information for the nodes in idList that exist.
+   */
+  // function getNodeDetailsById(idList, callback);
+
+  /**
+   * Gets child ids for a given id.
+   *
+   * @param {string} id 64-bit id in decimal string form of the parent
+   *     node.
+   * @param {Array.<string>} callback Called with the (possibly empty)
+   *     list of child ids.
+   */
+  // function getChildNodeIds(id);
+
   browser_sync::JsBackend* GetJsBackend();
 
   // Status-related getters. Typically GetStatusSummary will suffice, but
   // GetDetailedSyncStatus can be useful for gathering debug-level details of
   // the internals of the sync engine.
   Status::Summary GetStatusSummary() const;
   Status GetDetailedStatus() const;
 
   // Whether or not the Nigori node is encrypted using an explicit passphrase.
   bool IsUsingExplicitPassphrase();
@@ skipping 29 matching lines @@
  private:
   // An opaque pointer to the nested private class.
   SyncInternal* data_;
 
   DISALLOW_COPY_AND_ASSIGN(SyncManager);
 };
 
 }  // namespace sync_api
 
 #endif  // CHROME_BROWSER_SYNC_ENGINE_SYNCAPI_H_