Open Bug 1226594 Opened 9 years ago Updated 1 year ago

Pseudo-element unselected and rule-view shows outdated styles when a media-query changes the selected pseudo-element

Categories

(DevTools :: Inspector: Rules, defect, P3)

defect

Tracking

(Not tracked)

People

(Reporter: pbro, Unassigned)

References

(Blocks 1 open bug)

Details

(Whiteboard: [dt-q])

Attachments

(1 file)

Attached file doc_media_queries_2.html (deleted) —
Once bug 1222409 is fixed, the rule-view will once again refresh when the page is resized (which is useful when media-queries are defined). However, we need to fix the following use case: - open the attached HTML page - select the ::before pseudo element - have the inspector with the rule-view open - resize the page until a new media-query matches => the rule-view is empty and the follow error is thrown: Got an addition of an actor we didn't know about: server1.conn0.child1/domnode55 Because the media-query changes the pseudo-element, we get mutation events, but at the same time, the rule-view tries to refresh the styles for an element that's no longer there. (at least, that's what I think is happening, this needs more investigation)
Triaging (filter on CLIMBING SHOES).
Priority: -- → P2
Whiteboard: [btpp-fix-later]
Whiteboard: [btpp-fix-later] → [btpp-fix-later][todo-mr]
Filter on HOTFROG.
Whiteboard: [btpp-fix-later][todo-mr] → [btpp-fix-later]
Component: Developer Tools: Inspector → Developer Tools: CSS Rules Inspector
Product: Firefox → DevTools

The problem changed a bit, the rule-view is no longer empty. The ::before element becomes unselected (and no other element gets selected). The rule-view still contains the old ::before style.
So it's a bit better today, but we should still auto-reselect the ::before element.

Furthermore, the following errors appear in the browser console when this happen:

Error while calling actor 'pagestyle's method 'getLayout' node is null Actor.js:91:13
getLayout@resource://devtools/server/actors/styles.js:933:5
handler@resource://devtools/shared/protocol/Actor.js:154:37
onPacket@resource://devtools/server/debugger-server-connection.js:378:58
receiveMessage@resource://devtools/shared/transport/child-transport.js:66:16
Actor.js:96:15
Error while calling actor 'pagestyle's method 'getApplied' node is null Actor.js:91:13
getApplied@resource://devtools/server/actors/styles.js:585:5
handler@resource://devtools/shared/protocol/Actor.js:154:37
onPacket@resource://devtools/server/debugger-server-connection.js:378:58
receiveMessage@resource://devtools/shared/transport/child-transport.js:66:16
Actor.js:96:15
Protocol error (unknownError): node is null box-model.js:228:19
Protocol error (unknownError): node is null utils.js:194:11
Protocol error (unknownError): node is null utils.js:194:11
Error while calling actor 'pagestyle's method 'getApplied' node is null Actor.js:91:13
getApplied@resource://devtools/server/actors/styles.js:585:5
handler@resource://devtools/shared/protocol/Actor.js:154:37
onPacket@resource://devtools/server/debugger-server-connection.js:378:58
receiveMessage@resource://devtools/shared/transport/child-transport.js:66:16
Actor.js:96:15
Protocol error (unknownError): node is null utils.js:194:11
Protocol error (unknownError): node is null utils.js:194:11
No longer blocks: top-inspector-bugs
Priority: P2 → P3
Whiteboard: [btpp-fix-later] → [dt-q]
Summary: rule-view empty when pseudo-element selected and media-queries change the content and window is resized → Pseudo-element unselected and rule-view shows outdated styles when a media-query changes the selected pseudo-element
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: