Commit 144d0d0b authored by Harry Cutts's avatar Harry Cutts Committed by Commit Bot

ozone: evdev: remove redundant `break`s

There's no need for these since they all come right after a `return`.

Bug: none
Change-Id: I5ddf94cc2a06dcabf77044546dbb16393f7b8afc
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2465214
Auto-Submit: Harry Cutts <hcutts@chromium.org>
Reviewed-by: default avatarMichael Spang <spang@chromium.org>
Commit-Queue: Michael Spang <spang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#816585}
parent 52d78b5e
...@@ -448,25 +448,18 @@ bool IsDeviceOfType(const ui::GesturePropertyProvider::DevicePtr device, ...@@ -448,25 +448,18 @@ bool IsDeviceOfType(const ui::GesturePropertyProvider::DevicePtr device,
switch (type) { switch (type) {
case ui::DT_KEYBOARD: case ui::DT_KEYBOARD:
return (evdev_class == EvdevClassKeyboard); return (evdev_class == EvdevClassKeyboard);
break;
case ui::DT_MOUSE: case ui::DT_MOUSE:
return is_mouse; return is_mouse;
break;
case ui::DT_TOUCHPAD: case ui::DT_TOUCHPAD:
return (!is_mouse) && is_touchpad; return (!is_mouse) && is_touchpad;
break;
case ui::DT_TOUCHSCREEN: case ui::DT_TOUCHSCREEN:
return (evdev_class == EvdevClassTouchscreen); return (evdev_class == EvdevClassTouchscreen);
break;
case ui::DT_MULTITOUCH: case ui::DT_MULTITOUCH:
return is_touchpad; return is_touchpad;
break;
case ui::DT_MULTITOUCH_MOUSE: case ui::DT_MULTITOUCH_MOUSE:
return is_mouse && is_touchpad; return is_mouse && is_touchpad;
break;
case ui::DT_ALL: case ui::DT_ALL:
return true; return true;
break;
default: default:
break; break;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment