Fix autoupdate_EndToEndTest on veyron_rialto

ui/base/ime/chromeos/mock_component_extension_ime_manager.h

Index: ui/base/ime/chromeos/mock_component_extension_ime_manager.h
diff --git a/ui/base/ime/chromeos/mock_component_extension_ime_manager.h b/ui/base/ime/chromeos/mock_component_extension_ime_manager.h
new file mode 100644
index 0000000000000000000000000000000000000000..f027e40ffbf40f135740d808e8cf812ea82cb6c5
--- /dev/null
+++ b/ui/base/ime/chromeos/mock_component_extension_ime_manager.h
@@ -0,0 +1,22 @@
+// Copyright 2017 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.
+
+#ifndef UI_BASE_IME_MOCK_CHROMEOS_COMPONENT_EXTENSION_IME_MANAGER_H_
+#define UI_BASE_IME_MOCK_CHROMEOS_COMPONENT_EXTENSION_IME_MANAGER_H_
+
+#include "ui/base/ime/chromeos/component_extension_ime_manager.h"
+
+namespace chromeos {
+
+class UI_BASE_IME_EXPORT MockComponentExtensionIMEManager
+    : public ComponentExtensionIMEManager {
+ public:
+  input_method::InputMethodDescriptors GetXkbIMEAsInputMethodDescriptor() {

[xiyuan, 2017/04/27 19:34:22]

GetXkbIMEAsInputMethodDescriptor is not virtual so this might cause problems in
the future. If we want to mock it, we need to make it virtual in base class.

Since test in this CL does not really care about IME, why do we need to mock
this? Can we just call InputMethodManager::Initialize and Shutdown directly and
avoid this code?

[kumarniranjan, 2017/04/27 22:58:59]

There are two levels of mocking: one where I create MockInputMethodManager, and
a second level where I make this mock object use a
MockComponentExtensionIMEManager. I need both levels of mocking.

If I use InputMethodManager instead of MockInputMethodManager, the code runs
into a Segmentation fault in this function:
https://cs.chromium.org/chromium/src/ui/base/ime/chromeos/component_extension...

If I use MockInputMethodManager as I have done, but don't use a
MockComponentExtensionIMEManager, the code runs into a Segmentation fault in
this function:
https://cs.chromium.org/chromium/src/ui/base/ime/chromeos/component_extension...

So that is why I have to do both levels of mocking. I have changed
ComponentExtensionIMEManager::GetXkbIMEAsInputMethodDescriptor to be virtual.

+    return input_method::InputMethodDescriptors();
+  }
+};
+
+}  // namespace chromeos
+
+#endif  //  UI_BASE_IME_MOCK_CHROMEOS_COMPONENT_EXTENSION_IME_MANAGER_H_