Fix test by calling syncAll, r?markh draft
authorEthan Glasser-Camp <eglassercamp@mozilla.com>
Wed, 13 Jul 2016 16:33:27 -0400
changeset 390803 f70b2933bcc8b0074d3bf659350a7ee8f918e3a2
parent 390802 f3c9556273973b9a0c8209d83b6616f3498b9630
child 390804 0bdef084c5fca770205b1996003bc8dddee3e85d
child 393018 597a23e1f0e029b8fc0004857a80859677f3e141
push id23751
push usereglassercamp@mozilla.com
push dateThu, 21 Jul 2016 19:09:00 +0000
reviewersmarkh
milestone50.0a1
Fix test by calling syncAll, r?markh MozReview-Commit-ID: 67FRLGgR9ct
services/sync/modules/engines/storage-sync.js
--- a/services/sync/modules/engines/storage-sync.js
+++ b/services/sync/modules/engines/storage-sync.js
@@ -5,16 +5,17 @@
 this.EXPORTED_SYMBOLS = ['StorageSyncEngine'];
 
 var {classes: Cc, interfaces: Ci, utils: Cu} = Components;
 
 Cu.import("resource://services-sync/constants.js");
 Cu.import("resource://services-sync/engines.js");
 Cu.import("resource://services-sync/util.js");
 Cu.import("resource://services-common/async.js");
+Cu.import("resource://gre/modules/ExtensionStorageSync.jsm");
 
 /**
  * The Engine that manages syncing for the web extension "storage"
  * API, and in particular ext.storage.sync.
  *
  * ext.storage.sync is implemented using Kinto, so it has mechanisms
  * for syncing that we do not need to integrate in the Firefox Sync
  * framework, so this is something of a stub.
@@ -27,17 +28,17 @@ StorageSyncEngine.prototype = {
   _trackerObj: StorageSyncTracker,
   // we don't need these since we implement our own sync logic
   _storeObj: undefined,
   _recordObj: undefined,
 
   syncPriority: 10,
 
   _sync: function () {
-    this._log.warn("syncing ext.storage.sync");
+    return ExtensionStorageSync.syncAll();
   },
 };
 
 function StorageSyncTracker(name, engine) {
   Tracker.call(this, name, engine);
   Svc.Obs.add("weave:engine:start-tracking", this);
   Svc.Obs.add("weave:engine:stop-tracking", this);
 }