evdev: Support device hotplug with udev

ui/events/ozone/evdev/event_device_info.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.
 
 #include "ui/events/ozone/evdev/event_device_info.h"
 
 #include <linux/input.h>
 
 #include "base/logging.h"
 #include "base/threading/thread_restrictions.h"
 
 namespace ui {
 
 namespace {
 
 bool GetEventBits(int fd, unsigned int type, void* buf, unsigned int size) {
-  base::ThreadRestrictions::AssertIOAllowed();

[rjkroege, 2014/01/30 22:18:09]

for ozone, it would be really nice to have a "only io for events is allowed on
the ui thread" instead of "all I/O is allowed in here"  what do you think?

[spang, 2014/01/31 17:33:00]

Should that go in base::ThreadRestrictions?

I think that sounds like a separate change ;)

-
   if (ioctl(fd, EVIOCGBIT(type, size), buf) < 0) {
     DLOG(ERROR) << "failed EVIOCGBIT(" << type << ", " << size << ") on fd "
                 << fd;
     return false;
   }
 
   return true;
 }
 
 bool GetPropBits(int fd, void* buf, unsigned int size) {
-  base::ThreadRestrictions::AssertIOAllowed();
-
   if (ioctl(fd, EVIOCGPROP(size), buf) < 0) {
     DLOG(ERROR) << "failed EVIOCGPROP(" << size << ") on fd " << fd;
     return false;
   }
 
   return true;
 }
 
 bool BitIsSet(const unsigned long* bits, unsigned int bit) {
   return (bits[bit / EVDEV_LONG_BITS] & (1UL << (bit % EVDEV_LONG_BITS)));
@@ skipping 84 matching lines @@
   return BitIsSet(led_bits_, code);
 }
 
 bool EventDeviceInfo::HasProp(unsigned int code) const {
   if (code > INPUT_PROP_MAX)
     return false;
   return BitIsSet(prop_bits_, code);
 }
 
 }  // namespace ui