Pass a correct button name to OnCandidateClicked()

- Found by GCC 4.6: chrome/browser/chromeos/input_method/input_method_engine.cc:508:7: warning: variable 'pressed_button' set but not used [-Wunused-but-set-variable]
- To prevent this kind of error, added explicit types to enums in the class.

BUG=None
TEST=try


Review URL: http://codereview.chromium.org/8498003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108995 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 6 insertions, 4 deletions

diff --git a/chrome/browser/extensions/extension_input_ime_api.cc b/chrome/browser/extensions/extension_input_ime_api.cc
index 098a894..c54f5b9c 100644
--- a/chrome/browser/extensions/extension_input_ime_api.cc
+++ b/chrome/browser/extensions/extension_input_ime_api.cc
@@ -37,7 +37,8 @@ bool ReadMenuItems(
 
     std::string id;
     std::string label;
-    int style = chromeos::InputMethodEngine::MENU_ITEM_STYLE_NONE;
+    chromeos::InputMethodEngine::MenuItemStyle style =
+        chromeos::InputMethodEngine::MENU_ITEM_STYLE_NONE;
     bool visible = true;
     bool enabled = true;
     bool checked = false;
@@ -295,9 +296,10 @@ class ImeObserver : public chromeos::InputMethodEngine::Observer {
         extension_id_, events::kOnKeyEvent, json_args, profile_, GURL());
   }
 
-  virtual void OnCandidateClicked(const std::string& engine_id,
-                                  int candidate_id,
-                                  int button) {
+  virtual void OnCandidateClicked(
+      const std::string& engine_id,
+      int candidate_id,
+      chromeos::InputMethodEngine::MouseButtonEvent button) {
     if (profile_ == NULL || extension_id_.empty())
       return;