[Sync] Refactor JS event handling to reduce boilerplate

Split event handlers/routers to handle events and message replies
separately.

Reduce JS boilerplate.

Use OVERRIDE where appropriate.

BUG=
TEST=

Review URL: http://codereview.chromium.org/6930030

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84357 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 5 insertions, 4 deletions

diff --git a/chrome/browser/sync/test_profile_sync_service.h b/chrome/browser/sync/test_profile_sync_service.h
index d07e777..4c3e19d 100644
--- a/chrome/browser/sync/test_profile_sync_service.h
+++ b/chrome/browser/sync/test_profile_sync_service.h
@@ -8,6 +8,7 @@
 
 #include <string>
 
+#include "base/compiler_specific.h"
 #include "chrome/browser/sync/glue/data_type_manager_impl.h"
 #include "chrome/browser/sync/js_backend.h"
 #include "chrome/browser/sync/profile_sync_service.h"
@@ -60,14 +61,14 @@ class SyncBackendHostForProfileSyncTest
   virtual JsBackend* GetJsBackend();
 
   // JsBackend implementation.
-  virtual void SetParentJsEventRouter(JsEventRouter* router);
-  virtual void RemoveParentJsEventRouter();
-  virtual const JsEventRouter* GetParentJsEventRouter() const;
+  virtual void SetParentJsEventRouter(JsEventRouter* router) OVERRIDE;
+  virtual void RemoveParentJsEventRouter() OVERRIDE;
+  virtual const JsEventRouter* GetParentJsEventRouter() const OVERRIDE;
   // Fires an event identical to the message unless the message has
   // "delay" as a prefix, in which case a task to fire the identical
   // event is posted instead.
   virtual void ProcessMessage(const std::string& name, const JsArgList& args,
-                              const JsEventHandler* sender);
+                              const JsEventHandler* sender) OVERRIDE;
 
   virtual void StartConfiguration(Callback0::Type* callback);