Magnifier needs to follow the focus of the textfield.

ui/base/ime/input_method_chromeos.h

 // 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.
 
 #ifndef UI_BASE_IME_INPUT_METHOD_CHROMEOS_H_
 #define UI_BASE_IME_INPUT_METHOD_CHROMEOS_H_
 
 #include <set>
 #include <string>
 
@@ skipping 96 matching lines @@
   virtual void DeleteSurroundingText(int32 offset, uint32 length) override;
 
   // Hides the composition text.
   void HidePreeditText();
 
   // Callback function for IMEEngineHandlerInterface::ProcessKeyEvent.
   void ProcessKeyEventDone(uint32 id, ui::KeyEvent* event, bool is_handled);
 
   // Returns whether an input field is focused. Note that password field is not
   // considered as an input field.
   bool IsInputFieldFocused();

[Jun Mukai, 2014/10/22 22:22:54]

It is really weird to have IsInputFieldFocused and IsTextInputFieldFocused at
the same time... :-/

Can we rename this method?  An idea: IsNonPasswordInputFieldFocused().  shuchen
may have some other opinions though.

[jennyz, 2014/10/22 22:29:00]

Yes, it does sound weird. The only reason I didn't rename it is because it is
used in many places, I am trying to keep it as is. If no one is annoyed for
renaming it, I will go ahead rename it as Is
IsNonPasswordInputFieldFocus().

[Shu Chen, 2014/10/23 08:03:57]

Looks like it is only used in ui/base/ime/input_method_chromeos.cc, which you
already included in this cl.
So please rename it for better readability. Thanks!

[jennyz, 2014/10/23 18:10:11]

Done.

[jennyz, 2014/10/23 18:10:11]

Done.

 
+  // Returns true if an text input field is focused. This includes the password
+  // field.
+  bool IsTextInputFieldFocused();
+
   // All pending key events. Note: we do not own these object, we just save
   // pointers to these object so that we can abandon them when necessary.
   // They will be deleted in ProcessKeyEventDone().
   std::set<uint32> pending_key_events_;
 
   // Pending composition text generated by the current pending key event.
   // It'll be sent to the focused text input client as soon as we receive the
   // processing result of the pending key event.
   CompositionText composition_;
 
@@ skipping 20 matching lines @@
 
   // Used for making callbacks.
   base::WeakPtrFactory<InputMethodChromeOS> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(InputMethodChromeOS);
 };
 
 }  // namespace ui
 
 #endif  // UI_BASE_IME_INPUT_METHOD_CHROMEOS_H_