Fix stacked notification icon size and padding

Change-Id: Id28394f3d11e190c58b05e37dbdfbef34d95a7d4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1862857
Commit-Queue: Ahmed Mehfooz <amehfooz@chromium.org>
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#706225}

ash/system/message_center/stacked_notification_bar.cc

@@ -128,9 +128,7 @@ StackedNotificationBar::StackedNotificationBar(
   SetVisible(false);
 
   auto* layout = SetLayoutManager(std::make_unique<views::BoxLayout>(
-      views::BoxLayout::Orientation::kHorizontal,
-      gfx::Insets(0, kStackingNotificationClearAllButtonPadding.left(), 0, 0),
-      0));
+      views::BoxLayout::Orientation::kHorizontal));
   layout->set_cross_axis_alignment(
       views::BoxLayout::CrossAxisAlignment::kStretch);
 
@@ -139,7 +137,8 @@ StackedNotificationBar::StackedNotificationBar(
     notification_icons_container_->SetLayoutManager(
         std::make_unique<views::BoxLayout>(
             views::BoxLayout::Orientation::kHorizontal,
-            kStackedNotificationIconsContainerPadding));
+            kStackedNotificationIconsContainerPadding,
+            kStackedNotificationBarIconSpacing));
     AddChildView(notification_icons_container_);
   }
 

ash/system/tray/tray_constants.h

@@ -130,9 +130,10 @@ constexpr gfx::Insets kUnifiedSystemInfoDateViewPadding(3);
 
 // Constants used in StackedNotificationBar located on top of the message
 // center.
-constexpr gfx::Insets kStackedNotificationIconsContainerPadding(2, 0, 0, 6);
-constexpr int kStackedNotificationBarMaxIcons = 5;
-constexpr int kStackedNotificationIconSize = 15;
+constexpr gfx::Insets kStackedNotificationIconsContainerPadding(1, 16, 0, 8);
+constexpr int kStackedNotificationBarMaxIcons = 3;
+constexpr int kStackedNotificationBarIconSpacing = 6;
+constexpr int kStackedNotificationIconSize = 18;
 
 // Constants used in FeaturePodsView of UnifiedSystemTray.
 constexpr gfx::Size kUnifiedFeaturePodIconSize(48, 48);