Marco Bonardo [:mak] Assignee
	Description • 10 years ago

Bug 1068009 implements the single removal option, batch removals must still be implemented.

	Marco Bonardo [:mak] Assignee
	Comment 1 • 10 years ago

instead of batch removals we will implement an eraseEverything API that is more coherent with our use cases (empty the roots before a restore mostly, plus tests)

	Marco Bonardo [:mak] Assignee
	Comment 2 • 10 years ago

I'm not totally sold on the API name, if you have better ideas, I'm listening. Alternatives to erase could be wipe or empty... alternatives to Everything may be All (but sounds less scary), Store (but it's not very clear) or Roots (but then we must introduce the roots concept in the API)

	Marco Bonardo [:mak] Assignee
	Comment 3 • 10 years ago

oops I forgot tpo hg add the test

	Asaf Romano (gone)
	Comment 4 • 10 years ago

Comment on attachment 8506237 [details] [diff] [review]
patch v2

Review of attachment 8506237 [details] [diff] [review]:
-----------------------------------------------------------------

Again, at this point I prefer to review one giant patch, but this patch alone looks fine.

::: toolkit/components/places/Bookmarks.jsm
@@ +890,5 @@
> +      if (val !== null) {
> +        Object.defineProperty(item, prop, { value: val
> +                                          , writable: true
> +                                          , enumerable: false
> +                                          , configurable: false });

See my comments on the main bug.

	Marco Bonardo [:mak] Assignee
	Comment 5 • 10 years ago

Comment on attachment 8506237 [details] [diff] [review]
patch v2

patch merged into bug Bug 1068009.

	Marco Bonardo [:mak] Assignee
	Comment 6 • 10 years ago

fixed by Bug 1068009.