blimp: Fix input events sent from the client to the engine

blimp/client/input/blimp_input_handler_wrapper.cc

Index: blimp/client/input/blimp_input_handler_wrapper.cc
diff --git a/blimp/client/input/blimp_input_handler_wrapper.cc b/blimp/client/input/blimp_input_handler_wrapper.cc
index a06c92bf58b1608642872a83281710336188dccd..62516cbb637d7c83d336aba2f9d1edb3918904fe 100644
--- a/blimp/client/input/blimp_input_handler_wrapper.cc
+++ b/blimp/client/input/blimp_input_handler_wrapper.cc
@@ -44,7 +44,16 @@ void BlimpInputHandlerWrapper::HandleWebInputEvent(
   ui::InputHandlerProxy::EventDisposition disposition =
       input_handler_proxy_->HandleInputEvent(*input_event);
 
-  bool consumed = disposition == ui::InputHandlerProxy::DID_NOT_HANDLE;
+  bool consumed;

[Wez, 2016/01/15 20:58:18]

Initialize consumed to false here!

[Khushal, 2016/01/15 21:03:21]

Done.

+
+  switch (disposition) {
+    case ui::InputHandlerProxy::EventDisposition::DID_HANDLE:
+    case ui::InputHandlerProxy::EventDisposition::DROP_EVENT:
+      consumed = true;
+      break;
+    case ui::InputHandlerProxy::EventDisposition::DID_NOT_HANDLE:
+      consumed = false;

[Wez, 2016/01/15 20:58:18]

nit: This case should have a break; even though it's the final one, to guard
against later addition of a new case breaking things.

[Khushal, 2016/01/15 21:03:21]

Done.

+  }
 
   main_task_runner_->PostTask(
       FROM_HERE,