Move the local struct definitions off ui/base/ime.

Move the local struct defines from ui::IMEEngineHandlerInterface/ui::IMEEngineObserver
to input_method::InputMethodEngineBase.

Currently, some structs like KeyboardEvent are MouseButtonEvent are defined under ui/base/ime/, but not is not exported by the component library.
These structs are used for InputMethodEngine for translating the Chrome IME API to the native. They are not suppose to be used and exported in ui/base/ime.

This CL moves the following structs:
KeyboardEvent, SegmentStyle, SegmentInfo
and related functions from ui::IMEEngineHandlerInterface to input_method::InputMethodEngineBase.

Moves the other structs:
MenuItemStyle, CandidateWindowPosition, MenuItem, UsageEntry, Candidate, CandidateWindowProperty
and the relate functions from ui::IMEEngineHandlerInterface to chromeos::InputMethodEngine, as these are all ChromeOS specific structs.

And moves the whole class:
ui::IMEEngineObserver
to input_method::InputMethodEngineBase::Observer.

BUG=517773
TEST=No compiling errors.

Committed: https://crrev.com/1c9621e940227de56b55f2da1c889a0bfc53e521
Cr-Commit-Position: refs/heads/master@{#369979}

[Azure Wei, 2016-01-15 13:10:25]

Description was changed from

==========
Move local struct defines from IMEEngineHandlerInterface to InputMethodEngine.

BUG=517773
TEST=No compiling errors.
==========

to

==========
Move the local struct defines from
ui::IMEEngineHandlerInterface/ui::IMEEngineObserver 
to input_method::InputMethodEngineBase.

Currently, some structs like KeyboardEvent are MouseButtonEvent are defined
under ui/base/ime/, but not is not exported by the component library.
These structs are used for InputMethodEngine for translating the Chrome IME API
to the native. They are not suppose to be used and exported in ui/base/ime.

This CL moves the following structs:
KeyboardEvent, SegmentStyle, SegmentInfo, MenuItemStyle,
CandidateWindowPosition, MenuItem, UsageEntry, Candidate,
CandidateWindowProperty
and related functions from ui::IMEEngineHandlerInterface to
input_method::InputMethodEngineBase.

And moves struct:
MouseButtonEvent 
and the related functions from ui::IMEEngineObserver to
input_method::InputMethodEngineBase::Observer.



BUG=517773
TEST=No compiling errors.
==========

[Azure Wei, 2016-01-15 13:10:35]

azurewei@chromium.org changed reviewers:
+ shuchen@chromium.org

[Azure Wei, 2016-01-15 13:10:53]

Hi Shu, can you please review this CL? Thank you!

[Shu Chen, 2016-01-15 13:46:23]

https://codereview.chromium.org/1590863002/diff/1/ui/base/ime/ime_engine_obse...
File ui/base/ime/ime_engine_observer.h (right):

https://codereview.chromium.org/1590863002/diff/1/ui/base/ime/ime_engine_obse...
ui/base/ime/ime_engine_observer.h:13: class IMEEngineObserver {
The whole IMEEngineObserver can be moved to InputMethodEngineBase.

[Azure Wei, 2016-01-17 04:38:04]

Description was changed from

==========
Move the local struct defines from
ui::IMEEngineHandlerInterface/ui::IMEEngineObserver 
to input_method::InputMethodEngineBase.

Currently, some structs like KeyboardEvent are MouseButtonEvent are defined
under ui/base/ime/, but not is not exported by the component library.
These structs are used for InputMethodEngine for translating the Chrome IME API
to the native. They are not suppose to be used and exported in ui/base/ime.

This CL moves the following structs:
KeyboardEvent, SegmentStyle, SegmentInfo, MenuItemStyle,
CandidateWindowPosition, MenuItem, UsageEntry, Candidate,
CandidateWindowProperty
and related functions from ui::IMEEngineHandlerInterface to
input_method::InputMethodEngineBase.

And moves struct:
MouseButtonEvent 
and the related functions from ui::IMEEngineObserver to
input_method::InputMethodEngineBase::Observer.



BUG=517773
TEST=No compiling errors.
==========

to

==========
Move the local struct defines from
ui::IMEEngineHandlerInterface/ui::IMEEngineObserver 
to input_method::InputMethodEngineBase.

Currently, some structs like KeyboardEvent are MouseButtonEvent are defined
under ui/base/ime/, but not is not exported by the component library.
These structs are used for InputMethodEngine for translating the Chrome IME API
to the native. They are not suppose to be used and exported in ui/base/ime.

This CL moves the following structs:
KeyboardEvent, SegmentStyle, SegmentInfo
and related functions from ui::IMEEngineHandlerInterface to
input_method::InputMethodEngineBase.

Moves the other structs:
MenuItemStyle, CandidateWindowPosition, MenuItem, UsageEntry, Candidate,
CandidateWindowProperty
and the relate functions from ui::IMEEngineHandlerInterface to
chromeos::InputMethodEngine, as these are all ChromeOS specific structs.

And moves the whole class:
ui::IMEEngineObserver 
to input_method::InputMethodEngineBase::Observer.

BUG=517773
TEST=No compiling errors.
==========

[Azure Wei, 2016-01-17 04:43:09]

On 2016/01/15 13:46:23, Shu Chen wrote:
>
https://codereview.chromium.org/1590863002/diff/1/ui/base/ime/ime_engine_obse...
> File ui/base/ime/ime_engine_observer.h (right):
> 
>
https://codereview.chromium.org/1590863002/diff/1/ui/base/ime/ime_engine_obse...
> ui/base/ime/ime_engine_observer.h:13: class IMEEngineObserver {
> The whole IMEEngineObserver can be moved to InputMethodEngineBase.

Done. Can you please help review the newest patch set? Thank you!

[Shu Chen, 2016-01-17 14:31:00]

The CQ bit was checked by shuchen@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-01-17 14:31:13]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1590863002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1590863002/40001

[Shu Chen, 2016-01-17 14:55:47]

lgtm

[commit-bot: I haz the power, 2016-01-17 20:33:24]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-01-17 20:33:25]

Dry run: Try jobs failed on following builders:
  linux_android_rel_ng on tryserver.chromium.android (JOB_TIMED_OUT,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[Azure Wei, 2016-01-18 01:51:39]

The CQ bit was checked by azurewei@chromium.org

[commit-bot: I haz the power, 2016-01-18 01:51:57]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1590863002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1590863002/40001

[commit-bot: I haz the power, 2016-01-18 02:53:36]

Description was changed from

==========
Move the local struct defines from
ui::IMEEngineHandlerInterface/ui::IMEEngineObserver 
to input_method::InputMethodEngineBase.

Currently, some structs like KeyboardEvent are MouseButtonEvent are defined
under ui/base/ime/, but not is not exported by the component library.
These structs are used for InputMethodEngine for translating the Chrome IME API
to the native. They are not suppose to be used and exported in ui/base/ime.

This CL moves the following structs:
KeyboardEvent, SegmentStyle, SegmentInfo
and related functions from ui::IMEEngineHandlerInterface to
input_method::InputMethodEngineBase.

Moves the other structs:
MenuItemStyle, CandidateWindowPosition, MenuItem, UsageEntry, Candidate,
CandidateWindowProperty
and the relate functions from ui::IMEEngineHandlerInterface to
chromeos::InputMethodEngine, as these are all ChromeOS specific structs.

And moves the whole class:
ui::IMEEngineObserver 
to input_method::InputMethodEngineBase::Observer.

BUG=517773
TEST=No compiling errors.
==========

to

==========
Move the local struct defines from
ui::IMEEngineHandlerInterface/ui::IMEEngineObserver 
to input_method::InputMethodEngineBase.

Currently, some structs like KeyboardEvent are MouseButtonEvent are defined
under ui/base/ime/, but not is not exported by the component library.
These structs are used for InputMethodEngine for translating the Chrome IME API
to the native. They are not suppose to be used and exported in ui/base/ime.

This CL moves the following structs:
KeyboardEvent, SegmentStyle, SegmentInfo
and related functions from ui::IMEEngineHandlerInterface to
input_method::InputMethodEngineBase.

Moves the other structs:
MenuItemStyle, CandidateWindowPosition, MenuItem, UsageEntry, Candidate,
CandidateWindowProperty
and the relate functions from ui::IMEEngineHandlerInterface to
chromeos::InputMethodEngine, as these are all ChromeOS specific structs.

And moves the whole class:
ui::IMEEngineObserver 
to input_method::InputMethodEngineBase::Observer.

BUG=517773
TEST=No compiling errors.
==========

[commit-bot: I haz the power, 2016-01-18 02:53:36]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-01-18 02:54:47]

Description was changed from

==========
Move the local struct defines from
ui::IMEEngineHandlerInterface/ui::IMEEngineObserver 
to input_method::InputMethodEngineBase.

Currently, some structs like KeyboardEvent are MouseButtonEvent are defined
under ui/base/ime/, but not is not exported by the component library.
These structs are used for InputMethodEngine for translating the Chrome IME API
to the native. They are not suppose to be used and exported in ui/base/ime.

This CL moves the following structs:
KeyboardEvent, SegmentStyle, SegmentInfo
and related functions from ui::IMEEngineHandlerInterface to
input_method::InputMethodEngineBase.

Moves the other structs:
MenuItemStyle, CandidateWindowPosition, MenuItem, UsageEntry, Candidate,
CandidateWindowProperty
and the relate functions from ui::IMEEngineHandlerInterface to
chromeos::InputMethodEngine, as these are all ChromeOS specific structs.

And moves the whole class:
ui::IMEEngineObserver 
to input_method::InputMethodEngineBase::Observer.

BUG=517773
TEST=No compiling errors.
==========

to

==========
Move the local struct defines from
ui::IMEEngineHandlerInterface/ui::IMEEngineObserver
to input_method::InputMethodEngineBase.

Currently, some structs like KeyboardEvent are MouseButtonEvent are defined
under ui/base/ime/, but not is not exported by the component library.
These structs are used for InputMethodEngine for translating the Chrome IME API
to the native. They are not suppose to be used and exported in ui/base/ime.

This CL moves the following structs:
KeyboardEvent, SegmentStyle, SegmentInfo
and related functions from ui::IMEEngineHandlerInterface to
input_method::InputMethodEngineBase.

Moves the other structs:
MenuItemStyle, CandidateWindowPosition, MenuItem, UsageEntry, Candidate,
CandidateWindowProperty
and the relate functions from ui::IMEEngineHandlerInterface to
chromeos::InputMethodEngine, as these are all ChromeOS specific structs.

And moves the whole class:
ui::IMEEngineObserver
to input_method::InputMethodEngineBase::Observer.

BUG=517773
TEST=No compiling errors.

Committed: https://crrev.com/1c9621e940227de56b55f2da1c889a0bfc53e521
Cr-Commit-Position: refs/heads/master@{#369979}
==========

[commit-bot: I haz the power, 2016-01-18 02:54:47]

Patchset 3 (id:??) landed as
https://crrev.com/1c9621e940227de56b55f2da1c889a0bfc53e521
Cr-Commit-Position: refs/heads/master@{#369979}