523316 - Remove unused MOZ_PLACES_BOOKMARKS define

Status: VERIFIED FIXED

(Toolkit :: Places, defect)

Description

[Marco Bonardo [:mak]]

the only global switch for Places is MOZ_PLACES, MOZ_PLACES_BOOKMARKS is completely unused, and we can remove it.

Comment 1

[Ed Morley [:emorley]]

Given this is build config, should I be basing off of m-c instead of http://hg.mozilla.org/projects/places/ ?

Comment 2

[Shawn Wilsher :sdwilsh]

(In reply to comment #1)
> Given this is build config, should I be basing off of m-c instead of
> http://hg.mozilla.org/projects/places/ ?
mozilla-central, please.  We haven't merged m-c into it in a while (although I'll probably do that now).

Comment 3

[Ed Morley [:emorley]]

Attachment: Patch v1 (based on m-c rev 09b605eb3e0d)

Great - have based it off of m-c tip.

Comment 4

[Marco Bonardo [:mak]]

Comment on attachment 525031 [details] [diff] [review]
Patch v1 (based on m-c rev 09b605eb3e0d)

thanks!

Comment 5

[Ed Morley [:emorley]]

Just spotted this was pushed as:
http://hg.mozilla.org/mozilla-central/rev/d114ab94a859

Comment 6

[Ed Morley [:emorley]]

http://mxr.mozilla.org/mozilla-central/search?string=MOZ_PLACES_BOOKMARKS
...returns 0 results.

-> Verified