Commit 35e11eb2 authored by xhwang's avatar xhwang Committed by Commit bot

media: Fix widevine_cdm_manifest target

On Linux/ChromeOS, we don't use manifest.json for Widevine CDM.
Therefore the "widevine_cdm_manifest_file" list is empty. Then, in a GYP
build, copying an empty list to a destination will actually cause the
whole folder where the gyp file is located to be copied. See BUG for
more details.

This CL fixes this issue by only coping the manifest when the list is
not empty.

BUG=615500
TEST=Tested on Linux where the list is empty and on Windows where the
list is not empty.

Review-Url: https://codereview.chromium.org/2023383003
Cr-Commit-Position: refs/heads/master@{#397308}
parent 1bee53dc
......@@ -170,7 +170,7 @@
'target_name': 'widevine_cdm_manifest',
'type': 'none',
'conditions': [
[ 'branding == "Chrome"', {
[ 'widevine_cdm_manifest_file != []', {
'copies': [{
'destination': '<(PRODUCT_DIR)/WidevineCdm',
'files': [ '<(widevine_cdm_manifest_file)' ],
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment