media: unittests: depend on xdisplaycheck for isolate

media: unittests: depend on xdisplaycheck for isolate

BUG=470959
Committed: https://crrev.com/ff1d51c33ba28d2bfbc714aebe3856a104e2767d
Cr-Commit-Position: refs/heads/master@{#322546}

[llandwerlin-old, 2015-03-26 18:45:38]

lionel.g.landwerlin@intel.com changed reviewers:
+ dalecurtis@chromium.org

[llandwerlin-old, 2015-03-26 18:45:38]

PTAL, Thanks.

[llandwerlin-old, 2015-03-26 18:46:02]

lionel.g.landwerlin@intel.com changed reviewers:
+ maruel@chromium.org

[M-A Ruel, 2015-03-26 19:44:12]

small style nits, logic is good.

https://codereview.chromium.org/1032323003/diff/20001/media/media.gyp
File media/media.gyp (right):

https://codereview.chromium.org/1032323003/diff/20001/media/media.gyp#newcode...
media/media.gyp:1419: ['use_alsa==1', {
do you mind fixing alignment for this line while at it? Thanks.

https://codereview.chromium.org/1032323003/diff/20001/media/media.gyp#newcode...
media/media.gyp:1424: ['use_x11==1',
If you look around, format is:

['use_x11==1', {

https://codereview.chromium.org/1032323003/diff/20001/media/media.gyp#newcode...
media/media.gyp:1429: }
}],
is the idiomatic format.

[DaleCurtis, 2015-03-26 20:30:02]

lgtm % maruel's nits.

[llandwerlin-old, 2015-03-26 22:28:47]

https://codereview.chromium.org/1032323003/diff/20001/media/media.gyp
File media/media.gyp (right):

https://codereview.chromium.org/1032323003/diff/20001/media/media.gyp#newcode...
media/media.gyp:1419: ['use_alsa==1', {
On 2015/03/26 19:44:11, M-A Ruel wrote:
> do you mind fixing alignment for this line while at it? Thanks.

What's wrong with this particular line?

https://codereview.chromium.org/1032323003/diff/20001/media/media.gyp#newcode...
media/media.gyp:1424: ['use_x11==1',
On 2015/03/26 19:44:11, M-A Ruel wrote:
> If you look around, format is:
> 
> ['use_x11==1', {

Done.

https://codereview.chromium.org/1032323003/diff/20001/media/media.gyp#newcode...
media/media.gyp:1429: }
On 2015/03/26 19:44:11, M-A Ruel wrote:
> }],
> is the idiomatic format.

Done.

[llandwerlin-old, 2015-03-27 09:28:48]

The CQ bit was checked by lionel.g.landwerlin@intel.com

[llandwerlin-old, 2015-03-27 09:28:49]

The patchset sent to the CQ was uploaded after l-g-t-m from
dalecurtis@chromium.org
Link to the patchset: https://codereview.chromium.org/1032323003/#ps40001
(title: "nits")

[commit-bot: I haz the power, 2015-03-27 09:29:12]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1032323003/40001

[commit-bot: I haz the power, 2015-03-27 10:43:01]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-03-27 10:43:43]

Patchset 3 (id:??) landed as
https://crrev.com/ff1d51c33ba28d2bfbc714aebe3856a104e2767d
Cr-Commit-Position: refs/heads/master@{#322546}