Rework tree APIs to reflect Google style and more const-correctness....

views/controls/textfield/native_textfield_win.cc

Index: views/controls/textfield/native_textfield_win.cc
===================================================================
--- views/controls/textfield/native_textfield_win.cc	(revision 74117)
+++ views/controls/textfield/native_textfield_win.cc	(working copy)
@@ -436,8 +436,8 @@
       CHILDID_SELF, PROPID_ACC_ROLE, var);
 
   // Set the accessible name by getting the label text.
-  View* parent = textfield_->GetParent();
-  int label_index = parent->GetChildIndex(textfield_) - 1;
+  View* parent = textfield_->parent();
+  int label_index = parent->GetIndexOf(textfield_) - 1;

[sky, 2011/02/08 19:11:54]

Are you sure this is right if parent doesn't contain textfield_? If the code
ends up equivalent as:

static_cast<int>(parent->GetIndexOf(textfield_) - 1), then it can never be
negative.

PS I don't like size_t

   if (label_index >= 0) {
     // Try to find the name of this text field.
     // We expect it to be a Label preceeding this view (if it exists).