Chromoting Android app mouse/keyboard bugfixes and enhancements

remoting/client/jni/jni_interface.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.
 
 // This file defines functions that implement the static methods declared in a
 // closely-related Java class in the platform-specific user interface
 // implementation.  In effect, it is the entry point for all JNI calls *into*
 // the C++ codebase from Java.  The separate ChromotingJniRuntime class serves
 // as the corresponding exit point, and is responsible for making all JNI calls
 // *out of* the C++ codebase into Java.
@@ skipping 102 matching lines @@
   remoting::ChromotingJniRuntime::GetInstance()->session()->RedrawDesktop();
 }
 
 JNIEXPORT void JNICALL JNI_IMPLEMENTATION(mouseActionNative)(
     JNIEnv* env,
     jobject that,
     jint x,
     jint y,
     jint which_button,
     jboolean button_down) {
+  // Button must be within the bounds of the MouseEvent_MouseButton enum.
+  DCHECK(which_button >= 0 && which_button < 5);
+
   remoting::ChromotingJniRuntime::GetInstance()->session()->PerformMouseAction(
       x,
       y,
       static_cast<remoting::protocol::MouseEvent_MouseButton>(which_button),
       button_down);
 }
 
 JNIEXPORT void JNICALL JNI_IMPLEMENTATION(keyboardActionNative)(
     JNIEnv* env,
     jobject that,
     jint key_code,
     jboolean key_down) {
   remoting::ChromotingJniRuntime::GetInstance()->session()->
       PerformKeyboardAction(key_code, key_down);
 }
 
 }  // extern "C"