ChromeOS: login: Pass the use-cras option to guest mode.

ChromeOS: login: Pass the use-cras option to guest mode.

Pass the --use-cras (kUseCras) option to 'browse as guest' sessions.
Before this the guest session would use the alsa plugin.

BUG=chromium:129243
TEST=Manual, 'browse as guest', then play audio.  Check that the audio
server is being used (through logs or clients list in chrome://system).

Signed-off-by: Dylan Reid <dgreid@chromium.org>;


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

[dgreid, 2012-05-24 03:40:40]

Also addresses crosbug.com/31211

[no longer working on chromium, 2012-05-24 07:24:11]

On 2012/05/24 03:40:40, dgreid wrote:
> Also addresses crosbug.com/31211

lgtm

[scherkus (not reviewing), 2012-05-24 17:29:40]

lgtm -- you still may need an OWNERS

[dgreid, 2012-05-24 17:35:04]

add more owners.

[whywhat, 2012-05-24 17:36:03]

lgtm

[commit-bot: I haz the power, 2012-05-24 17:43:28]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dgreid@chromium.org/10452005/1

[commit-bot: I haz the power, 2012-05-24 18:10:42]

Try job failure for 10452005-1 (retry) on linux_chromeos for step "compile"
(clobber build).
It's a second try, previously, step "compile" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_chro...

[commit-bot: I haz the power, 2012-05-24 20:14:41]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dgreid@chromium.org/10452005/1

[commit-bot: I haz the power, 2012-05-24 20:40:38]

Try job failure for 10452005-1 (retry) on linux_chromeos for step "compile"
(clobber build).
It's a second try, previously, step "compile" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_chro...

[dgreid, 2012-05-25 17:12:24]

Slight modification to only pass this flag when the build has CRAS in it (which
fixes the -Dchromeos=1 on linux w/o cras build).  Everybody still OK with this?

[scherkus (not reviewing), 2012-05-25 17:32:46]

still lgtm

[commit-bot: I haz the power, 2012-05-25 17:37:35]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dgreid@chromium.org/10452005/1003

[commit-bot: I haz the power, 2012-05-25 18:20:17]

Try job failure for 10452005-1003 (retry) on mac_rel for step "check_deps".
It's a second try, previously, step "check_deps" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...

[dgreid, 2012-05-25 18:54:13]

Now it doesn't like my including a media file from here.  That makes sense. 
Would it be best to move the Cras flag to somewhere else (chrome_switches
maybe)?  Then change the media/audio_manage_linux to look there?

Thanks for any advise.

Dylan

[whywhat, 2012-05-25 18:57:17]

Sounds good. I think we mostly have all Chrome switches in one or two places.

On 2012/05/25 18:54:13, dgreid wrote:
> Now it doesn't like my including a media file from here.  That makes sense. 
> Would it be best to move the Cras flag to somewhere else (chrome_switches
> maybe)?  Then change the media/audio_manage_linux to look there?
> 
> Thanks for any advise.
> 
> Dylan

[commit-bot: I haz the power, 2012-05-27 00:49:25]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dgreid@chromium.org/10452005/14003

[commit-bot: I haz the power, 2012-05-27 02:07:01]

Change committed as 139207