Quick fix for NativeTextfieldViewsTest on win aura.

views/events/event_aura.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "views/events/event.h"
 
 #include "base/logging.h"
 #include "ui/aura/event.h"
 
 namespace views {
@@ skipping 10 matching lines @@
 // KeyEvent, public:
 
 KeyEvent::KeyEvent(const NativeEvent& native_event)
     : Event(native_event, native_event->type(), native_event->flags()),
       key_code_(static_cast<aura::KeyEvent*>(native_event)->key_code()),
       character_(0),
       unmodified_character_(0) {
 }
 
 uint16 KeyEvent::GetCharacter() const {
-  NOTIMPLEMENTED();
-  return key_code_;
+  return character_;

[oshima, 2011/10/05 20:15:09]

windows key event has the following code.

  return (native_event().message == WM_CHAR) ? key_code_ :
      GetCharacterFromKeyCode(key_code_, flags());

this isn't necessary any more (on win)?

[msw, 2011/10/05 20:45:34]

I'm not sure; I think the key_code/[unmodified_]/character code should be
revisited in a followup change. I'm not familiar with Yusuke's work, but it
appears a bit overly complex and platform-dependent... Should I investigate that
work as part of this CL?

[oshima, 2011/10/05 21:03:24]

No, if this works on the test, that should be enough for now. can you add him to
the bug?

[msw, 2011/10/05 22:52:20]

Added Yusuke to the bug; he's also cc'ed on this CR.

 }
 
 uint16 KeyEvent::GetUnmodifiedCharacter() const {
-  NOTIMPLEMENTED();
-  return key_code_;
+  return unmodified_character_;
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // MouseWheelEvent, public:
 
 MouseWheelEvent::MouseWheelEvent(const NativeEvent& native_event)
     : MouseEvent(native_event),
       offset_(0 /* TODO(beng): obtain */) {
 }
 
 }  // namespace views