Closed
Bug 351601
Opened 18 years ago
Closed 18 years ago
Accessing the new navigator.buildID in a build w/o NS_BUILD_ID set shouldn't throw exception.
Categories
(Core :: DOM: Core & HTML, defect)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
FIXED
People
(Reporter: jst, Assigned: jst)
References
Details
(Keywords: fixed1.8.1)
Attachments
(1 file)
593 bytes,
patch
|
sicking
:
review+
sicking
:
superreview+
beltzner
:
approval1.8.1+
|
Details | Diff | Splinter Review |
Right now if you access navigator.buildID in anything other than an official build you'll get an exception. This is not a big problem for code in general, but it's already been reported that one web app (zimbra) triggers this and the way it does it is by enumerating the properties on the navigator object.
The fix for this is trivial, patch coming up.
Assignee | ||
Comment 1•18 years ago
|
||
Assignee: general → jst
Status: NEW → ASSIGNED
Attachment #237006 -
Flags: superreview?(bugmail)
Attachment #237006 -
Flags: review?(bugmail)
Assignee | ||
Updated•18 years ago
|
Flags: blocking1.8.1?
Assignee | ||
Comment 2•18 years ago
|
||
The buildID navigator property was introduced by bug 345993.
Comment 3•18 years ago
|
||
See also bug 348069
Comment 4•18 years ago
|
||
*** Bug 348069 has been marked as a duplicate of this bug. ***
Attachment #237006 -
Flags: superreview?(bugmail)
Attachment #237006 -
Flags: superreview+
Attachment #237006 -
Flags: review?(bugmail)
Attachment #237006 -
Flags: review+
Assignee | ||
Updated•18 years ago
|
Attachment #237006 -
Flags: approval1.8.1?
Assignee | ||
Comment 5•18 years ago
|
||
Fix landed on the trunk.
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Updated•18 years ago
|
Flags: blocking1.8.1? → blocking1.8.1+
Comment 6•18 years ago
|
||
Comment on attachment 237006 [details] [diff] [review]
Don't throw if NS_BUILD_ID is not set.
a=beltzner on behalf of 181drivers
Attachment #237006 -
Flags: approval1.8.1? → approval1.8.1+
Comment 8•18 years ago
|
||
RCS file: /cvsroot/mozilla/testing/mochitest/tests/test_bug351601.html,v
done
Checking in test_bug351601.html;
/cvsroot/mozilla/testing/mochitest/tests/test_bug351601.html,v <-- test_bug351601.html
initial revision: 1.1
done
Flags: in-testsuite+
See Also: → https://launchpad.net/bugs/63857
Updated•6 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•