Remove unneeded separator when App badging is not enabled

Bug: 1136836
Change-Id: I36c98b23b714f36a09c7bce702bf9ef3c5347eba
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2464001Reviewed-by: Tim Song <tengs@chromium.org>
Commit-Queue: Ahmed Mehfooz <amehfooz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#815880}

ash/system/message_center/notifier_settings_view.cc

@@ -482,12 +482,12 @@ NotifierSettingsView::NotifierSettingsView() {
     app_badging_view->SetBorder(
         views::CreateSolidSidedBorder(0, 0, 0, 1, kTopBorderColor));
     header_view->AddChildView(std::move(app_badging_view));
-  }
 
-  // Separator between toggle button rows.
-  auto separator = std::make_unique<views::Separator>();
-  separator->SetColor(separator_color);
-  header_view->AddChildView(std::move(separator));
+    // Separator between toggle button rows.
+    auto separator = std::make_unique<views::Separator>();
+    separator->SetColor(separator_color);
+    header_view->AddChildView(std::move(separator));
+  }
 
   // Row for the quiet mode toggle button.
   auto quiet_mode_icon = std::make_unique<views::ImageView>();