commit | 5d76e33d69fa134bf0110781678feac57ae83e5c | [log] [tgz] |
---|---|---|
author | Chris Pickel <sfiera@chromium.org> | Tue May 15 12:18:57 2018 |
committer | Commit Bot <commit-bot@chromium.org> | Tue May 15 12:18:57 2018 |
tree | d626ba9bdfd9908564b35ae68fb1510d89ce4348 | |
parent | a17bbb370bf5597c6d00f2ef6a8bea3ca6e16e38 [diff] |
Revert "Create BrowserThread::IO thread before browser main loop to start" This reverts commit fc95501fb29e3e7487fff98c17e6aaba1046b330. Reason for revert: Findit (https://goo.gl/kROfz5) identified this CL at revision 557680 as the culprit for introducing flakiness in the tests as shown on: https://findit-for-me.appspot.com/waterfall/flake/flake-culprit?key=ag9zfmZpbmRpdC1mb3ItbWVyQwsSDEZsYWtlQ3VscHJpdCIxY2hyb21pdW0vZmM5NTUwMWZiMjllM2U3NDg3ZmZmOThjMTdlNmFhYmExMDQ2YjMzMAw Original change's description: > Create BrowserThread::IO thread before browser main loop to start > ServiceManager. > > We need a thread to post/execute tasks when starting the > ServiceManager. This thread needs to be created before the browser > main loop is initialized, and will be registered as the > BrowserThread::IO thread which is currently used by ServiceManager > connections. > > The creation of such a thread is moved to service_manager::main via > MainDelegate::CreateIOThreadAndGetTaskRunner(). Since it requires no > thread created before calling fork() on posix, we also move the setup > of sandbox before creating the IO thread. > > Bug: 740677, 729596 > Change-Id: I23ef57eb52bfb1eb363682dadf98c571c12afcd1 > Reviewed-on: https://chromium-review.googlesource.com/969098 > Commit-Queue: Xi Han <hanxi@chromium.org> > Reviewed-by: John Abd-El-Malek <jam@chromium.org> > Reviewed-by: Gabriel Charette <gab@chromium.org> > Cr-Commit-Position: refs/heads/master@{#557680} TBR=gab@chromium.org,jam@chromium.org,hanxi@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 740677, 729596 Change-Id: Ia548067acbf640010f4c8fbed29a0012a274af05 Reviewed-on: https://chromium-review.googlesource.com/1059167 Reviewed-by: Chris Pickel <sfiera@chromium.org> Commit-Queue: Chris Pickel <sfiera@chromium.org> Cr-Commit-Position: refs/heads/master@{#558668}
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 .