[Webkit-unassigned] [Bug 27552] remember last script displayed in Scripts panel
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Wed Nov 18 09:03:37 PST 2009
https://bugs.webkit.org/show_bug.cgi?id=27552
Pavel Feldman <pfeldman at chromium.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #43433|review? |review+, commit-queue+
Flag| |
--- Comment #9 from Pavel Feldman <pfeldman at chromium.org> 2009-11-18 09:03:37 PST ---
(From update of attachment 43433)
Looks good.
One comment though - you are using InspectorController.setting() in a sync
manner, whereas we are going away from that. We now think that there is agent
(with inspector backend) and a client (frontend). Interaction between those is
supposed to be serialized and asynchronous (imagine that remote client attaches
to the agent).
So we should either convert setting to async call with a callback or extract
such methods into another interface like InspectorHost. First seems to be
simpler.
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list