linux_aura: Fix the insets on LabelButtons.

chrome/browser/ui/views/toolbar/toolbar_button.cc

Index: chrome/browser/ui/views/toolbar/toolbar_button.cc
diff --git a/chrome/browser/ui/views/toolbar/toolbar_button.cc b/chrome/browser/ui/views/toolbar/toolbar_button.cc
index 7a285d39e02a23c3b1990408d2886dfa953b8195..98d484c9efc37edf04f7436f48b178f3a3e6dda9 100644
--- a/chrome/browser/ui/views/toolbar/toolbar_button.cc
+++ b/chrome/browser/ui/views/toolbar/toolbar_button.cc
@@ -11,6 +11,7 @@
 #include "ui/accessibility/ax_view_state.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/base/models/menu_model.h"
+#include "ui/base/theme_provider.h"
 #include "ui/gfx/display.h"
 #include "ui/gfx/screen.h"
 #include "ui/views/controls/button/label_button_border.h"
@@ -128,6 +129,13 @@ void ToolbarButton::GetAccessibleState(ui::AXViewState* state) {
   state->AddStateFlag(ui::AX_STATE_HASPOPUP);
 }
 
+void ToolbarButton::SetDefaultBorderInsets(views::LabelButtonBorder* border) {
+  ui::ThemeProvider* provider = GetThemeProvider();
+  if (provider && provider->UsingSystemTheme()) {

[msw, 2014/05/21 22:57:14]

nit: remove curly braces.

+    border->set_insets(gfx::Insets(2, 2, 2, 2));

[msw, 2014/05/21 22:57:14]

Can you document where these inset values come from?
Are GTK toolbar buttons different than all other GTK buttons?

+  }
+}
+
 void ToolbarButton::ShowContextMenuForView(View* source,
                                            const gfx::Point& point,
                                            ui::MenuSourceType source_type) {