[Webkit-unassigned] [Bug 53003] Web Inspector: [JSC] implement setting breakpoints by line:column

[bugzilla-daemon at webkit.org]

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


Vsevolod Vlasov <vsevik at chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |vsevik at chromium.org




--- Comment #22 from Vsevolod Vlasov <vsevik at chromium.org>  2012-06-06 13:06:06 PST ---
> > > Source/WebCore/bindings/js/ScriptDebugServer.h:153
> > > +    mutable BreakpointsInLine m_lastHitScriptBreakpoints;
> > 
> > Why do you need these fields to be mutable?
> Because I think it's most appropriate to change it in "hasBreakpoint", but "hasBreakpoint" is a const function. I just don't want to break the original design.

hasBreakpoint is used in one place only as far as I can see, doesn't look like a valuable design to keep to me :)

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