Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(64)

Side by Side Diff: media/cdm_paths.gypi

Issue 2004253002: media: More cleanup in cdm_paths.* (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: further limit where we use platform specific path Created 4 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « media/cdm/ppapi/cdm_paths.gni ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright 2016 The Chromium Authors. All rights reserved. 1 # Copyright 2016 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 # Common variables for media CDM. 5 # Common variables for media CDM.
6 6
7 # Naming and folder structure below are following the recommendation for Chrome 7 # Naming and folder structure below are following the recommendation for Chrome
8 # components. Component-updated CDMs must follow the same recommendation. 8 # components. Component-updated CDMs must follow the same recommendation.
9 9
10 # Note: This file must be in sync with cdm_paths.cc 10 # Note: This file must be in sync with cdm_paths.cc
11 11
12 # GN version: //media/cdm/ppapi/cdm_paths.gni 12 # GN version: //media/cdm/ppapi/cdm_paths.gni
13 { 13 {
14 'variables': { 14 'variables': {
15 'variables': { 15 'variables': {
16 'conditions': [ 16 'conditions': [
17 # OS name for components is close to "<(OS)" but has some differences. 17 # OS name for components is close to "<(OS)" but has some differences.
18 # Explicitly define what we use to avoid confusion. 18 # Explicitly define what we use to avoid confusion.
19 ['OS == "linux" and chromeos == 1', { 19 ['OS == "linux" and chromeos == 1', {
20 'component_os%': 'cros' 20 'component_os%': 'cros'
21 }, 'OS == "linux"', { 21 }, 'OS == "linux"', {
22 'component_os%': 'linux' 22 'component_os%': 'linux'
23 }, 'OS == "win"', { 23 }, 'OS == "win"', {
24 'component_os%': 'win' 24 'component_os%': 'win'
25 }, 'OS == "mac"', { 25 }, 'OS == "mac"', {
26 'component_os%': 'mac' 26 'component_os%': 'mac'
27 }, { 27 }, {
28 'component_os%': '<(OS)' 28 'component_os%': 'unsupported_platform'
29 }], 29 }],
30 # Architecture name for components is close to "<(current_cpu)" but has 30 # Architecture name for components is close to "<(current_cpu)" but has
31 # some differences. Explicitly define what we use to avoid confusion. 31 # some differences. Explicitly define what we use to avoid confusion.
32 ['target_arch == "ia32"', { 32 ['target_arch == "ia32"', {
33 'component_arch%': 'x86' 33 'component_arch%': 'x86'
34 }, 'target_arch == "x64"', { 34 }, 'target_arch == "x64"', {
35 'component_arch%': 'x64' 35 'component_arch%': 'x64'
36 }, 'target_arch == "arm"', { 36 }, 'target_arch == "arm"', {
37 'component_arch%': 'arm' 37 'component_arch%': 'arm'
38 }, { 38 }, {
39 'component_arch%': '<(current_cpu)' 39 'component_arch%': 'unsupported_arch'
40 }], 40 }],
41 ], 41 ],
42 }, 42 },
43 'conditions' : [ 43 'conditions' : [
44 # Only enable platform specific path for Win and Mac, where CDMs are 44 # Only enable platform specific path for Win and Mac, where CDMs are
45 # Chrome components. 45 # Chrome components.
46 # TODO(xhwang): Improve how we enable platform specific path. See 46 # TODO(xhwang): Improve how we enable platform specific path. See
47 # http://crbug.com/468584 47 # http://crbug.com/468584
48 ['OS == "win" or OS == "mac"', { 48 ['( OS == "win" or OS == "mac") and (target_arch == "ia32" or target_arch == "x64")', {
49 # Path of Clear Key and Widevine CDMs relative to the output dir. 49 # Path of Clear Key and Widevine CDMs relative to the output dir.
50 'widevine_cdm_path%': 'WidevineCdm/_platform_specific/<(component_os)_<( component_arch)', 50 'widevine_cdm_path%': 'WidevineCdm/_platform_specific/<(component_os)_<( component_arch)',
51 'clearkey_cdm_path%': 'ClearKeyCdm/_platform_specific/<(component_os)_<( component_arch)', 51 'clearkey_cdm_path%': 'ClearKeyCdm/_platform_specific/<(component_os)_<( component_arch)',
52 }, { 52 }, {
53 'widevine_cdm_path%': '.', 53 'widevine_cdm_path%': '.',
54 'clearkey_cdm_path%': '.', 54 'clearkey_cdm_path%': '.',
55 }], 55 }],
56 ] 56 ]
57 }, 57 },
58 } 58 }
OLDNEW
« no previous file with comments | « media/cdm/ppapi/cdm_paths.gni ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698