Extract a delegate interface from c/b/input_method to permit decoupling from c/b.

Extract a delegate interface from c/b/chromeos/input_method to permit decoupling from c/b.

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

[erikwright (departed), 2012-12-04 16:46:29]

zork@: PTAL.

With this change, the input_method implementation is nearly completely decoupled
from chrome/browser.

input_method_delegate_impl is considered part of the configuration layer and may
remain tightly coupled.

browser_state_monitor currently uses notification_service and
chrome_notification_type but this could be easily broken.

[Zachary Kuznia, 2012-12-05 07:47:53]

nona@, please have a look, too.

Looks generally good, please attach it to a bug.

[erikwright (departed), 2012-12-07 16:23:41]

nona@: ping

On 2012/12/05 07:47:53, Zachary Kuznia wrote:
> nona@, please have a look, too.
> 
> Looks generally good, please attach it to a bug.

[Seigo Nonaka, 2012-12-08 08:37:58]

lgtm

Lgtm, sorry for late reply.

https://codereview.chromium.org/11415266/diff/19007/chrome/browser/chromeos/i...
File chrome/browser/chromeos/input_method/mock_input_method_delegate.h (right):

https://codereview.chromium.org/11415266/diff/19007/chrome/browser/chromeos/i...
chrome/browser/chromeos/input_method/mock_input_method_delegate.h:24: virtual
void SetUserInputMethod(const std::string& input_method) OVERRIDE;
Nit: plz add override comment like "InputMethodDelegate overrides:" for
consistency.

[erikwright (departed), 2012-12-10 14:50:18]

jochen: PTAL for c/*.gypi and updated API name in c/b/ui/... .

[jochen (gone - plz use gerrit), 2012-12-10 14:52:56]

lgtm

[commit-bot: I haz the power, 2012-12-10 14:55:10]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/erikwright@chromium.org/11415266/25001

[commit-bot: I haz the power, 2012-12-10 22:03:46]

Change committed as 172140