Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(110)

Unified Diff: remoting/protocol/input_event_tracker.cc

Issue 1403583003: Use standard DOM code definitions instead of custom. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix Windows compile. Created 5 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: remoting/protocol/input_event_tracker.cc
diff --git a/remoting/protocol/input_event_tracker.cc b/remoting/protocol/input_event_tracker.cc
index 8e6274d61b1070f27fd1c842ecae828c04266b1a..ed7b49ab36e0b31dc046d103d71fa10176507b5d 100644
--- a/remoting/protocol/input_event_tracker.cc
+++ b/remoting/protocol/input_event_tracker.cc
@@ -6,7 +6,6 @@
#include "base/logging.h"
#include "remoting/proto/event.pb.h"
-#include "remoting/protocol/usb_key_codes.h"
namespace remoting {
namespace protocol {
@@ -18,7 +17,7 @@ InputEventTracker::InputEventTracker(InputStub* input_stub)
InputEventTracker::~InputEventTracker() {}
-bool InputEventTracker::IsKeyPressed(uint32 usb_keycode) const {
+bool InputEventTracker::IsKeyPressed(ui::DomCode usb_keycode) const {
return pressed_keys_.find(usb_keycode) != pressed_keys_.end();
}
@@ -28,10 +27,10 @@ int InputEventTracker::PressedKeyCount() const {
void InputEventTracker::ReleaseAll() {
// Release all pressed keys.
- for (uint32 keycode : pressed_keys_) {
+ for (auto keycode : pressed_keys_) {
KeyEvent event;
event.set_pressed(false);
- event.set_usb_keycode(keycode);
+ event.set_usb_keycode(static_cast<unsigned int>(keycode));
Sergey Ulanov 2015/10/12 21:45:57 use uint32_t instead of unsigned int.
Jamie 2015/10/12 22:26:09 Done.
input_stub_->InjectKeyEvent(event);
}
pressed_keys_.clear();
@@ -73,17 +72,17 @@ void InputEventTracker::ReleaseAllIfModifiersStuck(bool alt_expected,
bool os_expected,
bool shift_expected) {
bool alt_down =
- pressed_keys_.find(kUsbLeftAlt) != pressed_keys_.end() ||
- pressed_keys_.find(kUsbRightAlt) != pressed_keys_.end();
+ pressed_keys_.find(ui::DomCode::ALT_LEFT) != pressed_keys_.end() ||
+ pressed_keys_.find(ui::DomCode::ALT_RIGHT) != pressed_keys_.end();
bool ctrl_down =
- pressed_keys_.find(kUsbLeftControl) != pressed_keys_.end() ||
- pressed_keys_.find(kUsbRightControl) != pressed_keys_.end();
+ pressed_keys_.find(ui::DomCode::CONTROL_LEFT) != pressed_keys_.end() ||
+ pressed_keys_.find(ui::DomCode::CONTROL_RIGHT) != pressed_keys_.end();
bool os_down =
- pressed_keys_.find(kUsbLeftOs) != pressed_keys_.end() ||
- pressed_keys_.find(kUsbRightOs) != pressed_keys_.end();
+ pressed_keys_.find(ui::DomCode::OS_LEFT) != pressed_keys_.end() ||
+ pressed_keys_.find(ui::DomCode::OS_RIGHT) != pressed_keys_.end();
bool shift_down =
- pressed_keys_.find(kUsbLeftShift) != pressed_keys_.end() ||
- pressed_keys_.find(kUsbRightShift) != pressed_keys_.end();
+ pressed_keys_.find(ui::DomCode::SHIFT_LEFT) != pressed_keys_.end() ||
+ pressed_keys_.find(ui::DomCode::SHIFT_RIGHT) != pressed_keys_.end();
if ((alt_down && !alt_expected) || (ctrl_down && !ctrl_expected) ||
(os_down && !os_expected) || (shift_down && !shift_expected)) {
@@ -99,9 +98,9 @@ void InputEventTracker::InjectKeyEvent(const KeyEvent& event) {
if (event.has_pressed()) {
if (event.has_usb_keycode()) {
if (event.pressed()) {
- pressed_keys_.insert(event.usb_keycode());
+ pressed_keys_.insert(static_cast<ui::DomCode>(event.usb_keycode()));
} else {
- pressed_keys_.erase(event.usb_keycode());
+ pressed_keys_.erase(static_cast<ui::DomCode>(event.usb_keycode()));
}
}
}

Powered by Google App Engine
This is Rietveld 408576698