media: Fix widevine_cdm_manifest target

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.

Committed: https://crrev.com/35e11eb20c09bef5c5b8ba0eb3b352ab314d94d3
Cr-Commit-Position: refs/heads/master@{#397308}

[xhwang, 2016-06-01 21:59:32]

xhwang@chromium.org changed reviewers:
+ ddorwin@chromium.org

[xhwang, 2016-06-01 21:59:32]

PTAL

[ddorwin, 2016-06-01 23:21:09]

lgtm

[xhwang, 2016-06-01 23:23:01]

The CQ bit was checked by xhwang@chromium.org

[commit-bot: I haz the power, 2016-06-01 23:24:06]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2023383003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2023383003/1

[commit-bot: I haz the power, 2016-06-02 02:25:38]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-06-02 02:25:39]

Try jobs failed on following builders:
  linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[xhwang, 2016-06-02 03:55:05]

The CQ bit was checked by xhwang@chromium.org

[commit-bot: I haz the power, 2016-06-02 03:55:31]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2023383003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2023383003/1

[commit-bot: I haz the power, 2016-06-02 04:45:16]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-06-02 04:45:17]

Try jobs failed on following builders:
  linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[xhwang, 2016-06-02 05:46:33]

The CQ bit was checked by xhwang@chromium.org

[commit-bot: I haz the power, 2016-06-02 05:46:49]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2023383003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2023383003/1

[commit-bot: I haz the power, 2016-06-02 06:16:35]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-02 06:18:02]

Description was changed from

==========
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.
==========

to

==========
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.

Committed: https://crrev.com/35e11eb20c09bef5c5b8ba0eb3b352ab314d94d3
Cr-Commit-Position: refs/heads/master@{#397308}
==========

[commit-bot: I haz the power, 2016-06-02 06:18:02]

Patchset 1 (id:??) landed as
https://crrev.com/35e11eb20c09bef5c5b8ba0eb3b352ab314d94d3
Cr-Commit-Position: refs/heads/master@{#397308}