Adding new mock classes for input method handlers

Adding new mock classes for input method handlers.

We need to migrate current tests to ibus independent implementation.
Following tests currently depends to ibus modules.
- ui/base/ime/input_method_ibus_unittests.cc
- chrome/browser/chromeos/input_method/input_method_engein_ibus_browsertest.cc
 (to be renamed to input_method_engine_browsertest.cc)

BUG=275262
TEST=None

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

[Seigo Nonaka, 2013-08-30 13:42:42]

Hi Satrou, could you take a look?

[satorux1, 2013-09-02 01:45:40]

Please update the patch description and describe why these new mocks are needed.

https://codereview.chromium.org/23822002/diff/6001/chromeos/ime/mock_ime_cand...
File chromeos/ime/mock_ime_candidate_window_handler.cc (right):

https://codereview.chromium.org/23822002/diff/6001/chromeos/ime/mock_ime_cand...
chromeos/ime/mock_ime_candidate_window_handler.cc:51:
set_cursor_location_call_count_++;
nit: ++set_cursor_location_call_count_;

pre-increment is used elsewhere in the file.

https://codereview.chromium.org/23822002/diff/6001/chromeos/ime/mock_ime_cand...
File chromeos/ime/mock_ime_candidate_window_handler.h (right):

https://codereview.chromium.org/23822002/diff/6001/chromeos/ime/mock_ime_cand...
chromeos/ime/mock_ime_candidate_window_handler.h:14: : public
IBusPanelCandidateWindowHandlerInterface {
IBusPanel? Is this name still relevant? Are you going to rename the class soon?

https://codereview.chromium.org/23822002/diff/6001/chromeos/ime/mock_ime_engi...
File chromeos/ime/mock_ime_engine_handler.cc (right):

https://codereview.chromium.org/23822002/diff/6001/chromeos/ime/mock_ime_engi...
chromeos/ime/mock_ime_engine_handler.cc:26: focus_in_call_count_++;
++process_key_event_call_count_. pre-increment is more common. please fix other
places too

[Seigo Nonaka, 2013-09-02 03:03:10]

https://codereview.chromium.org/23822002/diff/6001/chromeos/ime/mock_ime_cand...
File chromeos/ime/mock_ime_candidate_window_handler.cc (right):

https://codereview.chromium.org/23822002/diff/6001/chromeos/ime/mock_ime_cand...
chromeos/ime/mock_ime_candidate_window_handler.cc:51:
set_cursor_location_call_count_++;
On 2013/09/02 01:45:40, satorux1 wrote:
> nit: ++set_cursor_location_call_count_;
> 
> pre-increment is used elsewhere in the file.

Done.

https://codereview.chromium.org/23822002/diff/6001/chromeos/ime/mock_ime_cand...
File chromeos/ime/mock_ime_candidate_window_handler.h (right):

https://codereview.chromium.org/23822002/diff/6001/chromeos/ime/mock_ime_cand...
chromeos/ime/mock_ime_candidate_window_handler.h:14: : public
IBusPanelCandidateWindowHandlerInterface {
On 2013/09/02 01:45:40, satorux1 wrote:
> IBusPanel? Is this name still relevant? Are you going to rename the class
soon?

Yes I will remove or rename IBus prefixed classes.
Can I concentrate to remove IBus related logic?
Bug filed as crbug.com/283612

https://codereview.chromium.org/23822002/diff/6001/chromeos/ime/mock_ime_engi...
File chromeos/ime/mock_ime_engine_handler.cc (right):

https://codereview.chromium.org/23822002/diff/6001/chromeos/ime/mock_ime_engi...
chromeos/ime/mock_ime_engine_handler.cc:26: focus_in_call_count_++;
On 2013/09/02 01:45:40, satorux1 wrote:
> ++process_key_event_call_count_. pre-increment is more common. please fix
other
> places too

Done.

[satorux1, 2013-09-03 06:20:38]

lgtm

[satorux1, 2013-09-03 06:20:38]

lgtm

[commit-bot: I haz the power, 2013-09-03 09:58:04]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/nona@chromium.org/23822002/12001

[commit-bot: I haz the power, 2013-09-03 18:43:44]

Retried try job too often on win7_aura for step(s) browser_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win7_aura&...

[commit-bot: I haz the power, 2013-09-03 18:44:55]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/nona@chromium.org/23822002/12001

[commit-bot: I haz the power, 2013-09-03 20:02:00]

Change committed as 221004