I think looking at performance is the right thing to do. Please try to use the GetMirror API in the code in V8Proxy.cpp, before adding devtools specific functions to the V8 debugger API. Remember that you can compile the JavaScript function taking the FunctionMirror object once. If using the mirror has a performance overhead then maybe providing more direct access to the script/line for a function on the FunctionMirror could solve the problem.
If more features are needed in the debugger API I think the API should be more general, like providing a full mirror like API. Regards, Søren On Tue, Feb 2, 2010 at 18:26, Ilya Tikhonovsky <[email protected]> wrote: > Hi, > > Please see preliminary patch for third step. > > https://bugs.webkit.org/show_bug.cgi?id=33995 > > as far as that portion of code will be called a number of times I think it > would be nice to have a performance metrics for these possible > implementations. > Yury: Where I can implement this perf test? > Søren: in context of timeline implementation it does not necessary to have > the position it the row because we only select the corresponding source > line, > but probably it will be useful for some other use cases. > The only situation when the position is really useful is an obfuscated > code. > > > <https://bugs.webkit.org/show_bug.cgi?id=33995> > Regards, > Tim. > > > > On Tue, Feb 2, 2010 at 6:55 AM, <[email protected]> wrote: > >> On 2010/02/02 00:41:55, loislo wrote: >> >>> A patch for v8 for extending Timeline panel's functionality. >>> >> Could you upload the patch where this new method is going to be used so >> that >> it's easier to evaluate whether the new API method is necessary? >> >> >> >> http://codereview.chromium.org/565007 >> > > -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
