[cros] Use same textfield settings across existing/new user pods and screen lock.

chrome/browser/chromeos/login/textfield_with_margin.cc

 // Copyright (c) 2010 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/chromeos/login/textfield_with_margin.h"
 
+#include "chrome/browser/chromeos/login/helper.h"
+
 namespace {
 
 // Holds ratio of the margin to the preferred text height.
 const double kTextMarginRate = 0.33;
 
 // Size of each vertical margin (top, bottom).
 const int kVerticalMargin = 3;
 
 }  // namespace
 
 namespace chromeos {
 
 TextfieldWithMargin::TextfieldWithMargin() {
 }
 
 void TextfieldWithMargin::Layout() {
   int margin = GetPreferredSize().height() * kTextMarginRate;
   SetHorizontalMargins(margin, margin);
   SetVerticalMargins(kVerticalMargin, kVerticalMargin);
   views::Textfield::Layout();
 }
 
+void TextfieldWithMargin::ViewHierarchyChanged(bool is_add,
+                                               views::View *parent,
+                                               views::View *child) {
+  Textfield::ViewHierarchyChanged(is_add, parent, child);
+  if (is_add && child == this) {
+    CorrectTextfieldFontSize(this);

[altimofeev, 2010/12/01 18:21:36]

Why correction is done here? AFAIK textfield updates all the properties right
after native control has been created. So, could you please explain why font
changing doesn't work without this fix?

[whywhat, 2010/12/01 18:27:47]

I guess it worked but the idea is to move it to one place. And as
TextfieldWithMargin is used on ScreenLockView, font there will be fixed too.

[Nikita (slow), 2010/12/01 18:29:16]

Ok, I've found out that I was missing second constructor of 
TextfieldWithMargin.

+  }
+}
+
 }  // namespace chromeos