Reland "Move TextInputController from CppBoundClass to gin::Wrappable."

Reland "Move TextInputController from CppBoundClass to gin::Wrappable."

Also, change the class to use chromium coding style.

BUG=297480, 331301
R=jochen@chromium.org

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

[tfarina, 2014-02-18 04:21:19]

Jochen, could you patch this in?

[jochen (gone - plz use gerrit), 2014-02-18 09:13:39]

lgtm

The original text input controller returns undefined on failure which breaks the
entire layout test, the new controller returns an empty array and so the test
actually does something.

I think we should rebaseline the test. I've added a temporary expectation so you
can land this.

https://codereview.chromium.org/169683006/diff/1/content/shell/renderer/test_...
File content/shell/renderer/test_runner/text_input_controller.cc (right):

https://codereview.chromium.org/169683006/diff/1/content/shell/renderer/test_...
content/shell/renderer/test_runner/text_input_controller.cc:225: if
(!view_->mainFrame()->firstRectForCharacterRange(location, length, rect))
this should be focusedFrame()

[tfarina, 2014-02-18 12:50:02]

The CQ bit was checked by tfarina@chromium.org

[commit-bot: I haz the power, 2014-02-18 12:50:13]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tfarina@chromium.org/169683006/90001

[tfarina, 2014-02-18 12:50:42]

Thanks Jochen!

https://codereview.chromium.org/169683006/diff/1/content/shell/renderer/test_...
File content/shell/renderer/test_runner/text_input_controller.cc (right):

https://codereview.chromium.org/169683006/diff/1/content/shell/renderer/test_...
content/shell/renderer/test_runner/text_input_controller.cc:225: if
(!view_->mainFrame()->firstRectForCharacterRange(location, length, rect))
On 2014/02/18 09:13:39, jochen wrote:
> this should be focusedFrame()

Done.

[commit-bot: I haz the power, 2014-02-18 16:32:58]

Change committed as 251773