From https://bugzilla.mozilla.org/show_bug.cgi?id=613588#c117 Just for coherence, should switch these to PlacesUtils

(In reply to Marco Bonardo from comment #0) > Just for coherence, should switch these to PlacesUtils coherence? It's actually a bug, since "this" is the global object in those nested functions, and those constants don't exist there, of course.

(In reply to neil@parkwaycc.co.uk from comment #2) > coherence? It's actually a bug, since "this" is the global object in those > nested functions, and those constants don't exist there, of course. ugh, sorry, missed the context. Will be better to backport the fix to Aurora.

Comment on attachment 626804 [details] [diff] [review] patch v1.0 [Approval Request Comment] Bug caused by (feature/regressing bug #): Bug 613588 User impact if declined: There is potential disruption in bookmarks json backups, drag&drop, copy&paste of livemark items Testing completed (on m-c, etc.): landing on m-c Risk to taking this patch (and alternatives if risky): just changing some "this" to the proper object, pretty safe String or UUID changes made by this patch: none

Comment on attachment 626804 [details] [diff] [review] patch v1.0 [Triage Comment] This hasn't been a significant user pain point, as dupes/feedback haven't been piling up, so this doesn't meet our criteria for landing after the final beta. Approving for Aurora 14, however.