If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

refactor nsIGlobalHistory

VERIFIED FIXED in mozilla0.9

Status

()

Core
History: Global
P2
normal
VERIFIED FIXED
17 years ago
17 years ago

People

(Reporter: Alec Flett, Assigned: Alec Flett)

Tracking

({embed})

Trunk
mozilla0.9
x86
Linux
embed
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: fix in hand)

Attachments

(1 attachment)

(Assignee)

Description

17 years ago
We need to refactor nsIGlobalHistory to make it easy for embedders to
reimplement a super-light history, for simple stuff like link coloring.
(Assignee)

Updated

17 years ago
Status: NEW → ASSIGNED
Keywords: embed
Priority: -- → P2
Target Milestone: --- → mozilla0.9

Updated

17 years ago
Blocks: 70229
(Assignee)

Comment 1

17 years ago
Created attachment 27164 [details] [diff] [review]
How does this look?
(Assignee)

Comment 2

17 years ago
don't worry about the slight rdf changes in nsGlobalHistory.cpp - I won't check
them in
(Assignee)

Comment 3

17 years ago
cc'ing waterson, my history-review-bitch
Whiteboard: fix in hand

Comment 4

17 years ago
This looks good. r=waterson
I'm looking in to providing a common interface for session and global history to
access elements like URI, title and a enumerator so that an embeddor can
provide his own UI for global and session History.  I have proposed some ideas
in  news://news.mozilla.org/3A9C3DD6.2000501@netscape.com and
news://news.mozilla.org/3AA03A3A.5040906@netscape.com.

The patch attached here is taking us closer to what is posted there. Please take
a look at it and provide your inputs.
(Assignee)

Comment 6

17 years ago
cool. fix is in.. thanks for the pointers radha, I'll take a look
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED

Comment 7

17 years ago
code-level fix, waiving my hands and marking VERIFIED Fixed
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.