Closed Bug 1069973 Opened 5 years ago Closed 5 years ago

Proxify both nsINavBookmarksService and Bookmarks.jsm through PlacesUtils.bookmarks.

Categories

(Toolkit :: Places, defect)

defect
Not set
Points:
3

Tracking

()

RESOLVED FIXED
mozilla35
Iteration:
35.2

People

(Reporter: mak, Assigned: mak)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

To limit breakage and speed up development we need a common proxy for new and old API.
Marco, please add this to the current iteration, I need it to proceed with bug 1068009
Blocks: 1068009
Flags: qe-verify-
Flags: needinfo?(mmucci)
Flags: firefox-backlog+
Added to IT 35.2

(In reply to Marco Bonardo [::mak] (needinfo? me) from comment #1)
> Marco, please add this to the current iteration, I need it to proceed with
> bug 1068009
Iteration: --- → 35.2
Flags: needinfo?(mmucci)
Attached patch patch v1Splinter Review
Attachment #8492237 - Flags: review?(mano)
i'm going to push the latest patches to Try now, just in case.
Depends on: 1068007
Comment on attachment 8492237 [details] [diff] [review]
patch v1

You can even inline the Proxy inside Object.freeze, if you wish.
Attachment #8492237 - Flags: review?(mano) → review+
inlined:
https://hg.mozilla.org/integration/fx-team/rev/369990915846
Target Milestone: --- → mozilla35
https://hg.mozilla.org/mozilla-central/rev/369990915846
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.