| commit | 7b2f1e0a7a2b9dcfd5b64252249146455cb21816 | [log] [tgz] |
|---|---|---|
| author | lukasza <lukasza@chromium.org> | Sat Apr 08 05:58:03 2017 +0900 |
| committer | Qijiang Fan <fqj@google.com> | Fri Jun 05 05:47:04 2020 +0900 |
| tree | 4e3dad516b51d0a1d3b937d983d28a50d17e40f3 | |
| parent | 07f0b9d94282eecbab8630c665e68106f960e34f [diff] |
Revert of [reland] Do not block in SimpleThread::Start(). (patchset #1 id:1 of https://codereview.chromium.org/2801073005/ ) Reason for revert: Speculatively reverting hoping to resolve lots of random failures on 2 bots, starting with the builds below: https://uberchromegw.corp.google.com/i/chromium.chromiumos/builders/Linux%20ChromiumOS%20Tests%20%28dbg%29%281%29/builds/24574 https://uberchromegw.corp.google.com/i/chromium.linux/builders/Linux%20Tests%20%28dbg%29%281%29%2832%29/builds/40019 Original issue's description: > [reland] Do not block in SimpleThread::Start(). > > This CL is identical to https://codereview.chromium.org/2664953004. > The race condition that it highlighted in a TaskScheduler test the > first time that it landed was fixed by > https://codereview.chromium.org/2798373002/ > > There is no good reason to block in SimpleThread::Start(). > In addition to introducing unnecessary latency, blocking in > Start() prevents a non-joinable DelegateSimpleThread from > being deleted from Run() without external synchronization > (it is wrong to delete SimpleThread while Start() is waiting > on the |event_| member). > > BUG= > TBR=gab@chromium.org,jam@chromium.org,henrika@chromium.org > CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel > > Review-Url: https://codereview.chromium.org/2801073005 > Cr-Commit-Position: refs/heads/master@{#462912} > Committed: https://chromium.googlesource.com/chromium/src/+/d5acadb2d99dfbbde0cbe251173b566ceb5d3441 TBR=gab@chromium.org,jam@chromium.org,henrika@chromium.org,fdoray@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG= Review-Url: https://codereview.chromium.org/2804393002 Cr-Commit-Position: refs/heads/master@{#462998} CrOS-Libchrome-Original-Commit: 6710dbd5124741db004176153c87c609435e7788