Make it possible to disable udev in linux

content/content_browser.gypi

 # Copyright (c) 2012 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.
 
 {
   'dependencies': [
     'browser/speech/proto/speech_proto.gyp:speech_proto',
     '../base/base.gyp:base_static',
     '../crypto/crypto.gyp:crypto',
     '../google_apis/google_apis.gyp:google_apis',
@@ skipping 374 matching lines @@
     'browser/devtools/render_view_devtools_agent_host.cc',
     'browser/devtools/render_view_devtools_agent_host.h',
     'browser/devtools/renderer_overrides_handler.cc',
     'browser/devtools/renderer_overrides_handler.h',
     'browser/devtools/tethering_handler.h',
     'browser/devtools/tethering_handler.cc',
     'browser/devtools/worker_devtools_manager.cc',
     'browser/devtools/worker_devtools_manager.h',
     'browser/devtools/worker_devtools_message_filter.cc',
     'browser/devtools/worker_devtools_message_filter.h',
-    'browser/device_monitor_linux.cc',
-    'browser/device_monitor_linux.h',
     'browser/device_monitor_mac.h',
     'browser/device_monitor_mac.mm',
+    'browser/device_monitor_udev.cc',
+    'browser/device_monitor_udev.h',
     'browser/device_orientation/data_fetcher_impl_android.cc',
     'browser/device_orientation/data_fetcher_impl_android.h',
     'browser/device_orientation/data_fetcher_shared_memory.h',
     'browser/device_orientation/data_fetcher_shared_memory_android.cc',
     'browser/device_orientation/data_fetcher_shared_memory_base.cc',
     'browser/device_orientation/data_fetcher_shared_memory_base.h',
     'browser/device_orientation/data_fetcher_shared_memory_default.cc',
     'browser/device_orientation/data_fetcher_shared_memory_mac.cc',
     'browser/device_orientation/data_fetcher_shared_memory_win.cc',
     'browser/device_orientation/device_data.h',
@@ skipping 826 matching lines @@
     '<(SHARED_INTERMEDIATE_DIR)/webkit/grit/devtools_resources_map.h',
     '<(SHARED_INTERMEDIATE_DIR)/content/browser/tracing/grit/tracing_resources.h',
     '<(SHARED_INTERMEDIATE_DIR)/ui/ui_resources/grit/webui_resources_map.cc',
   ],
   'conditions': [
     ['toolkit_views==1', {
       'dependencies': [
         '../ui/events/events.gyp:events',
       ],
     }],
-    ['OS!="win" and OS!="mac" and OS!="linux"', {
+    ['OS!="linux" or use_udev==0', {
       'sources': [
         'browser/gamepad/gamepad_platform_data_fetcher.cc',
       ]
     }],
     ['OS=="ios"', {
       'sources/': [
         # iOS only needs a small portion of content; exclude all the
         # implementation, and re-include what is used.
         ['exclude', '\\.(cc|mm)$'],
         ['include', '_ios\\.(cc|mm)$'],
@@ skipping 134 matching lines @@
         # For FcLangSetAdd call in render_sandbox_host_linux.cc
         '../build/linux/system.gyp:fontconfig',
         '../build/linux/system.gyp:gtk',
         # For XShm* in backing_store_x.cc
         '../build/linux/system.gyp:x11',
         '../dbus/dbus.gyp:dbus',
       ],
     }],
     ['OS=="linux"', {
       'dependencies': [
+        '../sandbox/sandbox.gyp:libc_urandom_override',
+      ],
+    }],
+    ['use_udev == 1', {
+      'dependencies': [
         '../build/linux/system.gyp:udev',
-        '../sandbox/sandbox.gyp:libc_urandom_override',
+      ],
+    }, {
+      'sources!': [

[rjkroege, 2013/11/18 22:32:19]

I didn't provide adequate information. I meant you to use an entry in
build/filename_rules.gypi to make this unnecessary. See the use_pango section
for example. The use of a naming rule can keep the gyp* file simpler.

All of these files if named with _udev.* would be excluded by the naming rule.
What do you think?

[Mostyn Bramley-Moore, 2013/11/18 23:32:26]

It might be a little confusing since there are other uses of udev in the chrome
layer, and these files don't end with _udev.*.  

Which reminds me of something from my old CL- this setting only affects the
content layer, should we rename it to something like content_use_udev?  I think
this is a reasonable distinction to make (ie disabling udev in content but not
in chrome).

+        'browser/device_monitor_udev.cc',
+        'browser/device_monitor_udev.h',
+        'browser/gamepad/gamepad_platform_data_fetcher_linux.cc',
+        'browser/udev_linux.cc',
+        'browser/udev_linux.h',
       ],
     }],
     ['OS=="linux" and use_aura==1', {
       'dependencies': [
         '../build/linux/system.gyp:dbus',
         '../build/linux/system.gyp:fontconfig',
         '../dbus/dbus.gyp:dbus',
       ],
     }],
     ['use_x11==1', {
@@ skipping 161 matching lines @@
         ['exclude', '^browser/speech/input_tag_speech_dispatcher_host\\.(cc|h)$'],
       ],
     }, {  # input_speech==1
       'dependencies': [
         '../third_party/flac/flac.gyp:libflac',
         '../third_party/speex/speex.gyp:libspeex',
       ],
     }],
   ],
 }