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)
Firefox
Sync
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.
Comment 1•7 years ago
|
||
Reinstating this code should be easy.
Assignee: nobody → tchiovoloni
Priority: -- → P1
Comment hidden (mozreview-request) |
Comment 3•7 years ago
|
||
mozreview-review |
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
Comment 5•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/4e3df6fb58ca
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox54:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 54
You need to log in
before you can comment on or make changes to this bug.
Description
•