ime: Implement the observer list for FakeInputMethod.

ime: Implement the observer list for FakeInputMethod.

BUG=none

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=194365

[sadrul, 2013-04-16 02:25:17]

https://codereview.chromium.org/13845017/diff/1/ui/base/ime/fake_input_method.cc
File ui/base/ime/fake_input_method.cc (right):

https://codereview.chromium.org/13845017/diff/1/ui/base/ime/fake_input_method...
ui/base/ime/fake_input_method.cc:114: void FakeInputMethod::OnFocus() {}
Is it necessary to trigger the observer callback for Focus/Blur too?

[Seigo Nonaka, 2013-04-16 07:49:18]

lgtm

https://codereview.chromium.org/13845017/diff/1/ui/base/ime/fake_input_method.cc
File ui/base/ime/fake_input_method.cc (right):

https://codereview.chromium.org/13845017/diff/1/ui/base/ime/fake_input_method...
ui/base/ime/fake_input_method.cc:114: void FakeInputMethod::OnFocus() {}
I think no as far as virtual keyboard is not the case.
Focus/Blur observer is used only in MockInputMethod and it is only used for
testing.

On 2013/04/16 02:25:18, sadrul wrote:
> Is it necessary to trigger the observer callback for Focus/Blur too?

[sadrul, 2013-04-16 15:14:55]

https://codereview.chromium.org/13845017/diff/1/ui/base/ime/fake_input_method.cc
File ui/base/ime/fake_input_method.cc (right):

https://codereview.chromium.org/13845017/diff/1/ui/base/ime/fake_input_method...
ui/base/ime/fake_input_method.cc:114: void FakeInputMethod::OnFocus() {}
On 2013/04/16 07:49:18, Seigo Nonaka wrote:
> I think no as far as virtual keyboard is not the case.
> Focus/Blur observer is used only in MockInputMethod and it is only used for
> testing.
> 
> On 2013/04/16 02:25:18, sadrul wrote:
> > Is it necessary to trigger the observer callback for Focus/Blur too?
> 

Ah, cool. Thanks!

[sadrul, 2013-04-16 15:15:56]

Committed patchset #1 manually as r194365 (presubmit successful).