Move towards event-driven JS on about:sync

chrome/browser/ui/webui/sync_internals_message_handler.h

Index: chrome/browser/ui/webui/sync_internals_message_handler.h
diff --git a/chrome/browser/ui/webui/sync_internals_message_handler.h b/chrome/browser/ui/webui/sync_internals_message_handler.h
index 5b652d5627ade67284270dda9fa7a8503d0bb108..b8f86bb66f08f382d00e4664cd2455118d2bef85 100644
--- a/chrome/browser/ui/webui/sync_internals_message_handler.h
+++ b/chrome/browser/ui/webui/sync_internals_message_handler.h
@@ -11,6 +11,7 @@
 #include "base/compiler_specific.h"
 #include "base/memory/weak_ptr.h"
 #include "base/values.h"
+#include "chrome/browser/sync/profile_sync_service_observer.h"
 #include "content/public/browser/web_ui_message_handler.h"
 #include "sync/js/js_controller.h"
 #include "sync/js/js_event_handler.h"
@@ -22,7 +23,8 @@ class ProfileSyncService;
 class SyncInternalsMessageHandler
     : public content::WebUIMessageHandler,
       public syncer::JsEventHandler,
-      public syncer::JsReplyHandler {
+      public syncer::JsReplyHandler,
+      public ProfileSyncServiceObserver {
  public:
   SyncInternalsMessageHandler();
   virtual ~SyncInternalsMessageHandler();
@@ -30,8 +32,8 @@ class SyncInternalsMessageHandler
   virtual void RegisterMessages() OVERRIDE;
 
   void ForwardToJsController(const std::string& name, const base::ListValue*);
-  void OnGetAboutInfo(const base::ListValue*);
-  void OnGetListOfTypes(const base::ListValue*);
+  void OnRequestUpdatedAboutInfo(const base::ListValue*);
+  void OnRequestListOfTypes(const base::ListValue*);

[Dan Beam, 2014/02/13 22:32:45]

nit: HandleChromeSendName is the convention in WebUI

[Dan Beam, 2014/02/13 22:32:45]

doc

[Dan Beam, 2014/02/13 22:32:45]

const base::ListValue* args

[rlarocque, 2014/02/13 23:34:30]

Done.

[rlarocque, 2014/02/13 23:34:30]

Done.

[rlarocque, 2014/02/13 23:34:30]

Done.

 
   // syncer::JsEventHandler implementation.
   virtual void HandleJsEvent(
@@ -43,8 +45,12 @@ class SyncInternalsMessageHandler
       const std::string& name,
       const syncer::JsArgList& args) OVERRIDE;
 
+  // ProfileSyncServiceObserver implementation.
+  virtual void OnStateChanged() OVERRIDE;
+
  private:
   void RegisterJsControllerCallback(const std::string& name);
+  void SendAboutInfo();

[Dan Beam, 2014/02/13 22:32:45]

doc

[rlarocque, 2014/02/13 23:34:30]

Done.

   ProfileSyncService* GetProfileSyncService();
 
   base::WeakPtr<syncer::JsController> js_controller_;