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+

Anders Carlsson
Reported 2014-01-11 11:16:30 PST
ScriptDebugServer should use a separate member function for its timer handler
Attachments
Patch (6.79 KB, patch)
2014-01-11 11:17 PST, Anders Carlsson 		sam: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Anders Carlsson
Comment 1 2014-01-11 11:17:50 PST
Created attachment 220940 [details] Patch
Anders Carlsson
Comment 2 2014-01-11 11:27:24 PST
Committed r161771: <http://trac.webkit.org/changeset/161771>
Joseph Pecoraro
Comment 3 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).
Note You need to log in before you can comment on or make changes to this bug.