[cros] Fix udevadm command line for camera presence detection.

[cros] Fix udevadm command line for camera presence detection.

Also added an overload of GetAppOutput which takes argv directly instead of CommandLine for launching programs where specific order of argument matters (like udevadm).

BUG=124845
TEST=Manual


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=134149

[Nikita (slow), 2012-04-25 14:30:50]

lgtm

[Ivan Korotkov, 2012-04-25 14:38:09]

Adding base/ OWNERS

[willchan no longer on Chromium, 2012-04-25 21:34:51]

lgtm

[willchan no longer on Chromium, 2012-04-25 21:36:09]

Actually, I take that back :P Why is the base/ change necessary? I just want to
make sure it's justified.

[Ivan Korotkov, 2012-04-26 07:50:59]

udevadm requires arguments to be in a specific order ('info' first) which
CommandLine doesn't allow (it orders all switches before non-switch arguments).
It's possible to circumvent by adding switches with AppendArg, but this isn't
error-prone (would break if AppendArg starts checking that it's argument isn't
actually a switch). So a clearer solution is to add an override with argv, like
there is for LaunchProcess().

[willchan no longer on Chromium, 2012-04-26 15:49:32]

Please update the CL description with this explanation and then LGTM.
On Apr 26, 2012 12:51 AM, <ivankr@chromium.org> wrote:

> udevadm requires arguments to be in a specific order ('info' first) which
> CommandLine doesn't allow (it orders all switches before non-switch
> arguments).
> It's possible to circumvent by adding switches with AppendArg, but this
> isn't
> error-prone (would break if AppendArg starts checking that it's argument
> isn't
> actually a switch). So a clearer solution is to add an override with argv,
> like
> there is for LaunchProcess().
>
>
https://chromiumcodereview.**appspot.com/10191019/<https://chromiumcodereview...
>

[willchan no longer on Chromium, 2012-04-26 15:56:45]

Please update the CL description with this explanation and then LGTM.
On Apr 26, 2012 12:51 AM, <ivankr@chromium.org> wrote:

> udevadm requires arguments to be in a specific order ('info' first) which
> CommandLine doesn't allow (it orders all switches before non-switch
> arguments).
> It's possible to circumvent by adding switches with AppendArg, but this
> isn't
> error-prone (would break if AppendArg starts checking that it's argument
> isn't
> actually a switch). So a clearer solution is to add an override with argv,
> like
> there is for LaunchProcess().
>
>
https://chromiumcodereview.**appspot.com/10191019/<https://chromiumcodereview...
>

[Ivan Korotkov, 2012-04-26 18:56:04]

Done

[commit-bot: I haz the power, 2012-04-26 18:56:18]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ivankr@chromium.org/10191019/6001

[commit-bot: I haz the power, 2012-04-26 20:24:38]

Change committed as 134149