746221 - browser_webconsole_bug_651501_document_body_autocomplete.js fails with compartment-per-global

Status: RESOLVED FIXED

(DevTools :: Console, defect)

Description

[Bobby Holley (:bholley)]

The test asserts that the number of properties on the object is the same as number of rows in the treeview. This breaks if the content object has any expandos in it, since the expandos are only visible in the compartment that created them. In this case, the 'constructor' expando is resolved and defined by nsDOMClassInfo on content.document, but it isn't visible in PropertyPanel.jsm.

Patch coming right up. This patch also checks for properties individually, so that it's easier to see what went wrong if there's every a discrepancy.

Comment 1

[Bobby Holley (:bholley)]

Attachment: Pull properties for comparison off the prototype to avoid mismatches with expandos. v1

Attaching a patch, flagging past for review.

Comment 2

[Panos Astithas (he/him) [:past] (please ni?)]

Comment on attachment 615777 [details] [diff] [review]
Pull properties for comparison off the prototype to avoid mismatches with expandos. v1

Review of attachment 615777 [details] [diff] [review]:
-----------------------------------------------------------------

Yes, the count comparison was something I always wanted to come back and improve. However I am wondering if we need to change our approach here to get expandos to appear in the property view, somehow.

Comment 3

[Bobby Holley (:bholley)]

Pushed to m-i: http://hg.mozilla.org/integration/mozilla-inbound/rev/c209b10216fb

Comment 4

[Bobby Holley (:bholley)]

(In reply to Panos Astithas [:past] from comment #2)
> However I am wondering if we need to change our approach here to
> get expandos to appear in the property view, somehow.

I can't think of a great way to do this, offhand. The basic issue is that each chrome scope gets its own clean Xray view of content DOM objects. So if the expandos are private to the compartment that created them, there's no great way to share them with JSMs and such.

Comment 5

[(no longer active)]

https://hg.mozilla.org/mozilla-central/rev/c209b10216fb