The error count button shows wrong number of errors
Categories
(DevTools :: Console, defect, P2)
Tracking
(firefox86 fixed)
Tracking | Status | |
---|---|---|
firefox86 | --- | fixed |
People
(Reporter: Honza, Assigned: nchevobbe)
References
Details
Attachments
(1 file)
The error count button is sometimes not synced with the actual amount of errors on the page.
STRs:
- Load http://janodvarko.cz/tests/fission/case10/index.html
- Open DevTools (I have Network the default, might be important that it isn't the Console panel)
- Reload the page (F5), the error count button should show 2 (try several times)
- Close and open DevTools (hit F12 twice)
- Reload the page, the error count button value is increased every time you reload the page (no reset)
Honza
Assignee | ||
Comment 1•4 years ago
|
||
Indeed we don't clear the error count on navigation from the toolbox.
If the console is enabled, we benefit from the fact that we hooked the "message clear" action to the update of the error count.
But if the another panel is displayed, we don't trigger that code and the counter keeps growing.
For simplicity's sake, I think that we should always clear the error count when navigating
Assignee | ||
Comment 2•4 years ago
|
||
We used to not do anything on navigation for the error count
at the toolbox level, but the test we had to check that the
count was reset on navigation was working; this is because
there's a hook in the console panel to clear the error count
when the console is cleared, and in the test, the console panel
was selected.
This patch fixes that and adds a new test that run some assertion
on reload, without ever enabling the console panel.
Since some assertions seem redundant with the test we already had,
we remove them from the old test.
Updated•4 years ago
|
Comment 4•4 years ago
|
||
bugherder |
Description
•