272556 - about: is broken

Status: RESOLVED FIXED

(Firefox :: General, defect)

Description

[Steffen Wilberg]

about: is the same as about:blank.

Comment 1

[Steffen Wilberg]

chrome://global/content/about.xhtml works fine.
But nsAbout.cpp still references chrome://global/locale/about.xhtml:
http://lxr.mozilla.org/seamonkey/source/xpfe/appshell/src/nsAbout.cpp#51

That's because bug 257592 changed that on the branch only.

-> bsmedberg.

Comment 2

[Ali Ebrahim]

Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8a6) Gecko/20041202
Firefox/1.0+

about: doesn't do anything for me. It doesn't work, but it's not the same as
about:blank either. Typing it simply has no effect (i.e. if a page is loaded, it
won't clear it).

Comment 3

[Steffen Wilberg]

Comment 0 is wrong (I tried it in a new tab, which already had about:blank
loaded in it).
about: does just nothing.

Comment 4

[Ryan VanderMeulen [:RyanVM]]

Attachment: Path corrected in nsAbout.cpp

Tested and confirmed that it works.

Comment 5

[Steffen Wilberg]

Comment on attachment 168344 [details] [diff] [review]
Path corrected in nsAbout.cpp

This breaks about: for Seamonkey:
http://bonsai.mozilla.org/cvsblame.cgi?file=mozilla/xpfe/global/jar.mn&mark=97#
79

We need to decide where about.xhtml should live: in content, or in locale. Then
we can make it work (again) for both Seamonkey and Firefox.

Comment 6

[Benjamin Smedberg]

Attachment: Move about.xhtml to content

There isn't really any localizable content in this file. Let's just move it to
content/. Otherwise, we could possibly use the license.dtd scheme that the
toolkit uses, but that's arcane and I regret doing it that way.

Comment 7

[Robert Kaiser]

Some people like to translate the copyright lines for their language, but I'd
also prefer to have the document in content, and eventually move the
localizeable parts/texts to some .dtd file (like I did for the seamonkey
mailnews start file) - but that can be done later, if needed.

Comment 8

[amano]

Does the patch need a superreview?

Comment 9

[amano]

The review for the patch is here. Does it need a superreview or just a volunteer
to check it in?

Comment 10

[Alec Flett]

Comment on attachment 170488 [details] [diff] [review]
Move about.xhtml to content

sr=alecf

Comment 11

[Jeremy Priest]

It would be nice if this was committed it has both a review and superreview.
Anybody going to be able to check it in?
It would be best before we get bitrotten.

/Ieremiou

Comment 12

[Allan Beaufour]

Checked in.

Comment 13

[Peter van der Woude [:Peter6]]

Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.8b2) Gecko/20050315
Firefox/1.0+

verified

Comment 14

[Mohammad Maruf Islam]

I have reproduced this bug with Firefox Nightly 1.0 on windows 8.1 64-bit .
Mozilla/5.0 (Windows; U; Windows NT 6.2; en-US; rv:1.8a6) Gecko/20041205 Firefox/1.0+

Verified as fixed with Firefox Aurora 45.0a2 (Build ID: 20151225004009)
Mozilla/5.0 (Windows NT 6.3; WOW64; rv:45.0) Gecko/20100101 Firefox/45.0

Verified as fixed with Firefox Nightly 46.0a1 (Build ID:20151224030228)
Mozilla/5.0 (Windows NT 6.3; WOW64; rv:46.0) Gecko/20100101 Firefox/46.0