Rework a loop in the native menu code to iterate over the vector of items we ...

views/controls/menu/native_menu_win.cc

Index: views/controls/menu/native_menu_win.cc
===================================================================
--- views/controls/menu/native_menu_win.cc	(revision 19515)
+++ views/controls/menu/native_menu_win.cc	(working copy)
@@ -346,16 +346,17 @@
 
 void NativeMenuWin::UpdateStates() {
   // A depth-first walk of the menu items, updating states.
-  for (int menu_index = first_item_index_;
-       menu_index < first_item_index_ + model_->GetItemCount(); ++menu_index) {
-    int model_index = menu_index - first_item_index_;
+  int model_index = 0;
+  std::vector<ItemData*>::const_iterator it;
+  for (it = items_.begin(); it != items_.end(); ++it, ++model_index) {
+    int menu_index = model_index + first_item_index_;
     SetMenuItemState(menu_index, model_->IsEnabledAt(model_index),
                      model_->IsItemCheckedAt(model_index), false);
     if (model_->IsLabelDynamicAt(model_index)) {
       SetMenuItemLabel(menu_index, model_index,
-                       model_->GetLabelAt(model_index));
+        model_->GetLabelAt(model_index));

[sky, 2009/06/29 21:40:06]

nit: align with (

     }
-    Menu2* submenu = items_[model_index]->submenu.get();
+    Menu2* submenu = (*it)->submenu.get();
     if (submenu)
       submenu->UpdateStates();
   }