Closed Bug 1321551 Opened 8 years ago Closed 8 years ago

Firefox hangs when clicking on event listener icon in page inspector

Categories

(DevTools :: Inspector, defect)

49 Branch
defect
Not set
critical

Tracking

(firefox-esr45 unaffected, firefox50- wontfix, firefox51 wontfix, firefox52 wontfix, firefox53 fixed)

RESOLVED FIXED
Firefox 53
Tracking Status
firefox-esr45 --- unaffected
firefox50 - wontfix
firefox51 --- wontfix
firefox52 --- wontfix
firefox53 --- fixed

People

(Reporter: fabrizio, Assigned: miker)

References

Details

(Keywords: hang, regression, reproducible)

Attachments

(1 file)

Attached image cap.PNG
User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:50.0) Gecko/20100101 Firefox/50.0
Build ID: 20161129173726

Steps to reproduce:

1. Go to http://editor.swagger.io/
2. Click 'got it' on the page
3. Open page inspector (HTML Pane)
4. Click on the little 'ev' icon next to the <html> tag at the top of the document source




Actual results:

Firefox hangs (hogs CPU, unresponsive)


Expected results:

List event listeners
Severity: normal → critical
Status: UNCONFIRMED → NEW
Component: Untriaged → Developer Tools: Inspector
Ever confirmed: true
Keywords: hang
Regression window:
https://hg.mozilla.org/integration/fx-team/pushloghtml?fromchange=f0210b36ff71d143c1041202f55ad924daa1c8d8&tochange=fb7111ef84f70cfba9ad445fd9fa492780401a31

412f3942777e	Michael Ratcliffe — Bug 1226548 - Event listeners tooltip breaks the syntax in popup if there's a comment before event handler in JS code. r=ochameau

[Tracking Requested - why for this release]:
Blocks: 1226548
Has Regression Range: --- → yes
Has STR: --- → yes
Version: 50 Branch → 49 Branch
This was fixed by bug 1315639.

Can you try again with the latest nightly and let us know if this works for you?
Assignee: nobody → mratcliffe
Depends on: 1315639
Flags: needinfo?(fabrizio.gotti)
Looks good in 53.0a1 (2016-12-06) (32-bit).


As an issue, it should be tracked separately.
The change is, unfortunately, too large to backport along with the test changes.
Status: NEW → RESOLVED
Closed: 8 years ago
Flags: needinfo?(fabrizio.gotti)
Resolution: --- → FIXED
Target Milestone: --- → Firefox 53
Mark 51 won't fix as the changes are too large.
This is way too late for 50
Sounds like a wontfix for 52 as well.
QA Whiteboard: [good first verify]
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: