Remove GridLayout::SetInsets in favor of an empty border on the host.

ui/message_center/views/message_center_view_unittest.cc

Index: ui/message_center/views/message_center_view_unittest.cc
diff --git a/ui/message_center/views/message_center_view_unittest.cc b/ui/message_center/views/message_center_view_unittest.cc
index 83101f61c2d69eb370d45f813eab73b56a44b92e..9edd0d80e7f93d2d1609f636ac909c06f2135660 100644
--- a/ui/message_center/views/message_center_view_unittest.cc
+++ b/ui/message_center/views/message_center_view_unittest.cc
@@ -578,7 +578,7 @@ TEST_F(MessageCenterViewTest, SizeAfterRemove) {
 
   EXPECT_FALSE(GetNotificationView(kNotificationId1));
   EXPECT_TRUE(GetNotificationView(kNotificationId2));
-  EXPECT_EQ(GetMessageListView()->height(), original_height);
+  EXPECT_EQ(original_height, GetMessageListView()->height());
 }
 
 TEST_F(MessageCenterViewTest, PositionAfterUpdate) {
@@ -851,7 +851,7 @@ TEST_F(MessageCenterViewTest,
 }
 
 TEST_F(MessageCenterViewTest, LockScreen) {
-  const int kLockedMessageCenterViewHeight = 50;
+  const int kLockedMessageCenterViewHeight = 51;

[sky, 2017/05/07 22:26:14]

How come you had to change this value (and the value on line 931 as well).

[Bret, 2017/05/09 22:03:09]

There's a 1-dip-high border added in MessageCenterView::Layout that was
previously being ignored by GridLayout. So now that it's being accounted for,
the height has increased by 1 dip.

 
   EXPECT_TRUE(GetNotificationView(kNotificationId1)->IsDrawn());
   EXPECT_TRUE(GetNotificationView(kNotificationId2)->IsDrawn());
@@ -928,7 +928,7 @@ TEST_F(MessageCenterViewTest, LockScreen) {
 }
 
 TEST_F(MessageCenterViewTest, NoNotification) {
-  const int kEmptyMessageCenterViewHeight = 50;
+  const int kEmptyMessageCenterViewHeight = 51;
 
   GetMessageCenterView()->SizeToPreferredSize();
   EXPECT_NE(kEmptyMessageCenterViewHeight, GetMessageCenterView()->height());