Closed
Bug 1232328
Opened 9 years ago
Closed 8 years ago
HAR Viewer redeclares functions; not ES6 compatible
Categories
(Core :: JavaScript Engine, defect)
Tracking
()
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
firefox44 | --- | unaffected |
firefox45 | - | affected |
People
(Reporter: mcmanus, Assigned: nihsanullah)
References
Details
(Keywords: regression)
http://www.softwareishard.com/har/viewer/ renders as blank page in nightly 45 linux x64 Built from https://hg.mozilla.org/mozilla-central/rev/99137d6d4061f408ae0869122649d8bdf489cc30 looks fine on current 44
WFM on https://hg.mozilla.org/mozilla-central/rev/9aa2dae27ae5bb4e0f9db2945a87068c4df073be
Keywords: regression,
regressionwindow-wanted
Regressed by bug 1071646: https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=d55129069a1f56f63573282911ad2dac17d6e7b1&tochange=ffd21df83fee3ee19c894df4f3b55e58c9f25e58
Blocks: 1071646
status-firefox44:
--- → unaffected
status-firefox45:
--- → affected
tracking-firefox45:
--- → ?
Component: DOM: Core & HTML → JavaScript Engine
Flags: needinfo?(shu)
Keywords: regressionwindow-wanted
Version: unspecified → 45 Branch
Comment 3•9 years ago
|
||
Reduce: with ({}) { function Pie() {} var Pie = "test"; }
Comment 4•9 years ago
|
||
For now, block-scoped functions and Annex B semantics have been backed out for breaking the web (Google Inbox being most prominent): https://hg.mozilla.org/integration/mozilla-inbound/rev/d2bec6ed7b30 Keeping NI open so I can investigate for when we re-land.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Comment 5•9 years ago
|
||
Actually let me reopen and NI Jan, this is a bug in his code. Jan, |function Pie() {}| and |var Pie = ...| in HAR Viewer in webapp/scripts/pageState.js break in ES6 because functions are block-scoped. Could you fix? I would've filed an issue on Google Code but there was some warning about how the project is archived and read-only.
Status: RESOLVED → REOPENED
Flags: needinfo?(shu) → needinfo?(odvarko)
Resolution: FIXED → ---
Updated•9 years ago
|
Summary: http://www.softwareishard.com/har/viewer/ renders as blank page → HAR Viewer redeclares functions; not ES6 compatible
Comment 6•9 years ago
|
||
Thanks for the note! The project is on github.com Here is the related issue + patch: https://github.com/janodvarko/harviewer/issues/50 How can I test this? If I understand the breaking changes have been backed out? Honza
Flags: needinfo?(odvarko) → needinfo?(shu)
Comment 7•9 years ago
|
||
Ah, forgot to note, the patch is in the repo (master), but not yet released (online) on the site. Honza
Comment 8•9 years ago
|
||
We can sit on it for now. I'll try to reland the breaking changes in a week or two when Inbox gets fixed. Then we can verify the patch fixes the problem.
Flags: needinfo?(shu)
Comment 11•8 years ago
|
||
Google Inbox has been fixed and functions-in-block has relanded. If :Honza has merged his tip fix we can close.
Assignee | ||
Comment 12•8 years ago
|
||
Thanks Shu. Honza let us know how it goes.
Flags: needinfo?(nihsanullah) → needinfo?(odvarko)
Assignee | ||
Updated•8 years ago
|
Assignee: nobody → nihsanullah
Comment 13•8 years ago
|
||
Yes, fixed and the online version is updated http://janodvarko.cz/har/viewer/ Honza
Flags: needinfo?(odvarko)
Updated•8 years ago
|
Status: REOPENED → RESOLVED
Closed: 9 years ago → 8 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•