Closed
Bug 1512046
Opened 6 years ago
Closed 6 years ago
Click on callstack grouped frames shouldn't propagate
Categories
(DevTools :: Console, defect, P1)
Tracking
(firefox65 verified)
VERIFIED
FIXED
Firefox 65
Tracking | Status | |
---|---|---|
firefox65 | --- | verified |
People
(Reporter: nchevobbe, Assigned: nchevobbe)
Details
Attachments
(1 file)
**Steps to reproduce** 1. Open the console 2. Evaluate the following: ``` x = new Error("ooops") x.stack = "a@http://exampl.com:1:1\n" + "grouped@http://react.js:1:1\n" + "grouped2@http://react.js:1:1"; console.error(x) ``` 3. You should see a callstack with a React group, click on it **Expected results** The group is expanded **Actual results** The group is expanded, but so is the message (the arrow icon on the left is rotated, and the console.error callstack is displayed under the react callstack) --- This is because the group click propagates, which shouldn't.
Assignee | ||
Comment 1•6 years ago
|
||
This also adds a console test to make sure we don't regress this in the future.
Pushed by nchevobbe@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/a9c528fe96d5 Stop propagation of callstack group click; r=Honza.
Comment 3•6 years ago
|
||
Backed out for failing devtools/client/webconsole/test/mochitest/browser_webconsole_error_with_grouped_stack.js Push with failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&selectedJob=215835772&revision=a9c528fe96d5df7ee16d1c9d8f84c03b04ef850b Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=215835772&repo=autoland&lineNumber=4154 Backout: https://hg.mozilla.org/integration/autoland/rev/c9ff07bf8ef3e76000e14e51ffbcd65dbafcd441
Flags: needinfo?(nchevobbe)
Assignee | ||
Comment 4•6 years ago
|
||
Erf, probably Bug 1508549 caused that. I fixed the test and pushed to TRY again: https://treeherder.mozilla.org/#/jobs?repo=try&revision=6e36591434de912c09c10e7d0d0c97c79dbcbfdd
Flags: needinfo?(nchevobbe)
Assignee | ||
Comment 5•6 years ago
|
||
latest try push https://treeherder.mozilla.org/#/jobs?repo=try&revision=f5e4c2802d9f76fb810d51f38c82b8bc7253309b
Pushed by nchevobbe@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/1d93612673dc Stop propagation of callstack group click; r=Honza.
Comment 7•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/1d93612673dc
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
status-firefox65:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 65
Updated•5 years ago
|
Flags: qe-verify+
Comment 8•5 years ago
|
||
I reproduced this issue using Fx 65.0a1, build ID: 20181204215402, on Windows 10 x64 and Ubuntu 18.04.1 LTS.
I can confirm this issue is fixed, I verified using Fx 65.0b12, build ID: 20190117232427, on Windows 10 x64, macOS X 10.13 and Ubuntu 18.04.1 LTS.
You need to log in
before you can comment on or make changes to this bug.
Description
•