Get rid of unused callbacks from ui::InputMethodObserver

ui/base/ime/input_method_base_unittest.cc

 // Copyright 2013 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 "ui/base/ime/input_method_base.h"
 
 #include "base/scoped_observer.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "ui/base/ime/dummy_text_input_client.h"
 #include "ui/base/ime/input_method_observer.h"
@@ skipping 111 matching lines @@
   virtual bool OnUntranslatedIMEMessage(
       const base::NativeEvent& event,
       InputMethod::NativeEventResult* result) OVERRIDE {
     return false;
   }
   virtual bool DispatchKeyEvent(const ui::KeyEvent&) OVERRIDE {
     return false;
   }
   virtual void CancelComposition(const TextInputClient* client) OVERRIDE {
   }
+  virtual void OnInputLocaleChanged() OVERRIDE {
+  }
   virtual std::string GetInputLocale() OVERRIDE{
     return "";
   }
   virtual base::i18n::TextDirection GetInputTextDirection() OVERRIDE {
     return base::i18n::UNKNOWN_DIRECTION;
   }
   virtual bool IsActive() OVERRIDE {
     return false;
   }
   virtual bool IsCandidatePopupOpen() const OVERRIDE {
@@ skipping 35 matching lines @@
   }
   virtual ~SimpleMockInputMethodObserver() {
   }
   void Reset() {
     on_caret_bounds_changed_ = 0;
     on_input_locale_changed_ = 0;
   }
   size_t on_caret_bounds_changed() const {
     return on_caret_bounds_changed_;
   }
-  size_t on_input_locale_changed() const {
-    return on_input_locale_changed_;
-  }
 
  private:
   // Overriden from InputMethodObserver.
   virtual void OnTextInputTypeChanged(const TextInputClient* client) OVERRIDE{
   }
   virtual void OnFocus() OVERRIDE{
   }
   virtual void OnBlur() OVERRIDE{
   }
-  virtual void OnUntranslatedIMEMessage(
-    const base::NativeEvent& event) OVERRIDE{
-  }
   virtual void OnCaretBoundsChanged(const TextInputClient* client) OVERRIDE{
     ++on_caret_bounds_changed_;
   }
-  virtual void OnInputLocaleChanged() OVERRIDE{
-    ++on_input_locale_changed_;
-  }
   virtual void OnTextInputStateChanged(const TextInputClient* client) OVERRIDE{
   }
   virtual void OnInputMethodDestroyed(const InputMethod* client) OVERRIDE{
   }
 
   size_t on_caret_bounds_changed_;
   size_t on_input_locale_changed_;
   DISALLOW_COPY_AND_ASSIGN(SimpleMockInputMethodObserver);
 };
 
@@ skipping 162 matching lines @@
     input_method.OnCaretBoundsChanged(NULL);
     EXPECT_EQ(0u, input_method_observer.on_caret_bounds_changed());
 
     // Must not fire the event
     input_method_observer.Reset();
     input_method.OnCaretBoundsChanged(&text_input_client_the_other);
     EXPECT_EQ(0u, input_method_observer.on_caret_bounds_changed());
   }
 }
 
-TEST(InputMethodBaseTest, OnInputLocaleChanged) {
-  DummyTextInputClient text_input_client;
-
-  SimpleMockInputMethodBase input_method;
-  SimpleMockInputMethodObserver input_method_observer;
-  InputMethodScopedObserver scoped_observer(&input_method_observer);
-  scoped_observer.Add(&input_method);
-
-  // Assume that the top-level-widget gains focus.
-  input_method.OnFocus();
-
-  {
-    SCOPED_TRACE("OnInputLocaleChanged callback can be fired even when no text "
-        "input client is focused");
-    ASSERT_EQ(NULL, input_method.GetTextInputClient());
-
-    input_method_observer.Reset();
-    input_method.OnInputLocaleChanged();
-    EXPECT_EQ(1u, input_method_observer.on_input_locale_changed());
-
-    input_method.SetFocusedTextInputClient(&text_input_client);
-    input_method_observer.Reset();
-    input_method.OnInputLocaleChanged();
-    EXPECT_EQ(1u, input_method_observer.on_input_locale_changed());
-  }
-}
-
 }  // namespace
 }  // namespace ui