1580064 - Replace ObjectInspector symbol with prefix

Status: RESOLVED FIXED

(DevTools :: Console, task)

Description

[Jason Laster [:jlast]]

What were you doing?

The Object Inspector symbol is used to avoid collision with object properties. The downside, is that once a path is created, it can no longer be re-created.

One possible work around is to use a prefix, which would allow the paths to be re-generated and help prevent collisions.

Another way, is to re-use the original symbol when making new copies of loaded objects. This might be sufficient

Comment 1

[Jason Laster [:jlast]]

Attachment: Bug 1580064 - Replace ObjectInspector symbol with better delimeter. r=nchevobbe

Comment 2

[Jason Laster [:jlast]]

Another way, is to re-use the original symbol when making new copies of loaded objects. This might be sufficient

When I wrote this, i was thinking the symbol was stored in redux. This will be difficult because we're creating new nodes for newly loaded properties. I think the easiest thing to do is to get remove the Symbol.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8bb7084014ff
Replace ObjectInspector symbol with better delimeter. r=nchevobbe

Comment 4

[Arthur Iakab [arthur_iakab]]

https://hg.mozilla.org/mozilla-central/rev/8bb7084014ff