Fixs bug where keyevents weren't created correctly on X

mojo/services/native_viewport/native_viewport_x11.cc

 // Copyright 2013 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 "mojo/services/native_viewport/native_viewport.h"
 
 #include <X11/Xlib.h>
 #include <X11/Xutil.h>
 
 #include "base/message_loop/message_loop.h"
@@ skipping 114 matching lines @@
         return false;
     }
   }
 
   virtual uint32_t DispatchEvent(const ui::PlatformEvent& event) OVERRIDE {
     if (event->type == ClientMessage) {
       Atom protocol = static_cast<Atom>(event->xclient.data.l[0]);
       if (protocol == atom_wm_delete_window_)
         delegate_->OnDestroyed();
     } else if (event->type == KeyPress || event->type == KeyRelease) {
-      ui::KeyEvent key_event(event, true);
+      ui::KeyEvent key_event(event, false);

[darin (slow to review), 2014/06/19 05:12:15]

heh, I have this in my tree too!

       delegate_->OnEvent(&key_event);
     } else if (event->type == ButtonPress || event->type == ButtonRelease ||
                event->type == MotionNotify) {
       ui::MouseEvent mouse_event(event);
       delegate_->OnEvent(&mouse_event);
     }
     return ui::POST_DISPATCH_NONE;
   }
 
   scoped_ptr<ui::PlatformEventSource> event_source_;
   NativeViewportDelegate* delegate_;
   gfx::Rect bounds_;
   XID window_;
   Atom atom_wm_protocols_;
   Atom atom_wm_delete_window_;
 
   DISALLOW_COPY_AND_ASSIGN(NativeViewportX11);
 };
 
 // static
 scoped_ptr<NativeViewport> NativeViewport::Create(
     shell::Context* context,
     NativeViewportDelegate* delegate) {
   return scoped_ptr<NativeViewport>(new NativeViewportX11(delegate)).Pass();
 }
 
 }  // namespace services
 }  // namespace mojo