Enhance embed_lite global history

RESOLVED INCOMPLETE

Status

()

RESOLVED INCOMPLETE
16 years ago
2 years ago

People

(Reporter: ccarlen, Assigned: ccarlen)

Tracking

Trunk
mozilla1.5beta
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Assignee)

Description

16 years ago
The embed_lite global history implementation only stores the URL and the last
visted date for each entry. In order to support autocomplete, some other data is
needed. The object which represents an entry can easily be expanded and they can
still be stored in memory in an nsHashTable. The thing which needs to change is
the disk file format: instead of a simple format containing 2 fields per line,
something more expandable is needed. I'm thinking XML, using expat, which is
small & fast enough.
(Assignee)

Updated

16 years ago
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla1.3beta

Updated

16 years ago
QA Contact: depstein → carosendahl
(Assignee)

Comment 1

16 years ago
Setting to milestone that's not passed.
Target Milestone: mozilla1.3beta → mozilla1.5beta

Comment 2

16 years ago
In nsEmbedGlobalHistory.cpp, the default number of new URLS untill history is
dumped to history.txt is 10:

static const PRInt32 kNewEntriesBetweenFlush = 10;

What is the reason for this? I am a K-Meleon user (which is still in development
of its global history implementation, and mostly works with the history.txt file
as of yet), and it is highly impractical to have such a set up. I have changed
the value to "1" url on my own buildIn nsEmbedGlobalHistory.cpp, the default
number of new URLS untill history is dumped to history.txt is 10:

static const PRInt32 kNewEntriesBetweenFlush = 10;

What is the reason for this? 

I am a K-Meleon user (which is still in the development of its global history
implementation, and mostly works with the history.txt file as of yet), and it is
highly impractical to have such a set up. I have changed the value to "1" URL on
my own build for us with K-Meleon.

Basically, I'm trying to figure out the rationale of waiting 10 URL before each
history dump. Thanks!
QA Contact: carosendahl → apis

Comment 3

2 years ago
Marking a bunch of bugs in the "Embedding: APIs" component INCOMPLETE in preparation to archive that component. If I have done this incorrectly, please reopen the bugs and move them to a more correct component as we don't have "embedding" APIs any more.
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → INCOMPLETE
You need to log in before you can comment on or make changes to this bug.