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

Rework a loop in the native menu code to iterate over the vector of items we have, rather than the actual system menu.

I believe this may fix a crash where the code indexes out of bounds in the vector. The trouble is the assumption that the native menu index matches the index into the vector. If someone has installed a windows addon this may not be true. It's safer to restrict the iteration to the bounds of the vector and index into the menu based on that.

http://crbug.com/14600
TEST=none


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=19558

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

OK

http://codereview.chromium.org/150061/diff/1/2
File views/controls/menu/native_menu_win.cc (right):

http://codereview.chromium.org/150061/diff/1/2#newcode357
Line 357: model_->GetLabelAt(model_index));
nit: align with (