1453096 - New bookmark sync differences problems

Status: NEW

(Firefox :: Sync, defect, P3)

Description

[TMart]

Attachment: local_sync_data_anonymized

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:61.0) Gecko/20100101 Firefox/61.0
Build ID: 20180410100115

Steps to reproduce:

Followed the instructions of the Blog post draft for the new Bookmark Sync: https://docs.google.com/document/d/1WoM7JEatr_QJ1CrGk2rwoSDy8OhkWj9vbEmfzwVJ8O4/edit


Actual results:

Encountered differences: 5 problems

Comment 1

[Thom Chiovoloni [:tcsc] (ex-moco)]

These are `pos` differences for separators. The old bookmark engine used these for deduping, but the new bookmark engine doesn't need them at all, and so the differences are not a problem.

Arguably, we shouldn't report them if the new engine is enabled, although it's not clear to me this is the right choice in all cases, since you may have other devices on the old engine (although, `pos` is only a minor issue at most...).

Comment 2

[Thom Chiovoloni [:tcsc] (ex-moco)]

This is causing prod TPS failures so I can ignore it no longer!

Comment 3

[Thom Chiovoloni [:tcsc] (ex-moco)]

Attachment: Bug 1453096 - Ignore 'pos' for separators in the bookmark validator r?lina

Comment 4

[BugBot [:suhaib / :marco/ :calixte]]

The bug assignee is inactive on Bugzilla, so the assignee is being reset.

Comment 5

[Suhaib Mujahid [:suhaib]]

Sorry, there was a problem with the detection of inactive users. I'm reverting the change.