Closed
Bug 732186
Opened 13 years ago
Closed 13 years ago
Deleting Live bookmarks causes Errors
Categories
(Toolkit :: Places, defect)
Tracking
()
RESOLVED
FIXED
mozilla13
People
(Reporter: alice0775, Assigned: mak)
References
Details
Attachments
(1 file)
(deleted),
patch
|
dietrich
:
review+
|
Details | Diff | Splinter Review |
Build Identifier:
http://hg.mozilla.org/mozilla-central/rev/1c3b291d0830
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:13.0) Gecko/20120301 Firefox/13.0a1 ID:20120301031135
Deleting Live bookmarks causes Errors.
(no other failure)
Reproducible: Always
1. Start Firefox with new profile and show Bookmarks toolbar
2. Subscribe RSS feed of https://bugzilla.mozilla.org/buglist.cgi?chfieldto=Now;chfield=[Bug%20creation];chfieldfrom=-96h;order=Bug%20Number;bug_status=UNCONFIRMED;bug_status=NEW;bug_status=ASSIGNED;bug_status=REOPENED;bug_status=RESOLVED;bug_status=VERIFIED;product=Core;product=Firefox;product=Mozilla%20Application%20Suite;product=Thunderbird;product=Toolkit;product=Tech%20Evangelism;product=MailNews%20Core;list_id=2503836
3. Popup Live bookmarks on Bookmarks toolbar and Confirm that items are enumerated
4. Delete the Live bookmarks.
Actual Results:
Errors at step4;
Timestamp: 2012/03/02 3:54:36
Error: this._resultObserversList is undefined
Source File: resource:///components/nsLivemarkService.js
Line: 967
Timestamp: 2012/03/02 3:54:54
Error: this._resultObserversList is undefined
Source File: resource:///components/nsLivemarkService.js
Line: 983
Timestamp: 2012/03/02 3:55:01
Error: this._resultObserversList is undefined
Source File: resource:///components/nsLivemarkService.js
Line: 983
Expected Results:
no error
Assignee | ||
Comment 1•13 years ago
|
||
likely a minor glitch, I noticed a similar error on shutdown.
Assignee: nobody → mak77
Blocks: livemarksIO
Assignee | ||
Comment 2•13 years ago
|
||
as I thought was a trivial and harmless thing, when a livemark is terminated it is deleted and the next abort tries to set the status, that tries to use the list.
Attachment #602178 -
Flags: review?(dietrich)
Comment 3•13 years ago
|
||
Comment on attachment 602178 [details] [diff] [review]
patch v1.0
Review of attachment 602178 [details] [diff] [review]:
-----------------------------------------------------------------
sounds ok. r=me with a test.
Attachment #602178 -
Flags: review?(dietrich) → review+
Assignee | ||
Comment 4•13 years ago
|
||
(In reply to Dietrich Ayala (:dietrich) from comment #3)
> Comment on attachment 602178 [details] [diff] [review]
> patch v1.0
>
> Review of attachment 602178 [details] [diff] [review]:
> -----------------------------------------------------------------
>
> sounds ok. r=me with a test.
which kind of test? everything works fine, the only problem was a spurious error thrown to the console.
Comment 5•13 years ago
|
||
Sorry, didn't realize it's just console spew. No test required then.
Assignee | ||
Comment 6•13 years ago
|
||
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla13
Assignee | ||
Updated•13 years ago
|
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment 7•13 years ago
|
||
Status: REOPENED → RESOLVED
Closed: 13 years ago → 13 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•