Mac AVFoundation: minor bugfix using QTKit for devices not working with AVF (Blackmagic).

Mac AVFoundation: minor bugfix using QTKit for devices not working with AVF (Blackmagic).

This is a minor bugfix to https://codereview.chromium.org/229063003,
changing the comparison of USB VID to ignore the case.

BUG=288562, 347371
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=266550

[mcasas, 2014-04-28 08:01:30]

tommi@: OWNERS stamp please.

[tommi (sloooow) - chröme, 2014-04-28 08:17:10]

https://codereview.chromium.org/255933006/diff/20001/media/video/capture/mac/...
File media/video/capture/mac/video_capture_device_mac.mm (right):

https://codereview.chromium.org/255933006/diff/20001/media/video/capture/mac/...
media/video/capture/mac/video_capture_device_mac.mm:28: // is written uppercase.
nit: written in uppercase

Btw, would it make sense to have a case insensitive check against the blacklist?

[mcasas, 2014-04-28 08:31:20]

tommi@ PTAL.

https://codereview.chromium.org/255933006/diff/20001/media/video/capture/mac/...
File media/video/capture/mac/video_capture_device_mac.mm (right):

https://codereview.chromium.org/255933006/diff/20001/media/video/capture/mac/...
media/video/capture/mac/video_capture_device_mac.mm:28: // is written uppercase.
On 2014/04/28 08:17:10, tommi wrote:
> nit: written in uppercase
> 
> Btw, would it make sense to have a case insensitive check against the
blacklist?

Yeah I doubted between that and forcing uppercase
in the list, but I guess the case-insensitive is more 
robust, so done.

[tommi (sloooow) - chröme, 2014-04-28 08:45:00]

lgtm

[mcasas, 2014-04-28 08:53:57]

The CQ bit was checked by mcasas@chromium.org

[commit-bot: I haz the power, 2014-04-28 08:54:25]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mcasas@chromium.org/255933006/40001

[commit-bot: I haz the power, 2014-04-28 10:39:07]

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

[commit-bot: I haz the power, 2014-04-28 10:39:07]

Try jobs failed on following builders:
  tryserver.chromium on win_chromium_rel

[mcasas, 2014-04-28 11:14:22]

The CQ bit was checked by mcasas@chromium.org

[commit-bot: I haz the power, 2014-04-28 11:15:10]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mcasas@chromium.org/255933006/40001

[commit-bot: I haz the power, 2014-04-28 13:59:48]

Change committed as 266550