Remove remaining uses of ScopedNestableTaskAllower.
ScopedNestableTaskAllower allows application tasks in native loops and
RunLoops that run within its scope. It is deprecated.
If the intent is to allow application tasks in a native loop, it
should be replaced with ScopedAllowApplicationTasksInNativeNestedLoop.
If the intent is to allow application tasks in a RunLoop, the RunLoop
should be constructed with Type::kNestableTasksAllowed.
If no loop (native or RunLoop) runs within the scope of the
ScopedNestableTaskAllower, it should just be removed.
We manually inspected the code to determine the appropriate replacement
for each remaining use.
Bug: 781352
Change-Id: I007bd0a0f9d6aa938f879f48864a9400808d486b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4035451
Reviewed-by: Gabriel Charette <[email protected]>
Owners-Override: Gabriel Charette <[email protected]>
Commit-Queue: Francois Pierre Doray <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1075631}
diff --git a/base/message_loop/message_pump_glib_unittest.cc b/base/message_loop/message_pump_glib_unittest.cc
index 16d5f87..4f01fa9 100644
--- a/base/message_loop/message_pump_glib_unittest.cc
+++ b/base/message_loop/message_pump_glib_unittest.cc
@@ -473,7 +473,7 @@
// Run a nested, straight GLib message loop.
{
- CurrentThread::ScopedNestableTaskAllower allow_nestable_tasks;
+ CurrentThread::ScopedAllowApplicationTasksInNativeNestedLoop allow;
runner->RunGLib();
}
@@ -505,7 +505,7 @@
// Run a nested, straight Gtk message loop.
{
- CurrentThread::ScopedNestableTaskAllower allow_nestable_tasks;
+ CurrentThread::ScopedAllowApplicationTasksInNativeNestedLoop allow;
runner->RunLoop();
}
