[base] Replace Contains{Key,Value} with Contains in //ash

This change replaces usages of base::ContainsKey() and base::ContainsValue() with base::Contains() in //ash. Reproduction: - sed -i 's/\bbase::ContainsKey\b/base::Contains/g' - sed -i 's/\bbase::ContainsValue\b/base::Contains/g' - git cl format This CL was uploaded by git cl split. R=afakhry@chromium.org Bug: 970209 Change-Id: I41038819fd08becdaf1a4470e77f3438752de9ac Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1648182 Auto-Submit: Jan Wilken Dörrie <jdoerrie@chromium.org> Reviewed-by: Ahmed Fakhry <afakhry@chromium.org> Commit-Queue: Ahmed Fakhry <afakhry@chromium.org> Cr-Commit-Position: refs/heads/master@{#666797}
FairyWorld · Jun 6, 2019 · 58b0c4b · 58b0c4b
1 parent 9194856
commit 58b0c4b
Show file tree

Hide file tree

Showing 47 changed files with 82 additions and 87 deletions.
diff --git a/ash/accelerators/accelerator_controller_impl.cc b/ash/accelerators/accelerator_controller_impl.cc
@@ -1865,7 +1865,7 @@ AcceleratorControllerImpl::GetAcceleratorProcessingRestriction(
     return RESTRICTION_PREVENT_PROCESSING;
   }
   if (Shell::Get()->power_button_controller()->IsMenuOpened() &&
-      !base::ContainsKey(actions_allowed_at_power_menu_, action)) {
+      !base::Contains(actions_allowed_at_power_menu_, action)) {
     return RESTRICTION_PREVENT_PROCESSING;
   }
   if (Shell::Get()->session_controller()->IsRunningInAppMode() &&
@@ -1882,7 +1882,7 @@ AcceleratorControllerImpl::GetAcceleratorProcessingRestriction(
     // cycling through its window elements.
     return RESTRICTION_PREVENT_PROCESSING_AND_PROPAGATION;
   }
-  if (base::ContainsKey(actions_needing_window_, action) &&
+  if (base::Contains(actions_needing_window_, action) &&
       Shell::Get()
           ->mru_window_tracker()
           ->BuildMruWindowList(kActiveDesk)

diff --git a/ash/accelerators/pre_target_accelerator_handler.cc b/ash/accelerators/pre_target_accelerator_handler.cc
@@ -113,7 +113,7 @@ bool PreTargetAcceleratorHandler::ShouldProcessAcceleratorNow(
   if (accelerator.IsCmdDown())
     return true;
 
-  if (base::ContainsValue(Shell::GetAllRootWindows(), target))
+  if (base::Contains(Shell::GetAllRootWindows(), target))
     return true;
 
   AcceleratorControllerImpl* accelerator_controller =

diff --git a/ash/app_list/paged_view_structure.cc b/ash/app_list/paged_view_structure.cc
@@ -304,7 +304,7 @@ bool PagedViewStructure::IsFullPage(int page_index) const {
 
 int PagedViewStructure::CalculateTargetSlot(const Page& page) const {
   size_t target_slot = page.size();
-  if (base::ContainsValue(page, apps_grid_view_->drag_view()))
+  if (base::Contains(page, apps_grid_view_->drag_view()))
     --target_slot;
   return static_cast<int>(target_slot);
 }

diff --git a/ash/assistant/assistant_screen_context_controller.cc b/ash/assistant/assistant_screen_context_controller.cc
@@ -133,10 +133,10 @@ std::unique_ptr<ui::LayerTreeOwner> CreateLayerForAssistantSnapshot(
             // Parent layer is excluded meaning that it's pointless to clone
             // current child and all its descendants. This reduces the number
             // of layers to create.
-            if (base::ContainsKey(blocked_layers, owner->layer()->parent()))
+            if (base::Contains(blocked_layers, owner->layer()->parent()))
               return nullptr;
 
-            if (base::ContainsKey(blocked_layers, owner->layer())) {
+            if (base::Contains(blocked_layers, owner->layer())) {
               // Blocked layers are replaced with solid black layers so that
               // they won't be included in the screenshot but still preserve
               // the window stacking.

diff --git a/ash/assistant/model/assistant_alarm_timer_model.cc b/ash/assistant/model/assistant_alarm_timer_model.cc
@@ -24,7 +24,7 @@ void AssistantAlarmTimerModel::RemoveObserver(
 }
 
 void AssistantAlarmTimerModel::AddAlarmTimer(const AlarmTimer& alarm_timer) {
-  DCHECK(!base::ContainsKey(alarms_timers_, alarm_timer.id));
+  DCHECK(!base::Contains(alarms_timers_, alarm_timer.id));
   alarms_timers_[alarm_timer.id] = alarm_timer;
   NotifyAlarmTimerAdded(alarm_timer, /*time_remaining=*/base::TimeTicks::Now() -
                                          alarm_timer.end_time);

diff --git a/ash/assistant/model/assistant_notification_model.cc b/ash/assistant/model/assistant_notification_model.cc
@@ -91,7 +91,7 @@ AssistantNotificationModel::GetNotificationsByType(
 
 bool AssistantNotificationModel::HasNotificationForId(
     const std::string& id) const {
-  return base::ContainsKey(notifications_, id);
+  return base::Contains(notifications_, id);
 }
 
 void AssistantNotificationModel::NotifyNotificationAdded(

diff --git a/ash/assistant/util/deep_link_util.cc b/ash/assistant/util/deep_link_util.cc
@@ -364,7 +364,7 @@ bool IsWebDeepLinkType(DeepLinkType type,
       DeepLinkType::kLists, DeepLinkType::kNotes, DeepLinkType::kReminders,
       DeepLinkType::kSettings};
 
-  return base::ContainsKey(kWebDeepLinks, type);
+  return base::Contains(kWebDeepLinks, type);
 }
 
 }  // namespace util

diff --git a/ash/display/cros_display_config.cc b/ash/display/cros_display_config.cc
@@ -557,7 +557,7 @@ mojom::DisplayConfigResult SetDisplayLayouts(
     if (root_id == display::kInvalidDisplayId) {
       // Look for a display with no layout info to use as the root.
       for (int64_t id : display_ids) {
-        if (!base::ContainsKey(layout_ids, id)) {
+        if (!base::Contains(layout_ids, id)) {
           root_id = id;
           break;
         }

diff --git a/ash/display/cros_display_config_unittest.cc b/ash/display/cros_display_config_unittest.cc
@@ -315,8 +315,8 @@ TEST_F(CrosDisplayConfigTest, SetLayoutMirroredMixed) {
   display::DisplayIdList id_list =
       display_manager()->GetMirroringDestinationDisplayIdList();
   ASSERT_EQ(2u, id_list.size());
-  EXPECT_TRUE(base::ContainsValue(id_list, displays[1].id()));
-  EXPECT_TRUE(base::ContainsValue(id_list, displays[3].id()));
+  EXPECT_TRUE(base::Contains(id_list, displays[1].id()));
+  EXPECT_TRUE(base::Contains(id_list, displays[3].id()));
 }
 
 TEST_F(CrosDisplayConfigTest, GetDisplayUnitInfoListBasic) {

diff --git a/ash/display/display_color_manager.cc b/ash/display/display_color_manager.cc
@@ -232,8 +232,7 @@ bool DisplayColorManager::SetDisplayColorMatrix(
     const display::DisplaySnapshot* display_snapshot,
     const SkMatrix44& color_matrix) {
   DCHECK(display_snapshot);
-  DCHECK(
-      base::ContainsValue(configurator_->cached_displays(), display_snapshot));
+  DCHECK(base::Contains(configurator_->cached_displays(), display_snapshot));
 
   if (!display_snapshot->has_color_correction_matrix()) {
     // This display doesn't support setting a CRTC matrix.

diff --git a/ash/display/display_move_window_util.cc b/ash/display/display_move_window_util.cc
@@ -56,7 +56,7 @@ bool CanHandleMoveActiveWindowBetweenDisplays() {
     return false;
 
   // The movement target window must be in window cycle list.
-  return base::ContainsValue(
+  return base::Contains(
       Shell::Get()->mru_window_tracker()->BuildWindowForCycleList(kActiveDesk),
       GetTargetWindow());
 }

diff --git a/ash/display/display_prefs.cc b/ash/display/display_prefs.cc
@@ -352,7 +352,7 @@ void LoadDisplayTouchAssociations(PrefService* local_state) {
       calibration_data_to_set = &calibration_data;
 
     if (calibration_data_to_set) {
-      if (!base::ContainsKey(touch_associations, fallback_identifier)) {
+      if (!base::Contains(touch_associations, fallback_identifier)) {
         touch_associations.emplace(
             fallback_identifier,
             display::TouchDeviceManager::AssociationInfoMap());
@@ -514,7 +514,7 @@ void StoreCurrentDisplayProperties(PrefService* pref_service) {
   const display::TouchDeviceIdentifier& fallback_identifier =
       display::TouchDeviceIdentifier::GetFallbackTouchDeviceIdentifier();
   display::TouchDeviceManager::AssociationInfoMap legacy_data_map;
-  if (base::ContainsKey(
+  if (base::Contains(
           display_manager->touch_device_manager()->touch_associations(),
           fallback_identifier)) {
     legacy_data_map =
@@ -563,7 +563,7 @@ void StoreCurrentDisplayProperties(PrefService* pref_service) {
 
     // Store the legacy format touch calibration data. This can be removed after
     // a couple of milestones when every device has migrated to the new format.
-    if (legacy_data_map.size() && base::ContainsKey(legacy_data_map, id)) {
+    if (legacy_data_map.size() && base::Contains(legacy_data_map, id)) {
       TouchDataToValue(legacy_data_map.at(id).calibration_data,
                        property_value.get());
     }

diff --git a/ash/display/screen_orientation_controller.cc b/ash/display/screen_orientation_controller.cc
@@ -342,7 +342,7 @@ void ScreenOrientationController::OnWindowDestroying(aura::Window* window) {
 void ScreenOrientationController::OnWindowVisibilityChanged(
     aura::Window* window,
     bool visible) {
-  if (base::ContainsKey(lock_info_map_, window))
+  if (base::Contains(lock_info_map_, window))
     ApplyLockForActiveWindow();
 }
 

diff --git a/ash/display/touch_calibrator_controller_unittest.cc b/ash/display/touch_calibrator_controller_unittest.cc
@@ -196,7 +196,7 @@ TEST_F(TouchCalibratorControllerTest, StartCalibration) {
 
   ui::EventTargetTestApi test_api(Shell::Get());
   ui::EventHandlerList handlers = test_api.GetPreTargetHandlers();
-  EXPECT_TRUE(base::ContainsValue(handlers, &touch_calibrator_controller));
+  EXPECT_TRUE(base::Contains(handlers, &touch_calibrator_controller));
 }
 
 TEST_F(TouchCalibratorControllerTest, KeyEventIntercept) {

diff --git a/ash/home_screen/home_launcher_gesture_handler.cc b/ash/home_screen/home_launcher_gesture_handler.cc
@@ -472,7 +472,7 @@ void HomeLauncherGestureHandler::OnWindowDestroying(aura::Window* window) {
     return;
   }
 
-  DCHECK(base::ContainsValue(hidden_windows_, window));
+  DCHECK(base::Contains(hidden_windows_, window));
   window->RemoveObserver(this);
   hidden_windows_.erase(
       std::find(hidden_windows_.begin(), hidden_windows_.end(), window));
@@ -809,7 +809,7 @@ bool HomeLauncherGestureHandler::SetUpWindows(Mode mode, aura::Window* window) {
     return false;
   }
 
-  DCHECK(base::ContainsValue(windows, first_window));
+  DCHECK(base::Contains(windows, first_window));
   DCHECK_NE(Mode::kNone, mode);
   base::RecordAction(base::UserMetricsAction(
       mode == Mode::kSlideDownToHide
@@ -831,7 +831,7 @@ bool HomeLauncherGestureHandler::SetUpWindows(Mode mode, aura::Window* window) {
                 SplitViewController::LEFT
             ? split_view_controller->right_window()
             : split_view_controller->left_window();
-    DCHECK(base::ContainsValue(windows, second_window));
+    DCHECK(base::Contains(windows, second_window));
     secondary_window_ = std::make_unique<ScopedWindowModifier>(second_window);
     GetSecondaryWindow()->AddObserver(this);
     base::EraseIf(windows, [this](aura::Window* elem) {

diff --git a/ash/host/ash_window_tree_host_unified.cc b/ash/host/ash_window_tree_host_unified.cc
@@ -87,7 +87,7 @@ void AshWindowTreeHostUnified::RegisterMirroringHost(
   aura::Window* src_root = mirroring_ash_host->AsWindowTreeHost()->window();
   src_root->SetEventTargeter(
       std::make_unique<UnifiedEventTargeter>(src_root, window(), delegate_));
-  DCHECK(!base::ContainsValue(mirroring_hosts_, mirroring_ash_host));
+  DCHECK(!base::Contains(mirroring_hosts_, mirroring_ash_host));
   mirroring_hosts_.push_back(mirroring_ash_host);
   mirroring_ash_host->AsWindowTreeHost()->window()->AddObserver(this);
 }

diff --git a/ash/ime/ime_controller.cc b/ash/ime/ime_controller.cc
@@ -274,7 +274,7 @@ std::vector<std::string> ImeController::GetCandidateImesForAccelerator(
 
   // Obtain the intersection of input_method_ids_to_switch and available_imes_.
   for (const mojom::ImeInfo& ime : available_imes_) {
-    if (base::ContainsValue(input_method_ids_to_switch, ime.id))
+    if (base::Contains(input_method_ids_to_switch, ime.id))
       candidate_ids.push_back(ime.id);
   }
   return candidate_ids;

diff --git a/ash/keyboard/ash_keyboard_controller_unittest.cc b/ash/keyboard/ash_keyboard_controller_unittest.cc
@@ -318,27 +318,27 @@ TEST_F(AshKeyboardControllerTest, EnableFlags) {
   std::vector<keyboard::mojom::KeyboardEnableFlag> enable_flags =
       test_client()->GetEnableFlags();
   EXPECT_TRUE(
-      base::ContainsValue(enable_flags, KeyboardEnableFlag::kExtensionEnabled));
+      base::Contains(enable_flags, KeyboardEnableFlag::kExtensionEnabled));
   EXPECT_EQ(enable_flags, test_observer()->enable_flags());
   EXPECT_TRUE(test_client()->IsKeyboardEnabled());
 
   // Set the enable override to disable the keyboard.
   test_client()->SetEnableFlag(KeyboardEnableFlag::kPolicyDisabled);
   enable_flags = test_client()->GetEnableFlags();
   EXPECT_TRUE(
-      base::ContainsValue(enable_flags, KeyboardEnableFlag::kExtensionEnabled));
+      base::Contains(enable_flags, KeyboardEnableFlag::kExtensionEnabled));
   EXPECT_TRUE(
-      base::ContainsValue(enable_flags, KeyboardEnableFlag::kPolicyDisabled));
+      base::Contains(enable_flags, KeyboardEnableFlag::kPolicyDisabled));
   EXPECT_EQ(enable_flags, test_observer()->enable_flags());
   EXPECT_FALSE(test_client()->IsKeyboardEnabled());
 
   // Clear the enable override; should enable the keyboard.
   test_client()->ClearEnableFlag(KeyboardEnableFlag::kPolicyDisabled);
   enable_flags = test_client()->GetEnableFlags();
   EXPECT_TRUE(
-      base::ContainsValue(enable_flags, KeyboardEnableFlag::kExtensionEnabled));
+      base::Contains(enable_flags, KeyboardEnableFlag::kExtensionEnabled));
   EXPECT_FALSE(
-      base::ContainsValue(enable_flags, KeyboardEnableFlag::kPolicyDisabled));
+      base::Contains(enable_flags, KeyboardEnableFlag::kPolicyDisabled));
   EXPECT_EQ(enable_flags, test_observer()->enable_flags());
   EXPECT_TRUE(test_client()->IsKeyboardEnabled());
 }

diff --git a/ash/keyboard/ui/keyboard_controller.cc b/ash/keyboard/ui/keyboard_controller.cc
@@ -416,7 +416,7 @@ bool KeyboardController::UpdateKeyboardConfig(
 }
 
 void KeyboardController::SetEnableFlag(mojom::KeyboardEnableFlag flag) {
-  if (!base::ContainsKey(keyboard_enable_flags_, flag))
+  if (!base::Contains(keyboard_enable_flags_, flag))
     keyboard_enable_flags_.insert(flag);
 
   // If there is a flag that is mutually exclusive with |flag|, clear it.
@@ -455,7 +455,7 @@ void KeyboardController::ClearEnableFlag(mojom::KeyboardEnableFlag flag) {
 }
 
 bool KeyboardController::IsEnableFlagSet(mojom::KeyboardEnableFlag flag) const {
-  return base::ContainsKey(keyboard_enable_flags_, flag);
+  return base::Contains(keyboard_enable_flags_, flag);
 }
 
 bool KeyboardController::IsKeyboardEnableRequested() const {

diff --git a/ash/media/media_notification_controller_impl.cc b/ash/media/media_notification_controller_impl.cc
@@ -138,7 +138,7 @@ void MediaNotificationControllerImpl::OnFocusGained(
     media_session::mojom::AudioFocusRequestStatePtr session) {
   const std::string id = session->request_id->ToString();
 
-  if (base::ContainsKey(notifications_, id))
+  if (base::Contains(notifications_, id))
     return;
 
   media_session::mojom::MediaControllerPtr controller;

diff --git a/ash/media/media_notification_view.cc b/ash/media/media_notification_view.cc
@@ -359,9 +359,9 @@ void MediaNotificationView::UpdateActionButtonsVisibility() {
 
   for (auto* view : button_row_->children()) {
     views::Button* action_button = views::Button::AsButton(view);
-    action_button->SetVisible(base::ContainsKey(
-        visible_actions,
-        static_cast<MediaSessionAction>(action_button->tag())));
+    action_button->SetVisible(
+        base::Contains(visible_actions,
+                       static_cast<MediaSessionAction>(action_button->tag())));
   }
 }
 
@@ -452,7 +452,7 @@ std::set<MediaSessionAction> MediaNotificationView::CalculateVisibleActions(
       continue;
     }
 
-    if (!base::ContainsKey(enabled_actions_, action))
+    if (!base::Contains(enabled_actions_, action))
       continue;
 
     visible_actions.insert(action);

diff --git a/ash/policy/policy_recommendation_restorer.cc b/ash/policy/policy_recommendation_restorer.cc
@@ -36,7 +36,7 @@ void PolicyRecommendationRestorer::ObservePref(const std::string& pref_name) {
   PrefService* prefs =
       Shell::Get()->session_controller()->GetSigninScreenPrefService();
   DCHECK(prefs);
-  DCHECK(!base::ContainsKey(pref_names_, pref_name));
+  DCHECK(!base::Contains(pref_names_, pref_name));
 
   if (!pref_change_registrar_) {
     pref_change_registrar_ = std::make_unique<PrefChangeRegistrar>();

diff --git a/ash/public/cpp/shell_window_ids.cc b/ash/public/cpp/shell_window_ids.cc
@@ -65,7 +65,7 @@ std::vector<int> GetActivatableShellWindowIds() {
 }
 
 bool IsActivatableShellWindowId(int id) {
-  return base::ContainsValue(GetActivatableShellWindowIds(), id);
+  return base::Contains(GetActivatableShellWindowIds(), id);
 }
 
 }  // namespace ash
diff --git a/ash/shelf/shelf_layout_manager_unittest.cc b/ash/shelf/shelf_layout_manager_unittest.cc
@@ -2343,7 +2343,7 @@ TEST_F(ShelfLayoutManagerTest, AutohideShelfForAutohideWhenActiveWindow) {
   auto* shelf_window = shelf->GetWindow();
   aura::Window* container = shelf_window->GetRootWindow()->GetChildById(
       kShellWindowId_AlwaysOnTopContainer);
-  EXPECT_TRUE(base::ContainsValue(container->children(), window_two));
+  EXPECT_TRUE(base::Contains(container->children(), window_two));
 
   widget_two->Maximize();
   EXPECT_EQ(SHELF_AUTO_HIDE, shelf->GetVisibilityState());

diff --git a/ash/system/bluetooth/unified_bluetooth_detailed_view_controller.cc b/ash/system/bluetooth/unified_bluetooth_detailed_view_controller.cc
@@ -46,7 +46,7 @@ void RemoveObsoleteBluetoothDevicesFromList(
     const std::set<BluetoothAddress>& new_device_address_list) {
   base::EraseIf(*device_list, [&new_device_address_list](
                                   const BluetoothDeviceInfoPtr& info) {
-    return !base::ContainsKey(new_device_address_list, info->address);
+    return !base::Contains(new_device_address_list, info->address);
   });
 }
 

diff --git a/ash/system/ime_menu/ime_menu_tray_unittest.cc b/ash/system/ime_menu/ime_menu_tray_unittest.cc
@@ -96,7 +96,7 @@ class ImeMenuTrayTest : public AshTestBase {
     }
     for (const auto& ime : ime_map) {
       // Tests that all the IMEs on the view is in the list of selected IMEs.
-      EXPECT_TRUE(base::ContainsValue(expected_ime_ids, ime.second));
+      EXPECT_TRUE(base::Contains(expected_ime_ids, ime.second));
 
       // Tests that the checked IME is the current IME.
       ui::AXNodeData node_data;

diff --git a/ash/system/message_center/arc/arc_notification_manager.cc b/ash/system/message_center/arc/arc_notification_manager.cc
@@ -422,7 +422,7 @@ void ArcNotificationManager::OpenNotificationSettings(const std::string& key) {
 
 void ArcNotificationManager::OpenNotificationSnoozeSettings(
     const std::string& key) {
-  if (!base::ContainsKey(items_, key)) {
+  if (!base::Contains(items_, key)) {
     DVLOG(3) << "Chrome requests to show a snooze setting gut on the"
              << "notification (key: " << key << "), but it is gone.";
     return;

diff --git a/ash/system/network/tray_network_state_model.cc b/ash/system/network/tray_network_state_model.cc
@@ -134,7 +134,7 @@ void TrayNetworkStateModel::OnGetDeviceStateList(
   devices_.clear();
   for (auto& device : devices) {
     NetworkType type = device->type;
-    if (base::ContainsKey(devices_, type))
+    if (base::Contains(devices_, type))
       continue;  // Ignore multiple entries with the same type.
     devices_.emplace(std::make_pair(type, std::move(device)));
   }

diff --git a/ash/system/screen_layout_observer.cc b/ash/system/screen_layout_observer.cc
@@ -57,8 +57,8 @@ base::string16 GetDisplaySize(int64_t display_id) {
   // to empty string if this happens on release build.
   const display::DisplayIdList id_list =
       display_manager->GetMirroringDestinationDisplayIdList();
-  const bool mirroring = display_manager->IsInMirrorMode() &&
-                         base::ContainsValue(id_list, display_id);
+  const bool mirroring =
+      display_manager->IsInMirrorMode() && base::Contains(id_list, display_id);
   DCHECK(!mirroring);
   if (mirroring)
     return base::string16();

diff --git a/ash/wm/ash_focus_rules.cc b/ash/wm/ash_focus_rules.cc
@@ -64,12 +64,11 @@ bool AshFocusRules::IsToplevelWindow(const aura::Window* window) const {
 
   // The window must exist within a container that supports activation.
   // The window cannot be blocked by a modal transient.
-  return base::ContainsValue(activatable_container_ids_,
-                             window->parent()->id());
+  return base::Contains(activatable_container_ids_, window->parent()->id());
 }
 
 bool AshFocusRules::SupportsChildActivation(const aura::Window* window) const {
-  return base::ContainsValue(activatable_container_ids_, window->id());
+  return base::Contains(activatable_container_ids_, window->id());
 }
 
 bool AshFocusRules::IsWindowConsideredVisibleForActivation(

diff --git a/ash/wm/desks/desk_mini_view_animations.cc b/ash/wm/desks/desk_mini_view_animations.cc
@@ -109,7 +109,7 @@ void PerformNewDeskMiniViewAnimation(
   begin_transform.Translate(shift_x, 0);
 
   for (const auto& mini_view : bar_view->mini_views()) {
-    const bool is_new = base::ContainsValue(new_mini_views, mini_view.get());
+    const bool is_new = base::Contains(new_mini_views, mini_view.get());
 
     ui::Layer* layer = mini_view->layer();
     if (is_new)

diff --git a/ash/wm/mru_window_tracker.cc b/ash/wm/mru_window_tracker.cc
@@ -156,7 +156,7 @@ MruWindowTracker::WindowList BuildWindowListInternal(
 
         // Only add windows that have not been added previously from
         // |mru_windows| (if available).
-        if (mru_windows && base::ContainsValue(*mru_windows, child))
+        if (mru_windows && base::Contains(*mru_windows, child))
           continue;
 
         windows.emplace_back(child);