350849 - User agent/build id missing from about:

Status: RESOLVED FIXED

(Firefox :: General, defect)

Description

[Simon Bünzli]

This might well be a DUPE since the issue has existed for over a week now.

Steps to reproduce:
1. Open about:

Actual result:
Error: uncaught exception: Permission denied to get property UnnamedClass.classes

Expected result:
Get an easily copyable version of the user agent (incl. build id).

Comment 1

[Nick Thomas [:nthomas] (UTC+12)]

CC'ing zach, since bug 345993 changed this code.

Comment 2

[Nickolay_Ponomarev]

bug 349985?

Comment 3

[Simon Bünzli]

This is rather a regression from bug 346186 and should be fixed by either backing out that code or at least try-catching it as long as bug 349985 isn't fixed.

Comment 4

[Zach Lipton [:zach]]

Yeah, this is 349985. I'm glad to see someone noticed this.

*** This bug has been marked as a duplicate of 349985 ***

*** This bug has been marked as a duplicate of 349985 ***

Comment 5

[Simon Bünzli]

No duplicate (unless you want to morph that other bug). I don't care what kind of privileges about: gets - as long as I get my user agent/build id displayed.

Comment 6

[Simon Bünzli]

Attachment: graciously handle lacking privileges

Drivers: This low-risk patch adds a try-catch block so that the user agent in about: is displayed even when Components.classes isn't available.

Comment 7

[Mike Connor [:mconnor]]

Comment on attachment 236226 [details] [diff] [review]
graciously handle lacking privileges

r+a=me, low risk patch to something we don't expose in the UI, and is fundamentally correct and safe

Comment 8

[Reed Loden [:reed]]

I'd also like to see a patch that removes the release notes link text entirely if it doesn't have the needed permission to display the link. :)

Comment 9

[Simon Bünzli]

(In reply to comment #8)
> I'd also like to see a patch that removes the release notes link text entirely
> if it doesn't have the needed permission to display the link. :)

You've got better chances of getting that wish granted when filing a new bug toward that end. ;)

Comment 10

[:Gavin Sharp [email: [e-mail]]]

mozilla/toolkit/content/about.xhtml 	1.20
mozilla/toolkit/content/about.xhtml 	1.8.8.11