Closed
Bug 196297
Opened 22 years ago
Closed 2 years ago
typed urls with spaces in them do not enter global history
Categories
(Toolkit :: Places, defect, P5)
Tracking
()
RESOLVED
WONTFIX
People
(Reporter: nisheeth_mozilla, Unassigned)
References
()
Details
Opening as an offshoot of the discussion in bug 161531.
sessionhistoryUI.js:
+
+ var url = entryToAdd.Value;
+ if (url.indexOf(" ") == -1) {
+ var fixedUpURI = gURIFixup.createFixupURI(url, 0);
+ gGlobalHistory.markPageAsTyped(fixedUpURI.spec);
+ }
+
// Put the value as it was typed by the user in to RDF
// Insert it to the beginning of the list.
- var fixedUpURI = gURIFixup.createFixupURI(entryToAdd.Value, 0);
- gGlobalHistory.markPageAsTyped(fixedUpURI.spec);
The above patch is from attachment 70207 [details] [diff] [review], the patch that fixed bug 126320
(Invalid typed urls should be hidden and expire quickly).
nsGlobalHistory::MarkPageAsTyped() only gets called if the typed url doesn't
have a space in it. This is wrong behavior because typed js urls can have
spaces in them and should get entered into history in order to be consistent.
Can data urls have spaces in them?
sure data urls often have spaces in them:
"data:text,hello world"
also bookmark keywords will have spaces in them:
"bug 196297" where 'bug' is a keyword for a bookmark to
http://bugzilla.mozilla.org/show_bug.cgi?id=%s
Reporter | ||
Updated•22 years ago
|
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla1.4alpha
Reporter | ||
Comment 3•22 years ago
|
||
Updating summary to make it more accurate.
Summary: typed js urls with spaces in them do not enter global history → typed urls with spaces in them do not enter global history
Reporter | ||
Updated•22 years ago
|
Target Milestone: mozilla1.4alpha → mozilla1.4beta
Updated•18 years ago
|
Component: History: Session → History: Global
Updated•15 years ago
|
QA Contact: chrispetersen → history.global
Updated•7 years ago
|
Component: History: Global → Places
Product: Core → Toolkit
Target Milestone: mozilla1.4beta → ---
Updated•7 years ago
|
Assignee: nisheeth_mozilla → nobody
Status: ASSIGNED → NEW
Priority: -- → P5
Updated•2 years ago
|
Severity: normal → S3
Comment 4•2 years ago
|
||
javascript: and data: are not stored in history anyway, typed is somehow losing its importance today (we are looking into better signals for the user workflow than having typed a url, moreover today typed is applied to far too many things, see bug 1330343)
Overall this bug is not useful.
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•