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

Status: NEW

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

Description

[Patrick Brosset <:pbro>]

Attachment: doc_media_queries_2.html

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)

Comment 1

[Patrick Brosset <:pbro>]

Triaging (filter on CLIMBING SHOES).

Comment 2

[Mike Ratcliffe [:miker] [:mratcliffe] [:mikeratcliffe]]

Filter on HOTFROG.

Comment 3

[Patrick Brosset <:pbro>]

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