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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 250803005: Devices without devnode are filtered out. (Closed)

Created:
6 years, 8 months ago by ygorshenin1
Modified:
6 years, 7 months ago
CC:
chromium-reviews
Visibility:
Public.

Description

Devices without devnode are filtered out. BUG=127016 TEST=manual R=rockot@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=266546

Patch Set 1 #

Patch Set 2 : Removed syspath. #

Patch Set 3 : Fix. #

Patch Set 4 : Rebase. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+27 lines, -15 lines) Patch
M device/hid/input_service_linux.cc View 1 2 4 chunks +26 lines, -14 lines 0 comments Download
M device/hid/input_service_linux_unittest.cc View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 20 (0 generated)
ygorshenin1
Ken, could you please take a look?
6 years, 8 months ago (2014-04-25 14:50:11 UTC) #1
Ken Rockot(use gerrit already)
lgtm
6 years, 8 months ago (2014-04-25 23:08:11 UTC) #2
ygorshenin1
The CQ bit was checked by ygorshenin@chromium.org
6 years, 8 months ago (2014-04-26 15:21:28 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/ygorshenin@chromium.org/250803005/40001
6 years, 8 months ago (2014-04-26 15:21:53 UTC) #4
commit-bot: I haz the power
The CQ bit was unchecked by commit-bot@chromium.org
6 years, 8 months ago (2014-04-26 16:02:24 UTC) #5
commit-bot: I haz the power
Try jobs failed on following builders: tryserver.chromium on chromium_presubmit
6 years, 8 months ago (2014-04-26 16:02:24 UTC) #6
ygorshenin1
The CQ bit was checked by ygorshenin@chromium.org
6 years, 8 months ago (2014-04-27 10:08:36 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/ygorshenin@chromium.org/250803005/40001
6 years, 8 months ago (2014-04-27 10:09:29 UTC) #8
commit-bot: I haz the power
The CQ bit was unchecked by commit-bot@chromium.org
6 years, 8 months ago (2014-04-27 10:31:29 UTC) #9
commit-bot: I haz the power
Try jobs failed on following builders: tryserver.chromium on chromium_presubmit
6 years, 8 months ago (2014-04-27 10:31:29 UTC) #10
Ken Rockot(use gerrit already)
On 2014/04/27 10:31:29, I haz the power (commit-bot) wrote: > Try jobs failed on following ...
6 years, 8 months ago (2014-04-27 17:12:58 UTC) #11
Ken Rockot(use gerrit already)
The CQ bit was checked by rockot@chromium.org
6 years, 8 months ago (2014-04-27 17:13:14 UTC) #12
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/ygorshenin@chromium.org/250803005/40001
6 years, 8 months ago (2014-04-27 17:13:59 UTC) #13
commit-bot: I haz the power
The CQ bit was unchecked by commit-bot@chromium.org
6 years, 8 months ago (2014-04-27 17:59:08 UTC) #14
commit-bot: I haz the power
Try jobs failed on following builders: tryserver.chromium on chromium_presubmit
6 years, 8 months ago (2014-04-27 17:59:09 UTC) #15
ygorshenin1
The CQ bit was checked by ygorshenin@chromium.org
6 years, 7 months ago (2014-04-28 10:34:26 UTC) #16
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/ygorshenin@chromium.org/250803005/60001
6 years, 7 months ago (2014-04-28 10:35:22 UTC) #17
commit-bot: I haz the power
The CQ bit was unchecked by commit-bot@chromium.org
6 years, 7 months ago (2014-04-28 11:54:32 UTC) #18
commit-bot: I haz the power
Try jobs failed on following builders: tryserver.chromium on linux_chromium_chromeos_rel
6 years, 7 months ago (2014-04-28 11:54:32 UTC) #19
ygorshenin1
6 years, 7 months ago (2014-04-28 13:51:44 UTC) #20
Message was sent while issue was closed. 
Committed patchset #4 manually as r266546 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698