Fix infinite loop causing iterator to go out of bounds.

ash/virtual_keyboard_controller.cc

 // Copyright 2014 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 "ash/virtual_keyboard_controller.h"
 
 #include <vector>
 
 #include "ash/shell.h"
 #include "ash/wm/maximize_mode/maximize_mode_controller.h"
@@ skipping 54 matching lines @@
 void VirtualKeyboardController::UpdateDevices() {
   ui::DeviceDataManager* device_data_manager =
       ui::DeviceDataManager::GetInstance();
 
   // Checks for touchscreens.
   has_touchscreen_ = device_data_manager->touchscreen_devices().size() > 0;
 
   // Checks for keyboards.
   has_external_keyboard_ = false;
   has_internal_keyboard_ = false;
-  std::vector<ui::KeyboardDevice> keyboards =
-      device_data_manager->keyboard_devices();
-  for (auto iter = keyboards.begin();
-       iter != keyboards.end() ||
-           (has_internal_keyboard_ && has_external_keyboard_);
-       ++iter) {
-    ui::InputDeviceType type = (*iter).type;
+  for (const ui::KeyboardDevice& device :
+       device_data_manager->keyboard_devices()) {
+    if (has_internal_keyboard_ && has_external_keyboard_)
+      break;
+    ui::InputDeviceType type = device.type;
     if (type == ui::InputDeviceType::INPUT_DEVICE_INTERNAL)
       has_internal_keyboard_ = true;
     if (type == ui::InputDeviceType::INPUT_DEVICE_EXTERNAL)
       has_external_keyboard_ = true;
   }
   // Update keyboard state.
   UpdateKeyboardEnabled();
 }
 
 void VirtualKeyboardController::UpdateKeyboardEnabled() {
@@ skipping 17 matching lines @@
   keyboard::SetTouchKeyboardEnabled(enabled);
   if (enabled) {
     Shell::GetInstance()->CreateKeyboard();
   } else {
     if (!keyboard::IsKeyboardEnabled())
       Shell::GetInstance()->DeactivateKeyboard();
   }
 }
 
 }  // namespace ash