MacViews: Implement a ui::InputMethod for Mac

MacViews: Implement a ui::InputMethod for Mac

Add InputMethodMac class and update InputMethodFactory to create it.
It doesn't do anything for now. We plan to bridge NSTextInputClient calls
directly to the focused ui::TextInputClient object.

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

[Andre, 2014-06-02 21:22:42]

Yuki, PTAL.
This is the first piece I'd like to land, more coming later as separate CLs.

[Yuki, 2014-06-03 07:50:15]

lgtm

https://codereview.chromium.org/312523003/diff/20001/ui/base/ime/input_method...
File ui/base/ime/input_method_mac.mm (right):

https://codereview.chromium.org/312523003/diff/20001/ui/base/ime/input_method...
ui/base/ime/input_method_mac.mm:41: return true;
I'm not sure if you really want to return true or not.
You may want to return false because your InputMethodMac doesn't work, or you
may want to return true because IME (NSTextInputClient or something) works on
Mac, too.

[Andre, 2014-06-03 17:55:03]

https://codereview.chromium.org/312523003/diff/20001/ui/base/ime/input_method...
File ui/base/ime/input_method_mac.mm (right):

https://codereview.chromium.org/312523003/diff/20001/ui/base/ime/input_method...
ui/base/ime/input_method_mac.mm:41: return true;
On 2014/06/03 07:50:15, Yuki wrote:
> I'm not sure if you really want to return true or not.
> You may want to return false because your InputMethodMac doesn't work, or you
> may want to return true because IME (NSTextInputClient or something) works on
> Mac, too.

I agree, it's not clear.
This will not be called on the Mac because the only call site is in
RenderWidgetHostViewAura.
I'm thinking true is better for now.

[Andre, 2014-06-03 17:55:07]

The CQ bit was checked by andresantoso@chromium.org

[commit-bot: I haz the power, 2014-06-03 17:55:41]

CQ is trying da patch. Follow status at

https://chromium-status.appspot.com/cq/andresantoso@chromium.org/312523003/20001

[commit-bot: I haz the power, 2014-06-03 21:36:29]

Change committed as 274636