Move X11 device processing to worker threads

ui/events/devices/device_util_linux.cc

 // Copyright 2014 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/devices/device_util_linux.h"
 
 #include <string>
 
 #include "base/files/file_enumerator.h"
 #include "base/files/file_path.h"
 #include "base/strings/string_util.h"
 
 namespace ui {
 
+// We consider the touchscreen to be internal if it is an I2c device. We search
+// all the dev input nodes registered by I2C devices to see if we can find
+// eventXXX.
 bool IsTouchscreenInternal(const base::FilePath& path) {
   std::string event_node = path.BaseName().value();

[sadrul, 2014/11/06 21:05:27]

Can you add CHECK_NE(TYPE_UI, MessageLoop::current()->type()) here?

[dnicoara, 2014/11/10 19:34:51]

Worker threads don't have a message loop. Is there anything else that you know
of, that I could try?

As a side note, I'm wondering why the ThreadRestrictions assertions weren't
triggered?

[sadrul, 2014/11/11 18:29:51]

Perhaps CHECK(!MessageLoopForUI::IsCurrent())?
I was wondering that too ... I have no idea. Perhaps because we happen to do
this before we set the restrictions?

   if (event_node.empty() || !StartsWithASCII(event_node, "event", false))
     return false;
 
   // Extract id "XXX" from "eventXXX"
   std::string event_node_id = event_node.substr(5);
 
   // I2C input device registers its dev input node at
   // /sys/bus/i2c/devices/*/input/inputXXX/eventXXX
   base::FileEnumerator i2c_enum(
       base::FilePath(FILE_PATH_LITERAL("/sys/bus/i2c/devices/")),
@@ skipping 10 matching lines @@
          input = input_enum.Next()) {
       if (input.BaseName().value().substr(5) == event_node_id)
         return true;
     }
   }
 
   return false;
 }
 
 }  // namespace