Remove true/false from RemoveAllChildViews() calls

This removes true, which is the default but disappearing argument from
the function call. For StatusAreaWidget::HandleLocaleChange() the call
is instead to RemoveAllChildViewsWithoutDeleting() which is a deprecated
function to better annotate existing non-removing cases.

This is part of removing this parameter which aligns with removal of
set_owned_by_client(). Follow-up will remove uses with non-constants.

Bug: 1044687
Change-Id: Ia33b2378ab6437e062a3a332ce2b40084cc335d7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3072410
Reviewed-by: Yuichiro Hanada <yhanada@chromium.org>
Reviewed-by: Ahmed Fakhry <afakhry@chromium.org>
Reviewed-by: Thomas Tellier <tellier@google.com>
Commit-Queue: Peter Boström <pbos@chromium.org>
Cr-Commit-Position: refs/heads/master@{#908885}

diff --git a/ash/system/phonehub/task_continuation_view.cc b/ash/system/phonehub/task_continuation_view.cc
index 6cd2b96..f0d1676 100644
--- a/ash/system/phonehub/task_continuation_view.cc
+++ b/ash/system/phonehub/task_continuation_view.cc

@@ -141,7 +141,7 @@
 
 void TaskContinuationView::TaskChipsView::Reset() {
   task_chips_.Clear();
-  RemoveAllChildViews(true /* delete_children */);
+  RemoveAllChildViews();
 }
 
 gfx::Point TaskContinuationView::TaskChipsView::GetButtonPosition(int index) {