Closed Bug 479082 Opened 12 years ago Closed 12 years ago
Complete should not show unvisited not-bookmarked-elsewhere children of livemark containers
I'm not sure how to test this offhand, but we need a test that makes sure livemarks get filtered out of autocomplete.
This could probably go in the autocomplete/ tests directory. There isn't any existing code there to add livemarks in that head_* file, but check out.. unit/test_exclude_livemarks.js http://mxr.mozilla.org/mozilla-central/source/toolkit/components/places/tests/unit/test_exclude_livemarks.js
Summary: AutoComplete test for filtering out livemarks → AutoComplete should not show unvisited not-bookmarked-elsewhere children of livemark containers
Added an addLivemark function to head_autocomplete.js. Tests autocomplete on both URIs and titles, not sure if that's necessary. Not familiar with autocomplete at all, so there are probably problems with this patch...
Attachment #363174 - Flags: review?(sdwilsh)
Comment on attachment 363174 [details] [diff] [review] v1 r=sdwilsh
Attachment #363174 - Flags: review?(sdwilsh) → review+
Status: NEW → RESOLVED
Closed: 12 years ago
OS: Mac OS X → All
Hardware: x86 → All
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.2a1
Marco - this bug wasn't marked as checkin-needed for a reason...
oh sorry, was a test only and with a review, back it out if needed :\
You need to log in before you can comment on or make changes to this bug.