Add TextfieldTest.OnKeyPressBinding, which requires r273791.

Add TextfieldTest.OnKeyPressBinding, which requires r273791.

Fulfills the unit test request in the review of r273791.
https://codereview.chromium.org/307783003#msg3

Use a no-op TextEditKeyBindingsDelegateAuraLinux.
(replicates Emacs bindings ignoring [Shift+]Ctrl+Z)
Minor cleanup and test renaming.

BUG=377355
TEST=NONE
R=oshima@chromium.org

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

[msw, 2014-06-04 00:40:31]

Hey Oshima, please take a look; thanks!

[oshima, 2014-06-04 00:56:15]

lgtm

https://codereview.chromium.org/310283002/diff/1/ui/views/controls/textfield/...
File ui/views/controls/textfield/textfield_unittest.cc (right):

https://codereview.chromium.org/310283002/diff/1/ui/views/controls/textfield/...
ui/views/controls/textfield/textfield_unittest.cc:45: #elif defined(OS_LINUX) &&
!defined(OS_CHROMEOS)
I somehow thought we recommend if/endif for each config here, but I may be
wrong.

https://codereview.chromium.org/310283002/diff/1/ui/views/controls/textfield/...
ui/views/controls/textfield/textfield_unittest.cc:621:
ui::SetTextEditKeyBindingsDelegate(NULL);
optional: consider using scoped object to set/reset the delegate for test.

[msw, 2014-06-04 01:02:54]

Landing; thanks.

https://codereview.chromium.org/310283002/diff/1/ui/views/controls/textfield/...
File ui/views/controls/textfield/textfield_unittest.cc (right):

https://codereview.chromium.org/310283002/diff/1/ui/views/controls/textfield/...
ui/views/controls/textfield/textfield_unittest.cc:45: #elif defined(OS_LINUX) &&
!defined(OS_CHROMEOS)
On 2014/06/04 00:56:15, oshima wrote:
> I somehow thought we recommend if/endif for each config here, but I may be
> wrong. 

Changed to match that pattern.

https://codereview.chromium.org/310283002/diff/1/ui/views/controls/textfield/...
ui/views/controls/textfield/textfield_unittest.cc:621:
ui::SetTextEditKeyBindingsDelegate(NULL);
On 2014/06/04 00:56:15, oshima wrote:
> optional: consider using scoped object to set/reset the delegate for test.

Not worth it for one test, IMO.

[msw, 2014-06-04 01:02:57]

The CQ bit was checked by msw@chromium.org

[commit-bot: I haz the power, 2014-06-04 01:04:36]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/msw@chromium.org/310283002/60001

[commit-bot: I haz the power, 2014-06-04 11:30:32]

Change committed as 274777