Bug 126819

Summary: ScriptDebugServer should use a separate member function for its timer handler
Product: WebKit Reporter: Anders Carlsson <andersca>
Component: New BugsAssignee: Anders Carlsson <andersca>
Status: RESOLVED FIXED    
Severity: Normal    
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch sam: review+

Description Anders Carlsson 2014-01-11 11:16:30 PST
ScriptDebugServer should use a separate member function for its timer handler
Comment 1 Anders Carlsson 2014-01-11 11:17:50 PST
Created attachment 220940 [details]
Patch
Comment 2 Anders Carlsson 2014-01-11 11:27:24 PST
Committed r161771: <http://trac.webkit.org/changeset/161771>
Comment 3 Joseph Pecoraro 2014-01-11 13:44:33 PST
Comment on attachment 220940 [details]
Patch

Although I don't disagree with the change. What is the reason why here?

In any case, we want to remove the Timer, since this will need to be pushed down into JavaScriptCore soon anyways (which doesn't have a platform/Timer and we should be able to avoid it).