Variety of tweaks to View API.

views/controls/menu/submenu_view.cc

 // Copyright (c) 2011 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 "views/controls/menu/submenu_view.h"
 
 #include "ui/base/accessibility/accessible_view_state.h"
 #include "ui/gfx/canvas.h"
 #include "views/controls/menu/menu_config.h"
 #include "views/controls/menu/menu_controller.h"
@@ skipping 37 matching lines @@
   // The menu may not have been closed yet (it will be hidden, but not
   // necessarily closed).
   Close();
 
   delete scroll_view_container_;
 }
 
 int SubmenuView::GetMenuItemCount() {
   int count = 0;
   for (int i = 0; i < child_count(); ++i) {
-    if (GetChildViewAt(i)->GetID() == MenuItemView::kMenuItemViewID)
+    if (GetChildViewAt(i)->id() == MenuItemView::kMenuItemViewID)
       count++;
   }
   return count;
 }
 
 MenuItemView* SubmenuView::GetMenuItemAt(int index) {
   for (int i = 0, count = 0; i < child_count(); ++i) {
-    if (GetChildViewAt(i)->GetID() == MenuItemView::kMenuItemViewID &&
+    if (GetChildViewAt(i)->id() == MenuItemView::kMenuItemViewID &&
         count++ == index) {
       return static_cast<MenuItemView*>(GetChildViewAt(i));
     }
   }
   NOTREACHED();
   return NULL;
 }
 
 void SubmenuView::Layout() {
   // We're in a ScrollView, and need to set our width/height ourselves.
   if (!parent())
     return;
 
   // Use our current y, unless it means part of the menu isn't visible anymore.
   int pref_height = GetPreferredSize().height();
   int new_y;
   if (pref_height > parent()->height())
     new_y = std::max(parent()->height() - pref_height, y());
   else
     new_y = 0;
   SetBounds(x(), new_y, parent()->width(), pref_height);
 
   gfx::Insets insets = GetInsets();
   int x = insets.left();
   int y = insets.top();
   int menu_item_width = width() - insets.width();
   for (int i = 0; i < child_count(); ++i) {
     View* child = GetChildViewAt(i);
-    if (child->IsVisible()) {
+    if (child->visible()) {
       gfx::Size child_pref_size = child->GetPreferredSize();
       child->SetBounds(x, y, menu_item_width, child_pref_size.height());
       y += child_pref_size.height();
     }
   }
 }
 
 gfx::Size SubmenuView::GetPreferredSize() {
   if (!has_children())
     return gfx::Size();
 
   max_accelerator_width_ = 0;
   int max_width = 0;
   int height = 0;
   for (int i = 0; i < child_count(); ++i) {
     View* child = GetChildViewAt(i);
-    gfx::Size child_pref_size = child->IsVisible() ?
+    gfx::Size child_pref_size = child->visible() ?
         child->GetPreferredSize() : gfx::Size();
     max_width = std::max(max_width, child_pref_size.width());
     height += child_pref_size.height();
-    if (child->GetID() == MenuItemView::kMenuItemViewID) {
+    if (child->id() == MenuItemView::kMenuItemViewID) {
       MenuItemView* menu = static_cast<MenuItemView*>(child);
       max_accelerator_width_ =
           std::max(max_accelerator_width_, menu->GetAcceleratorTextWidth());
     }
   }
   if (max_accelerator_width_ > 0) {
     max_accelerator_width_ +=
         MenuConfig::instance().label_to_accelerator_padding;
   }
   gfx::Insets insets = GetInsets();
@@ skipping 248 matching lines @@
       item_bounds.set_height(kDropIndicatorHeight);
       return item_bounds;
 
     default:
       // Don't render anything for on.
       return gfx::Rect();
   }
 }
 
 }  // namespace views