[Webkit-unassigned] [Bug 28983] WebInspector: Wrap primitive values (as objects) in InspectorController::wrap

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=28983





--- Comment #4 from Patrick Mueller <pmuellr at yahoo.com>  2009-09-08 06:29:40 PDT ---
(In reply to comment #3)
> > I can imagine other uses within the debugger for the ability to get the
> > properties of an arbitrary object, so I think this needs to be a 1st class
> > capability.  For instance, I've been thinking of writing some kind of object
> > explorer where you could see a graphical layout of an object, it's properties,
> > their properties, etc.  Such a tool would also need a way to get a list of
> > properties of an arbitrary object.
> 
> No, it would not. There are generic capabilities that would be reused for that.
> Like dumping object structure in console (by means of dir(object)), allows
> getting and rendering arbitrary objects. You don't need to write an object
> explorer - it is already in place.

I'm suggesting I might want to write an alternative UI to explore objects and
their properties.  I think you're suggesting that no one will ever need this. 
Which is also saying that the presentation can't be separated from the model,
in terms of getting the properties of an object - there IS only one way to ever
do this and render the UI.

I'm guessing that with the new proxy stuff here, there are memory- and
performance-related impacts to making calls like
InspectorController.getProperties(); namely, every property that comes back
over as a proxy will be, for the life of the session, tracked in some kind of
proxy-handle/object map.  Is that the concern with exposing a capability of
getting the properties from an object?  It seems like a pretty basic API that
you want in your toolkit when building debugging tools.  Do we just need to be
careful when using these APIs?  Is there some cleanup we can perform?  Do we
need to add weak support to the JS engines so we can automagically clean up the
mapping tables?

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.