Closed Bug 549192 Opened 10 years ago Closed 10 years ago
History view not updated after deleting entry
Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.3a2pre) Gecko/20100227 Minefield/3.7a2pre ID:20100227031019 After deleting a history entry, the view is not updated and the deleted entry is still visible. This affects the history sidebar (when set to "View By Last Visited") and the Places Library window. Closing and reopening the view shows the correct list of entries. Tested with new profile, no extensions. Steps to reproduce: 1. go to http://www.mozilla.org/ 2. go to http://planet.mozilla.org/ 3. go to https://wiki.mozilla.org/ 4. open places library window, select History > Today 5. delete entry for planet.mozilla.org (via context menu or delete key) Expected behaviour: history entry disappears from list Actual behaviour: view is not updated, entry remains visible Regression range: firefox-3.7a2pre-2010-02-25-03: works firefox-3.7a2pre-2010-02-26-03: fails http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=c390c6f279ee&tochange=475768f37b1a mano: Could this be a regression from bug 520659?
10 years ago
Depends on: 520659
Vista too, I believe, hence platform -> all.
OS: Linux → All
(In reply to comment #0) > This affects the history sidebar (when set to "View By Last > Visited") ... Error: uncaught exception: [Exception... "Component returned failure code: 0x80070057 (NS_ERROR_ILLEGAL_VALUE) [nsINavHistoryContainerResultNode.getChildIndex]" nsresult: "0x80070057 (NS_ERROR_ILLEGAL_VALUE)" location: "JS frame :: chrome://browser/content/places/treeView.js :: PTV__getRowForNode :: line 187" data: no]
actually we have landed a couple fixes to the treeview in bug 555293, among them also a typo. So please, always report your UA.
"a typo fix", clearly.
(In reply to comment #2) > *** Bug 555974 has been marked as a duplicate of this bug. *** In fact that bug is moving entry by revisiting a location but probably the same cause. (In reply to comment #4) > actually we have landed a couple fixes to the treeview in bug 555293, among > them also a typo. > So please, always report your UA. Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9.3a4pre) Gecko/20100330 Minefield/3.7a4pre ID:20100330035832
Patch coming, likely without a test for now...
Assignee: nobody → mano
Status: NEW → ASSIGNED
Is this not more likely a regression of bug 492797?
Attachment #448345 - Flags: review?(mak77) → review+
I can't make vtime a constant and also ++ it...
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.