[Webkit-unassigned] [Bug 126598] Web Inspector: Move InjectedScript classes into JavaScriptCore

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Jan 7 18:26:56 PST 2014


--- Comment #34 from Joseph Pecoraro <joepeck at webkit.org>  2014-01-07 18:24:39 PST ---
(In reply to comment #22)
> (From update of attachment 220562 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=220562&action=review
> >> Source/JavaScriptCore/inspector/JSInjectedScriptHost.cpp:85
> >> +
> > 
> > Because you have a non-trivial destructor, I believe you need this in the class definition:
> > 
> >     static const bool needsDestruction = true;
> > 
> > You should set a breakpoint on this function and verify that it is actually called.
> Good find! I had tested this worked as expected when I was inspecting a JSContext but I'll add this and verify when debugging a Page and make sure I didn't regress anything there.

Hmm, adding this line makes the Inspector crash on construction of the object. I'll look into this tomorrow.

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