Closed Bug 484094 Opened 15 years ago Closed 12 years ago

Dynamic Containers should use nsINavBookmarksObserver::OnBeforeItemRemoved

Categories

(Toolkit :: Places, defect)

defect
Not set
normal

Tracking

()

RESOLVED WONTFIX

People

(Reporter: sdwilsh, Unassigned)

References

Details

Dynamic Containers should use nsINavBookmarksObserver::OnBeforeItemRemoved instead of it's own special casing code.
i think i already removed that completely in the redesign, the service that populates then should be a bookmarks observer by itself if needed.
I am getting this error in the Error console while testing Beta 1 - Error: [Exception... "'JavaScript component does not have a method named: "onBeforeItemRemoved"' when calling method: [nsINavBookmarkObserver::onBeforeItemRemoved]"  nsresult: "0x80570030 (NS_ERROR_XPC_JSOBJECT_HAS_NO_FUNCTION_NAMED)"  location: "JS frame :: file:///Users/mozilla/Desktop/BETA/Firefox.app/Contents/MacOS/components/nsLivemarkService.js :: LS_deleteLivemarkChildren :: line 582"  data: no]
Source File: file:///Users/mozilla/Desktop/BETA/Firefox.app/Contents/MacOS/components/nsLivemarkService.js
Line: 582.

Is this related?
no, livemarks are not yet dynamic containers, that's a separate issue.
no more dynamic containers fwiw
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.