|
Revert of Reland: Lazily initialize MessageLoop for faster thread startup (patchset #5 id:160001 of https://codereview.chromium.org/1129953004/)
Reason for revert:
Massive data race reports, see https://crbug.com/489263
Original issue's description:
> Reland: Lazily initialize MessageLoop for faster thread startup
>
> Original review: https://codereview.chromium.org/1011683002/
>
> Reverted because it's suspected for following flakiness issues:
> http://crbug.com/485157 - Windows race
> http://crbug.com/485091 - Android ThreadWatcher
> http://crbug.com/485178 - interactive_ui_tests Menu* tests
>
> PS1 is the original patch set that gets reverted.
>
> BUG= 465458, 485157, 485091, 485178
> TBR=jam
>
> Committed: https://crrev.com/8b6133a69f16702a32a3c3104630c4d9ac393b7a
> Cr-Commit-Position: refs/heads/master@{#330329}
TBR=thakis@chromium.org,toyoshim@chromium.org,jam@chromium.org,kinuko@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG= 465458, 485157, 485091, 485178
Committed: https://crrev.com/787e3347eb49a59db5e6fe0374f50464f36d48e3
Cr-Commit-Position: refs/heads/master@{#330351}
|
Unified diffs |
Side-by-side diffs |
Delta from patch set |
Stats (+197 lines, -300 lines) |
Patch |
 |
M |
base/message_loop/incoming_task_queue.h
|
View
|
|
3 chunks |
+0 lines, -10 lines |
0 comments
|
Download
|
 |
M |
base/message_loop/incoming_task_queue.cc
|
View
|
|
3 chunks |
+10 lines, -26 lines |
0 comments
|
Download
|
 |
M |
base/message_loop/message_loop.h
|
View
|
|
5 chunks |
+8 lines, -37 lines |
0 comments
|
Download
|
 |
M |
base/message_loop/message_loop.cc
|
View
|
|
4 chunks |
+33 lines, -47 lines |
0 comments
|
Download
|
 |
M |
base/message_loop/message_loop_proxy_impl.h
|
View
|
|
1 chunk |
+0 lines, -3 lines |
0 comments
|
Download
|
 |
M |
base/message_loop/message_loop_proxy_impl.cc
|
View
|
|
1 chunk |
+1 line, -6 lines |
0 comments
|
Download
|
 |
M |
base/message_loop/message_pump_perftest.cc
|
View
|
|
3 chunks |
+5 lines, -0 lines |
0 comments
|
Download
|
 |
M |
base/threading/platform_thread.h
|
View
|
|
1 chunk |
+0 lines, -4 lines |
0 comments
|
Download
|
 |
M |
base/threading/platform_thread_win.cc
|
View
|
|
1 chunk |
+2 lines, -3 lines |
0 comments
|
Download
|
 |
M |
base/threading/thread.h
|
View
|
|
6 chunks |
+12 lines, -25 lines |
0 comments
|
Download
|
 |
M |
base/threading/thread.cc
|
View
|
|
7 chunks |
+97 lines, -109 lines |
0 comments
|
Download
|
 |
M |
base/threading/thread_id_name_manager_unittest.cc
|
View
|
|
3 chunks |
+6 lines, -6 lines |
0 comments
|
Download
|
 |
M |
base/threading/thread_unittest.cc
|
View
|
|
1 chunk |
+2 lines, -3 lines |
0 comments
|
Download
|
 |
M |
chrome/browser/io_thread.h
|
View
|
|
1 chunk |
+1 line, -0 lines |
0 comments
|
Download
|
 |
M |
chrome/browser/io_thread.cc
|
View
|
|
1 chunk |
+5 lines, -0 lines |
0 comments
|
Download
|
 |
M |
chrome/browser/metrics/thread_watcher_android_unittest.cc
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
 |
M |
chrome/browser/metrics/thread_watcher_unittest.cc
|
View
|
|
2 chunks |
+4 lines, -4 lines |
0 comments
|
Download
|
 |
M |
chrome/browser/sync/glue/sync_backend_registrar_unittest.cc
|
View
|
|
1 chunk |
+0 lines, -3 lines |
0 comments
|
Download
|
 |
M |
content/browser/browser_thread_impl.cc
|
View
|
|
1 chunk |
+7 lines, -1 line |
0 comments
|
Download
|
 |
M |
content/public/browser/browser_thread_delegate.h
|
View
|
|
1 chunk |
+3 lines, -0 lines |
0 comments
|
Download
|
 |
M |
content/public/test/test_browser_thread.h
|
View
|
|
1 chunk |
+0 lines, -4 lines |
0 comments
|
Download
|
 |
M |
content/public/test/test_browser_thread.cc
|
View
|
|
1 chunk |
+0 lines, -4 lines |
0 comments
|
Download
|
 |
M |
net/android/network_change_notifier_android.cc
|
View
|
|
1 chunk |
+0 lines, -3 lines |
0 comments
|
Download
|
 |
M |
net/test/embedded_test_server/embedded_test_server.cc
|
View
|
|
1 chunk |
+0 lines, -1 line |
0 comments
|
Download
|
Total messages: 4 (0 generated)
|