Save/Restore ALSA volume/mute fix

Save/Restore ALSA volume/mute fix

After getting volume and mute to restore (Issue 6118006), there was one case where it did not: if volume 
was adjusted for the first time after a fresh install, it would not be remembered on next boot until 
mute was pressed at least once.

BUG=chromium-os:10470
TEST=Manual, volume should be the same after adjusting after fresh install and rebooting

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

[davejcool, 2011-01-21 19:29:15]

A very small update to the last Save/Restore ALSA volume settings patch.

http://codereview.chromium.org/6321014/diff/1/chrome/browser/chromeos/audio_m...
File chrome/browser/chromeos/audio_mixer_alsa.cc (left):

http://codereview.chromium.org/6321014/diff/1/chrome/browser/chromeos/audio_m...
chrome/browser/chromeos/audio_mixer_alsa.cc:304:
DoSetVolumeDb_Locked(pref_volume);
This check should have been removed once I added the block above to get the
current volume/mute value if we've never saved anything to prefs yet.

[zhurunz, 2011-01-21 19:41:28]

LGTM.

[scherkus (not reviewing), 2011-01-21 21:26:04]

LGTM w/ nit

http://codereview.chromium.org/6321014/diff/1/chrome/browser/chromeos/audio_m...
File chrome/browser/chromeos/audio_mixer_alsa.cc (right):

http://codereview.chromium.org/6321014/diff/1/chrome/browser/chromeos/audio_m...
chrome/browser/chromeos/audio_mixer_alsa.cc:304:
DoSetVolumeDb_Locked(pref_volume);
nit: indentation is off here

[davejcool, 2011-01-21 22:39:29]

http://codereview.chromium.org/6321014/diff/1/chrome/browser/chromeos/audio_m...
File chrome/browser/chromeos/audio_mixer_alsa.cc (right):

http://codereview.chromium.org/6321014/diff/1/chrome/browser/chromeos/audio_m...
chrome/browser/chromeos/audio_mixer_alsa.cc:304:
DoSetVolumeDb_Locked(pref_volume);
On 2011/01/21 21:26:04, scherkus wrote:
> nit: indentation is off here

D'oh!  This'll be formatted right in the next CL...