From fa1348784a1eef04761ddf690d0fed740587d346 Mon Sep 17 00:00:00 2001 From: "akalin@chromium.org" Date: Tue, 2 Aug 2011 18:39:55 +0000 Subject: [Sync] Rewrote about:sync backend to use WeakHandles This simplifies the handling of JS messages, replies, and events. Made ProcessJsMessage take a WeakHandle to a JsReplyHandler instead of having a JsEventRouter class (which is now removed). Add JsBackend parameter to OnInitializationComplete(). Renamed JsFrontend to JsController. Renamed JsEventHandlerList to SyncJsController. Handle events that happen before sync is initialized. Made about:sync work in incognito mode. BUG=90942,76809 TEST= Review URL: http://codereview.chromium.org/7518014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@95119 0039d316-1c4b-4281-b951-d872f2087c98 --- chrome/browser/sync/js_test_util.cc | 20 ++++++++++++++++---- 1 file changed, 16 insertions(+), 4 deletions(-) (limited to 'chrome/browser/sync/js_test_util.cc') diff --git a/chrome/browser/sync/js_test_util.cc b/chrome/browser/sync/js_test_util.cc index 7884ea2..eea7576 100644 --- a/chrome/browser/sync/js_test_util.cc +++ b/chrome/browser/sync/js_test_util.cc @@ -109,17 +109,29 @@ MockJsBackend::MockJsBackend() {} MockJsBackend::~MockJsBackend() {} -MockJsFrontend::MockJsFrontend() {} +WeakHandle MockJsBackend::AsWeakHandle() { + return WeakHandle(AsWeakPtr()); +} + +MockJsController::MockJsController() {} -MockJsFrontend::~MockJsFrontend() {} +MockJsController::~MockJsController() {} MockJsEventHandler::MockJsEventHandler() {} +WeakHandle MockJsEventHandler::AsWeakHandle() { + return WeakHandle(AsWeakPtr()); +} + MockJsEventHandler::~MockJsEventHandler() {} -MockJsEventRouter::MockJsEventRouter() {} +MockJsReplyHandler::MockJsReplyHandler() {} + +MockJsReplyHandler::~MockJsReplyHandler() {} -MockJsEventRouter::~MockJsEventRouter() {} +WeakHandle MockJsReplyHandler::AsWeakHandle() { + return WeakHandle(AsWeakPtr()); +} } // namespace browser_sync -- cgit v1.1