Closed Bug 1260089 Opened 9 years ago Closed 9 years ago

[TV][Sync][v2.5] Ignore a deleting operation for a non-existing bookmark record

Categories

(Firefox OS Graveyard :: Sync, defect, P1)

ARM
Gonk (Firefox OS)
defect

Tracking

(blocking-b2g:2.5+, b2g-v2.5 fixed, b2g-master fixed)

RESOLVED FIXED
blocking-b2g 2.5+
Tracking Status
b2g-v2.5 --- fixed
b2g-master --- fixed

People

(Reporter: selee, Assigned: selee)

References

Details

(Whiteboard: [ft:conndevices])

Attachments

(2 files)

Show a warning message for deleting a non-existing bookmark record and return early.
blocking-b2g: --- → 2.5?
Comment on attachment 8735389 [details] [review] [gaia] weilonge:seanlee/DataSync/master/Bug1260089 > mozilla-b2g:master Hey Michiel, Could you help to review this patch? Thank you!
Attachment #8735389 - Flags: review?(mbdejong)
Comment on attachment 8735389 [details] [review] [gaia] weilonge:seanlee/DataSync/master/Bug1260089 > mozilla-b2g:master r+ (apart from TreeHerder which looks a mess, but I guess that's just business as usual nowadays).
Attachment #8735389 - Flags: review?(mbdejong) → review+
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Comment on attachment 8735389 [details] [review] [gaia] weilonge:seanlee/DataSync/master/Bug1260089 > mozilla-b2g:master [Approval Request Comment] [Bug caused by] (feature/regressing bug #): Bug 1194108 [User impact] if declined: Sync feature will crash when deleting a non-existing record. [Testing completed]: Verified by Partner and test-case. [Risk to taking this patch] (and alternatives if risky): minor [String changes made]: none
Attachment #8735389 - Flags: approval-gaia-v2.5?
blocking-b2g: 2.5? → 2.5+
Whiteboard: [ft:conndevices]
Comment on attachment 8735389 [details] [review] [gaia] weilonge:seanlee/DataSync/master/Bug1260089 > mozilla-b2g:master Approve for TV 2.5
Attachment #8735389 - Flags: approval-gaia-v2.5? → approval-gaia-v2.5+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: