Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(904)

Unified Diff: ui/views/controls/label_unittest.cc

Issue 2480813003: Reduce views::Border creation verbosity by promoting factory functions (Closed)
Patch Set: fix bad merge Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ui/views/controls/label_unittest.cc
diff --git a/ui/views/controls/label_unittest.cc b/ui/views/controls/label_unittest.cc
index 05265584bb466d860e917e9f0998a61400780222..a7de6eb5acce332cd3356095806a1c0f88d50c52 100644
--- a/ui/views/controls/label_unittest.cc
+++ b/ui/views/controls/label_unittest.cc
@@ -384,8 +384,8 @@ TEST_F(LabelTest, SingleLineSizing) {
EXPECT_EQ(size, label.GetPreferredSize());
const gfx::Insets border(10, 20, 30, 40);
- label.SetBorder(Border::CreateEmptyBorder(
- border.top(), border.left(), border.bottom(), border.right()));
+ label.SetBorder(CreateEmptyBorder(border.top(), border.left(),
+ border.bottom(), border.right()));
const gfx::Size size_with_border = label.GetPreferredSize();
EXPECT_EQ(size_with_border.height(), size.height() + border.height());
EXPECT_EQ(size_with_border.width(), size.width() + border.width());
@@ -465,8 +465,8 @@ TEST_F(LabelTest, MultiLineSizing) {
// Test everything with borders.
gfx::Insets border(10, 20, 30, 40);
- label.SetBorder(Border::CreateEmptyBorder(
- border.top(), border.left(), border.bottom(), border.right()));
+ label.SetBorder(CreateEmptyBorder(border.top(), border.left(),
+ border.bottom(), border.right()));
// SizeToFit and borders.
label.SizeToFit(0);
« no previous file with comments | « ui/views/controls/button/vector_icon_button.cc ('k') | ui/views/controls/menu/menu_scroll_view_container.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698