Bug 98978 - [EFL][WK2] Change the range of locking in WorkQueueEfl.cpp
Summary: [EFL][WK2] Change the range of locking in WorkQueueEfl.cpp
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit EFL (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Byungwoo Lee
URL:
Keywords:
Depends on: 98580 101015
Blocks:
  Show dependency treegraph
 
Reported: 2012-10-10 18:48 PDT by Byungwoo Lee
Modified: 2012-11-02 09:30 PDT (History)
8 users (show)

See Also:


Attachments
Patch (3.53 KB, patch)
2012-10-11 18:29 PDT, Byungwoo Lee
no flags Details | Formatted Diff | Diff
Patch (3.81 KB, patch)
2012-11-02 06:42 PDT, Byungwoo Lee
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Byungwoo Lee 2012-10-10 18:48:22 PDT
To protect suspicious/dangerous problem (a source of dead lock), locking range should be adjusted.
Comment 1 Byungwoo Lee 2012-10-11 18:29:58 PDT
Created attachment 168329 [details]
Patch
Comment 2 Byungwoo Lee 2012-10-14 23:46:01 PDT
I tested layout test and unit test on EFL, and there was no regressions.
Comment 3 Byungwoo Lee 2012-11-02 01:02:32 PDT
This needs to be rebased after Bug 101015
Comment 4 Byungwoo Lee 2012-11-02 06:42:56 PDT
Created attachment 172048 [details]
Patch
Comment 5 Mikhail Pozdnyakov 2012-11-02 07:27:01 PDT
Comment on attachment 172048 [details]
Patch

looks sane
Comment 6 WebKit Review Bot 2012-11-02 09:30:47 PDT
Comment on attachment 172048 [details]
Patch

Clearing flags on attachment: 172048

Committed r133307: <http://trac.webkit.org/changeset/133307>
Comment 7 WebKit Review Bot 2012-11-02 09:30:52 PDT
All reviewed patches have been landed.  Closing bug.