BoxLayout now suports per-view margins.

ui/views/layout/box_layout_unittest.cc

Index: ui/views/layout/box_layout_unittest.cc
diff --git a/ui/views/layout/box_layout_unittest.cc b/ui/views/layout/box_layout_unittest.cc
index d2d073e5cb78d3cbae27b80b204e13adab569fce..3995189ab254a0e3b43955518dab6332b9c91984 100644
--- a/ui/views/layout/box_layout_unittest.cc
+++ b/ui/views/layout/box_layout_unittest.cc
@@ -629,4 +629,72 @@ TEST_F(BoxLayoutTest, MinimumCrossAxisHorizontal) {
   EXPECT_EQ(gfx::Size(20, 30), layout->GetPreferredSize(host_.get()));
 }
 
+TEST_F(BoxLayoutTest, MarginsUncollapsedHorizontal) {
+  BoxLayout* layout = new BoxLayout(BoxLayout::kHorizontal, 0, 0, 0, false);
+  host_->SetLayoutManager(layout);
+  View* v1 = new StaticSizedView(gfx::Size(20, 10));
+  v1->SetProperty(kMarginsKey, new gfx::Insets(5, 5, 5, 5));
+  host_->AddChildView(v1);
+  View* v2 = new StaticSizedView(gfx::Size(20, 10));
+  v2->SetProperty(kMarginsKey, new gfx::Insets(6, 4, 6, 4));
+  host_->AddChildView(v2);
+
+  EXPECT_EQ(gfx::Size(58, 22), layout->GetPreferredSize(host_.get()));
+  host_->SizeToPreferredSize();
+  layout->Layout(host_.get());
+  EXPECT_EQ(gfx::Rect(5, 5, 20, 12), v1->bounds());
+  EXPECT_EQ(gfx::Rect(34, 6, 20, 10), v2->bounds());
+}
+
+TEST_F(BoxLayoutTest, MarginsCollapsedHorizontal) {
+  BoxLayout* layout = new BoxLayout(BoxLayout::kHorizontal, 0, 0, 0, true);
+  host_->SetLayoutManager(layout);
+  View* v1 = new StaticSizedView(gfx::Size(20, 10));
+  v1->SetProperty(kMarginsKey, new gfx::Insets(5, 5, 5, 5));
+  host_->AddChildView(v1);
+  View* v2 = new StaticSizedView(gfx::Size(20, 10));
+  v2->SetProperty(kMarginsKey, new gfx::Insets(6, 4, 6, 4));
+  host_->AddChildView(v2);
+
+  EXPECT_EQ(gfx::Size(54, 22), layout->GetPreferredSize(host_.get()));
+  host_->SizeToPreferredSize();
+  layout->Layout(host_.get());
+  EXPECT_EQ(gfx::Rect(5, 6, 20, 10), v1->bounds());
+  EXPECT_EQ(gfx::Rect(30, 6, 20, 10), v2->bounds());
+}
+
+TEST_F(BoxLayoutTest, MarginsUncollapsedVertical) {
+  BoxLayout* layout = new BoxLayout(BoxLayout::kVertical, 0, 0, 0, false);
+  host_->SetLayoutManager(layout);
+  View* v1 = new StaticSizedView(gfx::Size(20, 10));
+  v1->SetProperty(kMarginsKey, new gfx::Insets(5, 5, 5, 5));
+  host_->AddChildView(v1);
+  View* v2 = new StaticSizedView(gfx::Size(20, 10));
+  v2->SetProperty(kMarginsKey, new gfx::Insets(6, 4, 6, 4));
+  host_->AddChildView(v2);
+
+  EXPECT_EQ(gfx::Size(30, 42), layout->GetPreferredSize(host_.get()));
+  host_->SizeToPreferredSize();
+  layout->Layout(host_.get());
+  EXPECT_EQ(gfx::Rect(5, 5, 20, 10), v1->bounds());
+  EXPECT_EQ(gfx::Rect(4, 26, 22, 10), v2->bounds());
+}
+
+TEST_F(BoxLayoutTest, MarginsCollapsedVertical) {
+  BoxLayout* layout = new BoxLayout(BoxLayout::kVertical, 0, 0, 0, true);
+  host_->SetLayoutManager(layout);
+  View* v1 = new StaticSizedView(gfx::Size(20, 10));
+  v1->SetProperty(kMarginsKey, new gfx::Insets(5, 5, 5, 5));
+  host_->AddChildView(v1);
+  View* v2 = new StaticSizedView(gfx::Size(20, 10));
+  v2->SetProperty(kMarginsKey, new gfx::Insets(6, 4, 6, 4));
+  host_->AddChildView(v2);
+
+  EXPECT_EQ(gfx::Size(30, 37), layout->GetPreferredSize(host_.get()));
+  host_->SizeToPreferredSize();
+  layout->Layout(host_.get());
+  EXPECT_EQ(gfx::Rect(5, 5, 20, 10), v1->bounds());
+  EXPECT_EQ(gfx::Rect(5, 21, 20, 10), v2->bounds());
+}
+
 }  // namespace views