Status

P3
enhancement
RESOLVED FIXED
19 years ago
5 years ago

People

(Reporter: rbs, Assigned: caillon)

Tracking

Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

19 years ago
RFE to display a clickable list of all the supported about:*

Updated

19 years ago
Target Milestone: --- → Future

Comment 1

19 years ago
Neat idea.
Keywords: helpwanted

Comment 2

18 years ago
I'd like to do it.  :)

Comment 3

18 years ago
mass move, v2.
qa to me.
QA Contact: tever → benc

Comment 4

18 years ago
See also bug 87513, error page for about:foo (where foo isn't recognized).

Comment 5

18 years ago
does this belong in the help component?

Comment 6

18 years ago
I can do this, I will make one based on all that is currently in the tree.

Comment 7

18 years ago
Having about:about would certainly make my life easier, I would not need to 
write a test document for about by hand.
Keywords: mozilla1.0

Updated

18 years ago
Blocks: 96712

Comment 8

17 years ago
Faking it using Mozilla keywords!
http://www.mozilla.org/docs/end-user/keywords.html

Create a bookmark as normal to whatever it is you want
Then go to bookmark properties
set the keyword to 
about:about

Now if you type about:about in the location bar mozilla will automagically
follow your link.  


Comment 9

17 years ago
I also think this should be included under the Help menu, since it should be
pretty visible in order to be effective.

Comment 10

16 years ago
-> XP apps. 

Someone needs to create content in a patch and checkin.

Not much networking to be done here...
Assignee: gagan → sgehani
Component: Networking → XP Apps
QA Contact: benc → paw
(Reporter)

Comment 11

16 years ago
Just stumbled in bug 140034: about:buildconfig

These days, there are so many about:* that it is virtually impossible to
discover/stumble on all of them, let alone remember them. It is getting to the
point that this bug isn't just an 'enhancement' anymore. Not being able to
discover the about:* options means that not being able to use them.
Taking.  I just hacked up a quick implementation of this.  I'll need to do some
cleaning up before I post an actual patch, but should be out the door relatively
soon.
Assignee: sgehani → caillon
Keywords: helpwanted, mozilla1.0
Target Milestone: Future → ---
Okay, I just stumbled across a minor roadblock. 
http://lxr.mozilla.org/mozilla/source/caps/src/nsScriptSecurityManager.cpp#1230
prevents me from linking to most about: URLs.  What I have will display all
abouts and give you links, but only a couple (about:, about:blank,
about:mozilla) are actually clickable....
Created attachment 121598 [details] [diff] [review]
First crack at this

This seems to play nicely now.
Comment on attachment 121598 [details] [diff] [review]
First crack at this

Oh, btw:  I don't really care much to do work beyond where I have it other than
review comment updates.  If someone wants to make this localizable, or figure
out a way to tell if a link is unusable as listed (such as about:cache-entry
which requires ?querystring params), be my guest.  I just want to get a review,
super-review, and once I do, I'll get a security review and land this, and any
other work can be done at that point.  Unless someone wants to modify the
patch.


>Index: xpfe/global/resources/content/aboutAbout.html
>===================================================================

>+  var referrer;
>+  try {
>+    referrer = Components.classes["@mozilla.org/network/standard-url;1"].createInstance(Components.interfaces.nsIURI);
>+    referrer.spec = "chrome://global/content/aboutAbout.html";
>+  } catch (e) {
>+    referrer = { spec: "chrome://global/content/aboutAbout.html" };
>+  }


The above code snippet was inadvertently left in and should be removed, btw.
Created attachment 128064 [details] [diff] [review]
Patch

Also includes a minor change to caps to allow chrome (resource) uris to link to
about: uris.
Attachment #121598 - Attachment is obsolete: true

Comment 17

16 years ago
Comment on attachment 128064 [details] [diff] [review]
Patch

r/sr=darin

this looks good to me.	you probably want to get someone more familiar
with xpfe-land like jag to review this as well.
Attachment #128064 - Flags: superreview+
Attachment #128064 - Flags: review?(bryner)
Attachment #128064 - Flags: review?(bryner) → review+
Let there be about:about.
Status: NEW → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED
re
-        { "about",           DenyProtocol   },
+        { "about",           ChromeProtocol },
and comment 13:

Who made the security review? IIRC, we had exploits in about: and strong
restrictions were the fix.
Darin looked at it, but here's the explanation.

The exploits were because it was set to AllowProtocol meaning the world could
link to about: URIs.  This is completely safe because it still restricts the
linking to only ChromeProtocol URIs (only chrome: or resource:)

For reference, the about: exploit bug was bug 91714.
Yes, it seems to be safe.
Thanks for the answer.
FYI, I was thinking about bug 134315 (and others, IIRC).
Product: Core → Mozilla Application Suite

Updated

10 years ago
Blocks: 220253
You need to log in before you can comment on or make changes to this bug.