commit | eae7e77a64f007fd05b8362f884ee3ea78f3f40e | [log] [tgz] |
---|---|---|
author | Avi Drissman <avi@chromium.org> | Mon Dec 24 23:17:04 2018 |
committer | Commit Bot <commit-bot@chromium.org> | Mon Dec 24 23:17:04 2018 |
tree | 35058d0333142ceaf4286fa6aeacc05046e857ab | |
parent | da8098513403f8896dd9ddaa1ffc49200fd0bbab [diff] |
Reland "Oilpan: Replace Main/WorkerThreadMutex with one ActiveThreadMutex" This reverts commit 15a84b7dc0b8e783f0cec043b5eaed3b212073f8. Reason for revert: This turns out to not be the reason for the failure; relanding. Original change's description: > Revert "Oilpan: Replace Main/WorkerThreadMutex with one ActiveThreadMutex" > > This reverts commit dc3fde2e427aa046fd4f0a7a16f8fd72b9c6a96b. > > Reason for revert: fast/js/try-catch-crash.html is burning on WebKit Linux Trusty MSAN; this is a suspect > > Original change's description: > > Oilpan: Replace Main/WorkerThreadMutex with one ActiveThreadMutex > > > > TSAN reported lock-order-inversion because we were using two mutexes. Simplify the code to use just one. > > > > Bug: 915200 > > Change-Id: I5dbda4ca21cd4f5fa95ecb1c9b51922262798169 > > Reviewed-on: https://chromium-review.googlesource.com/c/1384384 > > Commit-Queue: Keishi Hattori <keishi@chromium.org> > > Reviewed-by: Kentaro Hara <haraken@chromium.org> > > Reviewed-by: Michael Lippautz <mlippautz@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#618754} > > TBR=haraken@chromium.org,keishi@chromium.org,mlippautz@chromium.org > > # Not skipping CQ checks because original CL landed > 1 day ago. > > Bug: 915200 > Change-Id: If4be6a8156292d27790a7f12a62f2d559c730236 > Reviewed-on: https://chromium-review.googlesource.com/c/1390426 > Reviewed-by: Avi Drissman <avi@chromium.org> > Commit-Queue: Avi Drissman <avi@chromium.org> > Cr-Commit-Position: refs/heads/master@{#618828} TBR=avi@chromium.org,haraken@chromium.org,keishi@chromium.org,mlippautz@chromium.org Change-Id: Iab67bab6c66d0ac8feb9cdafe9e814497142bf3c No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 915200 Reviewed-on: https://chromium-review.googlesource.com/c/1390434 Reviewed-by: Avi Drissman <avi@chromium.org> Commit-Queue: Avi Drissman <avi@chromium.org> Cr-Commit-Position: refs/heads/master@{#618845}
Chromium is an open-source browser project that aims to build a safer, faster, and more stable way for all users to experience the web.
The project's web site is https://www.chromium.org.
Documentation in the source is rooted in docs/README.md.
Learn how to Get Around the Chromium Source Code Directory Structure .