Bug 1336186 - Check separator position in sync bookmark validator. r=markh
authorThom Chiovoloni <tchiovoloni@mozilla.com>
Wed, 15 Feb 2017 16:30:59 -0500
changeset 373345 4e3df6fb58ca02d1bbc0eab666d0d368db3bc4ac
parent 373344 9f5fd56ce6ce81afc6d3b6a453aca05118fc2886
child 373346 f7c0b8d47a9b9bfad434038e6043fb91a7c46f2a
push id10863
push userjlorenzo@mozilla.com
push dateMon, 06 Mar 2017 23:02:23 +0000
treeherdermozilla-aurora@0931190cd725 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmarkh
bugs1336186, 1276152
milestone54.0a1
Bug 1336186 - Check separator position in sync bookmark validator. r=markh This reverts bug 1276152. MozReview-Commit-ID: Cdv4XPYl4cp
services/sync/modules/bookmark_validator.js
--- a/services/sync/modules/bookmark_validator.js
+++ b/services/sync/modules/bookmark_validator.js
@@ -761,16 +761,21 @@ class BookmarkValidator {
       } else {
         switch (server.type) {
           case "bookmark":
           case "query":
             if (!areURLsEqual(server.bmkUri, client.bmkUri)) {
               differences.push("bmkUri");
             }
             break;
+          case "separator":
+            if (server.pos != client.pos) {
+              differences.push("pos");
+            }
+            break;
           case "livemark":
             if (server.feedUri != client.feedUri) {
               differences.push("feedUri");
             }
             if (server.siteUri != client.siteUri) {
               differences.push("siteUri");
             }
             break;