Closed
Bug 1505031
Opened 6 years ago
Closed 6 years ago
JavaScript error: aboutPerformance.js, ReferenceError: event is not defined
Categories
(Toolkit :: Performance Monitoring, defect)
Toolkit
Performance Monitoring
Tracking
()
RESOLVED
FIXED
mozilla65
People
(Reporter: florian, Assigned: florian)
References
Details
Attachments
(1 file)
1.80 KB,
patch
|
Felipe
:
review+
jcristau
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
[Tracking Requested - why for this release]: this code works on Nightly but will fail on beta and release. JavaScript error: chrome://global/content/aboutPerformance.js, line 1104: ReferenceError: event is not defined And same error at line 1158. This happens when dom.window.event.enabled is false, which is true for release builds since bug 1493869.
Assignee | ||
Comment 1•6 years ago
|
||
Attachment #9022944 -
Flags: review?(felipc)
Updated•6 years ago
|
Attachment #9022944 -
Flags: review?(felipc) → review+
Pushed by florian@queze.net: https://hg.mozilla.org/integration/mozilla-inbound/rev/1a62566c1baf stop using the global 'event' object in about:performance, r=felipe.
Comment 3•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/1a62566c1baf
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
status-firefox65:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla65
Assignee | ||
Comment 4•6 years ago
|
||
Comment on attachment 9022944 [details] [diff] [review] Patch [Beta/Release Uplift Approval Request] Feature/Bug causing the regression: 1491384, 1496782 User impact if declined: Clicking or double clicking won't work (and will produce a JS error in the browser console) in about:performance on release builds. Is this code covered by automated tests?: No Has the fix been verified in Nightly?: No Needs manual test from QE?: No If yes, steps to reproduce: List of other uplifts needed: none Risk to taking this patch: Low Why is the change risky/not risky? (and alternatives if risky): Trivial fix String changes made/needed: none
Attachment #9022944 -
Flags: approval-mozilla-beta?
Comment 5•6 years ago
|
||
Comment on attachment 9022944 [details] [diff] [review] Patch trivial fix for new about:performance, approved for 64.0b8
Attachment #9022944 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Comment 6•6 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-beta/rev/4bf79d20c644
Updated•6 years ago
|
Updated•6 years ago
|
Flags: qe-verify-
You need to log in
before you can comment on or make changes to this bug.
Description
•