Closed Bug 1336186 Opened 7 years ago Closed 7 years ago

Update bookmark validator to check "pos" of separator records again

Categories

(Firefox :: Sync, defect, P1)

defect

Tracking

()

RESOLVED FIXED
Firefox 54
Tracking Status
firefox54 --- fixed

People

(Reporter: tcsc, Assigned: tcsc)

Details

Attachments

(1 file)

AIUI now that bug 1228827 has landed, there's no reason we shouldn't check this. This was undone in bug 1276152 because we didn't try very hard to keep it consistent, but now we do?

This might be invalid, if I misunderstood the recently resolved issue.
Reinstating this code should be easy.
Assignee: nobody → tchiovoloni
Priority: -- → P1
Comment on attachment 8837813 [details]
Bug 1336186 - Check separator position in sync bookmark validator.

https://reviewboard.mozilla.org/r/112790/#review114372
Attachment #8837813 - Flags: review?(markh) → review+
Pushed by tchiovoloni@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/4e3df6fb58ca
Check separator position in sync bookmark validator. r=markh
https://hg.mozilla.org/mozilla-central/rev/4e3df6fb58ca
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 54
You need to log in before you can comment on or make changes to this bug.