Tweaks to make the toolkit-views HTTP Auth dialog more Maccy on MAc

chrome/browser/ui/views/login_view.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 "chrome/browser/ui/views/login_view.h"
 
 #include "chrome/grit/generated_resources.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/views/controls/label.h"
 #include "ui/views/controls/textfield/textfield.h"
 #include "ui/views/layout/grid_layout.h"
 #include "ui/views/layout/layout_constants.h"
 
+#if defined(OS_MACOSX)
+static const int kMessageWidth = 365;
+static const int kTextfieldStackHorizontalSpacing = 10;
+#else
 static const int kMessageWidth = 320;
 static const int kTextfieldStackHorizontalSpacing = 30;
+#endif
 
 using password_manager::LoginModel;
 using views::GridLayout;
 
 ///////////////////////////////////////////////////////////////////////////////
 // LoginView, public:
 
 LoginView::LoginView(const base::string16& explanation,
                      LoginModel* model)
     : username_field_(new views::Textfield()),
@@ skipping 40 matching lines @@
   layout->StartRow(0, labels_column_set_id);
   layout->AddView(username_label_);
   layout->AddView(username_field_);
 
   layout->AddPaddingRow(0, views::kRelatedControlVerticalSpacing);
 
   layout->StartRow(0, labels_column_set_id);
   layout->AddView(password_label_);
   layout->AddView(password_field_);
 
+#if !defined(OS_MACOSX)
+  // TODO(tapted): Is this ever needed? DialogClientView already adds
+  // kRelatedControlVerticalSpacing to account for padding between the password
+  // field and the buttons.
   layout->AddPaddingRow(0, views::kUnrelatedControlVerticalSpacing);
+#endif
 
   if (login_model_)
     login_model_->AddObserver(this);
 }
 
 LoginView::~LoginView() {
   if (login_model_)
     login_model_->RemoveObserver(this);
 }
 
@@ skipping 23 matching lines @@
 
 void LoginView::OnLoginModelDestroying() {
   login_model_->RemoveObserver(this);
   login_model_ = NULL;
 }
 
 const char* LoginView::GetClassName() const {
   return "LoginView";
 }