[Webkit-unassigned] [Bug 28078] WebInspector: Move object properties read / write access into InjectedScript

[bugzilla-daemon at webkit.org]

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


Pavel Feldman <pfeldman at chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #34309|0                           |1
        is obsolete|                            |
  Attachment #34321|                            |review?
               Flag|                            |




--- Comment #3 from Pavel Feldman <pfeldman at chromium.org>  2009-08-07 13:43:23 PDT ---
Created an attachment (id=34321)
 --> (https://bugs.webkit.org/attachment.cgi?id=34321)
patch

(In reply to comment #2)
> (From update of attachment 34309 [details])
> 
> 
> I think this should go into a new file and not ObjectPropertiesSection.js,
> since it is used in other places and not specific to the
> ObjectPropertiesSection.
> 

Done.

> I am not too fond of the Ref suffix. Do you like ObjectProxy better?
>

Done.

> 
> You should add a comment about this, this looks temporary.
>

Done (added TODO).

> 
> You hould add a comment about this being temporary.


Done (added TODO).

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