Include version number with ClearKey and WideVine cdmadapters

media/media_cdm.gypi

 # 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.
 
 {
   'variables': {
     'conditions': [
       ['OS == "android"', {
         # Android doesn't use ffmpeg.
         'use_ffmpeg%': 0,
       }, {  # 'OS != "android"'
         'use_ffmpeg%': 1,
       }],
     ],
     # Set |use_fake_video_decoder| to 1 to ignore input frames in |clearkeycdm|,
     # and produce video frames filled with a solid color instead.
     'use_fake_video_decoder%': 0,
     # Set |use_libvpx| to 1 to use libvpx for VP8 decoding in |clearkeycdm|.
     'use_libvpx%': 0,
   },
   'conditions': [
     ['enable_pepper_cdms==1', {
-      'targets': [
+        'includes': [
+          '../build/util/version.gypi',
+        ],
+        'targets': [
         {
           'target_name': 'clearkeycdm',
           'type': 'none',
           # TODO(tomfinegan): Simplify this by unconditionally including all the
           # decoders, and changing clearkeycdm to select which decoder to use
           # based on environment variables.
           'conditions': [
             ['use_fake_video_decoder == 1' , {
               'defines': ['CLEAR_KEY_CDM_USE_FAKE_VIDEO_DECODER'],
               'sources': [
@@ skipping 53 matching lines @@
             'cdm/ppapi/external_clear_key/cdm_video_decoder.cc',
             'cdm/ppapi/external_clear_key/cdm_video_decoder.h',
             'cdm/ppapi/external_clear_key/clear_key_cdm.cc',
             'cdm/ppapi/external_clear_key/clear_key_cdm.h',
             'cdm/ppapi/external_clear_key/clear_key_cdm_common.h',
           ],
           # TODO(jschuh): crbug.com/167187 fix size_t to int truncations.
           'msvs_disabled_warnings': [ 4267, ],
         },
         {
+          'target_name': 'clearkeycdmadapter_resources',
+          'type': 'none',
+          'conditions': [
+            ['branding == "Chrome"', {
+              'variables': {
+                'branding_path': '../chrome/app/theme/google_chrome/BRANDING',
+              },
+            }, { # else branding!="Chrome"
+              'variables': {
+                'branding_path': '../chrome/app/theme/chromium/BRANDING',
+              },
+            }],
+          ],
+          'variables': {
+            'output_dir': '.',
+            'template_input_path': '../chrome/app/chrome_version.rc.version',
+          },
+          'sources': [
+            'clearkeycdmadapter.ver',
+          ],
+          'includes': [
+            '../chrome/version_resource_rules.gypi',
+          ],
+        },

[xhwang, 2014/10/31 18:25:04]

Is there a way to actually put these in the cdmadapter target, but somehow be
able to customize 

 'sources': [
             'clearkeycdmadapter.ver',
  ],

or INTERNAL_NAME and ORIGINAL_FILENAME directly?

[jrummell, 2014/10/31 23:03:11]

From what I've been able to tell, it appears that variables are limited in
scope. The only way to pass them would be to include a .gypi file (which would
see any variables defined before the include). I don't think that will help
much, since it would still be independent of the cdmadapter rules.

Given that .gyp will be going away, is it worth putting in the effort to try to
have the version stuff in one place?

+        {
           'target_name': 'clearkeycdmadapter',
           'type': 'none',
           # Check whether the plugin's origin URL is valid.
           'defines': ['CHECK_DOCUMENT_URL'],
           'dependencies': [
             '<(DEPTH)/ppapi/ppapi.gyp:ppapi_cpp',
             'media_cdm_adapter.gyp:cdmadapter',
             'clearkeycdm',
+            'clearkeycdmadapter_resources',
+          ],
+          'sources': [
+            '<(SHARED_INTERMEDIATE_DIR)/clearkeycdmadapter_version.rc',
           ],
           'conditions': [
             ['os_posix == 1 and OS != "mac" and enable_pepper_cdms==1', {
               # Because clearkeycdm has type 'loadable_module' (see comments),
               # we must explicitly specify this dependency.
               'libraries': [
                 # Built by clearkeycdm.
                 '<(PRODUCT_DIR)/libclearkeycdm.so',
               ],
             }],
           ],
         },
       ],
     }],
   ],
 }