Handle WM_SYSKEYDOWN in NativeTextfieldWin.

views/controls/textfield/native_textfield_win.cc

 // Copyright (c) 2010 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/controls/textfield/native_textfield_win.h"
 
 #include <algorithm>
 
 #include "app/keyboard_codes.h"
 #include "app/keyboard_code_conversion_win.h"
@@ skipping 881 matching lines @@
                                          UINT repeat_count,
                                          UINT flags) {
   ScopedFreeze freeze(this, GetTextObjectModel());
 
   Textfield::Controller* controller = textfield_->GetController();
   bool handled = false;
   if (controller) {
     Event::EventType type;
     switch (message) {
       case WM_KEYDOWN:
+      case WM_SYSKEYDOWN:

[Finnur, 2011/01/11 21:27:26]

This would take care of the NOTREACHED, but I'm not sure what this means in the
grand scheme of things. Maybe the person who reviewed your big changelist that
introduced this should take a look?

       case WM_CHAR:
         type = Event::ET_KEY_PRESSED;
         break;
       case WM_KEYUP:

[Finnur, 2011/01/11 21:28:54]

Will you not get WM_SYSKEYUP also?

         type = Event::ET_KEY_RELEASED;
         break;
       default:
         NOTREACHED() << "Unknown message:" << message;
         // Passing through to avoid crash on release build.
         type = Event::ET_KEY_PRESSED;
     }
     KeyEvent key_event(type,
                        app::KeyboardCodeForWindowsKeyCode(key),
                        KeyEvent::GetKeyStateFlags(),
@@ skipping 195 matching lines @@
 ////////////////////////////////////////////////////////////////////////////////
 // NativeTextfieldWrapper, public:
 
 // static
 NativeTextfieldWrapper* NativeTextfieldWrapper::CreateWrapper(
     Textfield* field) {
   return new NativeTextfieldWin(field);
 }
 
 }  // namespace views