Update shelf spacing in Chrome OS according to the MD specs

ash/system/tray/tray_constants.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ash/system/tray/tray_constants.h"
 
 #include "ash/ash_switches.h"
+#include "ash/material_design/material_design_controller.h"
+#include "base/logging.h"
 #include "third_party/skia/include/core/SkColor.h"
 
 namespace ash {
 
 const int kPaddingFromRightEdgeOfScreenBottomAlignment = 7;
 const int kPaddingFromBottomOfScreenBottomAlignment = 7;
 const int kPaddingFromOuterEdgeOfLauncherVerticalAlignment = 8;
 const int kPaddingFromInnerEdgeOfLauncherVerticalAlignment = 9;
 const int kPaddingFromBottomOfScreenVerticalAlignment = 10;
 
 // Padding used to position the system menu relative to the status area.
 const int kBubblePaddingHorizontalBottom = 6;
 const int kBubblePaddingHorizontalSide = 10;
 const int kBubblePaddingVerticalBottom = 3;
 const int kBubblePaddingVerticalSide = 15;
 
-const int kPaddingFromEdgeOfShelf = 3;
+// Padding used on empty border to Adjust the size of status tray dark

[tdanderson, 2016/05/26 22:55:56]

nit: Adjust -> adjust

[yiyix, 2016/06/02 03:54:54]

Done.

+// background.

[tdanderson, 2016/05/26 22:55:56]

Also, I am not a fan of how the comment is worded. Let's discuss in person.

[yiyix, 2016/06/02 03:54:53]

Done. I update the name as we have discussed in person.

+const int kEmptyBoarderPadding = 3;
 
 // Top inset of system tray bubble for bottom anchor alignment.
 const int kTrayBubbleAnchorTopInsetBottomAnchor = 3;
 
 const int kTrayImageItemHorizontalPaddingBottomAlignment = 1;
 const int kTrayImageItemHorizontalPaddingVerticalAlignment = 1;
 const int kTrayImageItemVerticalPaddingVerticalAlignment = 1;
 
 // Size of tray items on the primary axis.
 const int kTrayItemSize = 32;
@@ skipping 39 matching lines @@
 const int kTrayPopupMaxWidth = 500;
 const int kNotificationIconWidth = 40;
 const int kNotificationButtonWidth = 32;
 const int kTrayNotificationContentsWidth = kTrayPopupMinWidth -
     (kNotificationIconWidth + kNotificationButtonWidth +
      (kTrayPopupPaddingHorizontal / 2) * 3);
 
 const int kTrayAvatarCornerRadius = 2;
 const int kTrayAvatarSize = 32;
 
-const int kTraySpacing = 4;
+const int kMessageCenterBubblePadding = 4;
 const int kShelfItemHeight = 38;
 
+// static
+int GetShelfStatusAreaInset(ShelfStatusAreaInset constant) {

[tdanderson, 2016/05/26 22:55:56]

I would just call this GetTrayConstant(). Similarly rename ShelfStatusAreaInset.

[yiyix, 2016/06/02 03:54:54]

Done.

+  const int kShelfStatusAreaInset[] = {0, 5};
+  const int kTraySpacing[] = {4, 2};
+  const int kPaddingFromEdgeOfShelf[] = {3, 0};
+
+  int mode = MaterialDesignController::Mode::NON_MATERIAL;
+  if (MaterialDesignController::IsShelfMaterial())
+    mode = MaterialDesignController::Mode::MATERIAL_NORMAL;
+
+  switch (constant) {
+    case ShelfStatusAreaInset::PADDING_FROM_EDGE_OF_SHELF:
+      return kShelfStatusAreaInset[mode];
+    case TRAY_SPACING:
+      return kTraySpacing[mode];
+    case PADDING_FROM_EDGE_TO_OVERVIEW_BUTTON:

[tdanderson, 2016/05/26 22:55:56]

nit: be consistent with the naming of the constants and the naming of the enum
items. (here and at line 102)

[yiyix, 2016/06/02 03:54:53]

Done.

+      return kPaddingFromEdgeOfShelf[mode];
+  }
+  NOTREACHED();
+  return 0;
+}
+
 }  // namespace ash