[Webkit-unassigned] [Bug 41072] New: Remove the JSLock when JSC_MULTIPLE_THREADS is disabled

[bugzilla-daemon at webkit.org]

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

           Summary: Remove the JSLock when JSC_MULTIPLE_THREADS is
                    disabled
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: All
        OS/Version: All
            Status: UNCONFIRMED
          Severity: Minor
          Priority: P2
         Component: JavaScriptCore
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: yi.4.shen at nokia.com


For some platform like Symbian, on which the JSC_MULTIPLE_THREADS is disabled, the implementation of JSLock is empty. While the JSLock are used very often in JavaScriptcall, for example, 

void JSEventListener::handleEvent(...) {
    ...
    JSLock lock(SilenceAssertionsOnly);
    ...
}

We can remove this kind of useless JSLock by using macro like following, to improve the performance and battery life for the mobile device (especially for the webkit-based widget which may keep running javascript all the time).

void JSEventListener::handleEvent(...) {
    ...
#if ENABLE(JSC_MULTIPLE_THREADS)
    JSLock lock(SilenceAssertionsOnly);
#endif
    ...
}

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