Adds the UserInputMonitor implementation for Mac.

media/media.gyp

 # 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.
 
 {
   'variables': {
     'chromium_code': 1,
     # Override to dynamically link the cras (ChromeOS audio) library.
     'use_cras%': 0,
     # Option e.g. for Linux distributions to link pulseaudio directly
@@ skipping 286 matching lines @@
         'base/sinc_resampler.cc',
         'base/sinc_resampler.h',
         'base/stream_parser.cc',
         'base/stream_parser.h',
         'base/stream_parser_buffer.cc',
         'base/stream_parser_buffer.h',
         'base/text_track.h',
         'base/user_input_monitor.cc',
         'base/user_input_monitor.h',
         'base/user_input_monitor_linux.cc',
-        'base/user_input_monitor_mac.mm',
+        'base/user_input_monitor_mac.cc',
         'base/user_input_monitor_win.cc',
         'base/video_decoder.cc',
         'base/video_decoder.h',
         'base/video_decoder_config.cc',
         'base/video_decoder_config.h',
         'base/video_frame.cc',
         'base/video_frame.h',
         'base/video_renderer.cc',
         'base/video_renderer.h',
         'base/video_util.cc',
@@ skipping 311 matching lines @@
         ['OS=="linux"', {
           'variables': {
             'conditions': [
               ['sysroot!=""', {
                 'pkg-config': '../build/linux/pkg-config-wrapper "<(sysroot)" "<(target_arch)"',
               }, {
                 'pkg-config': 'pkg-config'
               }],
             ],
           },
+          'sources': [
+            'base/keyboard_event_counter.cc',
+            'base/keyboard_event_counter.h',
+          ],
           'conditions': [
             ['use_x11==1', {
               'link_settings': {
                 'libraries': [
                   '-lX11',
                   '-lXdamage',
                   '-lXext',
                   '-lXfixes',
                   '-lXtst',
                 ],
@@ skipping 25 matching lines @@
                 'audio/cras/cras_input.cc',
                 'audio/cras/cras_input.h',
                 'audio/cras/cras_unified.cc',
                 'audio/cras/cras_unified.h',
               ],
             }],
           ],
         }],
         ['OS!="linux"', {
           'sources!': [
             'audio/cras/audio_manager_cras.cc',

[Mark Mentovai, 2013/08/26 15:11:01]

We normally use excludes for this:

List keyboard_event_counter.{cc,h} in the main 'sources' list, and then list
them again here in 'sources!' to exclude them from the build on non-Linux.

[jiayl, 2013/08/26 17:16:58]

That won't work when the Windows impl is added. I would need to add a new
condition as OS != "linux" && OS != "windows".

[Mark Mentovai, 2013/08/26 19:07:55]

jiayl wrote:
That’s correct, you’ll need to do that once you add Windows.

Adding sources inside OS-specific blocks is not done in Chrome code. We always
add all sources and then conditionally exclude them.

The reason we do this is to make all of the source files visible in development
environments. With the source files listed unconditionally in 'sources', Xcode
and Visual Studio, for example, will list those files. That way, it’s easy for
another developer to find the files, view them, and make changes, even if
they’re editing on a Mac or Windows system that won’t compile them.

[jiayl, 2013/08/26 19:30:09]

Done. Thanks for the explanation.

             'audio/cras/audio_manager_cras.h',
             'audio/cras/cras_input.cc',
             'audio/cras/cras_input.h',
             'audio/cras/cras_unified.cc',
             'audio/cras/cras_unified.h',
           ],
         }],
         ['use_pulseaudio==1', {
           'cflags': [
             '<!@(pkg-config --cflags libpulse)',
@@ skipping 979 matching lines @@
           'sources': [
             'tools/media_bench/media_bench.cc',
           ],
           # TODO(jschuh): crbug.com/167187 fix size_t to int truncations.
           'msvs_disabled_warnings': [ 4267, ],
         },
       ],
     }],
   ],
 }