Bug 106501

Summary: [WK2] Get rid of LocalTerminationDisabler
Product: WebKit Reporter: Alexey Proskuryakov <ap>
Component: WebKit2Assignee: Alexey Proskuryakov <ap>
Status: RESOLVED FIXED    
Severity: Normal CC: andersca, beidson, eric.carlson, feature-media-reviews, sam, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
proposed patch andersca: review+

Alexey Proskuryakov
Reported 2013-01-09 16:39:27 PST
LocalTerminationDisabler stops auto-termination timer in a scope. But the timer can't possibly fire inside the scope anyway, unless an event loop is entered recursively. I don't think that we ever do that, or ever will. So, I think that LocalTerminationDisabler is a complete no-op.
Attachments
proposed patch (13.34 KB, patch)
2013-01-09 16:41 PST, Alexey Proskuryakov 		andersca: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Alexey Proskuryakov
Comment 1 2013-01-09 16:41:29 PST
Created attachment 182011 [details] proposed patch
Alexey Proskuryakov
Comment 2 2013-01-09 16:47:58 PST
Committed <http://trac.webkit.org/changeset/139257>.
Note You need to log in before you can comment on or make changes to this bug.