app_shell: Do simple audio initialization.

chromeos/dbus/cras_audio_client_stub_impl.h

 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROMEOS_DBUS_CRAS_AUDIO_CLIENT_STUB_IMPL_H_
 #define CHROMEOS_DBUS_CRAS_AUDIO_CLIENT_STUB_IMPL_H_
 
 #include "chromeos/dbus/cras_audio_client.h"
 
 namespace chromeos {
@@ skipping 18 matching lines @@
   virtual void SetOutputUserMute(bool mute_on) OVERRIDE;
   virtual void SetInputNodeGain(uint64 node_id, int32 gain) OVERRIDE;
   virtual void SetInputMute(bool mute_on) OVERRIDE;
   virtual void SetActiveOutputNode(uint64 node_id) OVERRIDE;
   virtual void SetActiveInputNode(uint64 node_id) OVERRIDE;
   virtual void AddActiveInputNode(uint64 node_id) OVERRIDE;
   virtual void RemoveActiveInputNode(uint64 node_id) OVERRIDE;
   virtual void AddActiveOutputNode(uint64 node_id) OVERRIDE;
   virtual void RemoveActiveOutputNode(uint64 node_id) OVERRIDE;
 
- protected:
   // Helper functions for testing
   virtual void SetAudioDevices(const AudioNodeList& audio_nodes);
   virtual void ChangeAudioNodes(const AudioNodeList& new_nodes);

[Daniel Erat, 2014/08/28 04:00:29]

these are called in a bunch of places; let me know if you want me to add
ForTesting suffixes to their names. (since they're only defined in the stub, i
don't think there's much danger of them being misused, though.)

[jennyz, 2014/08/28 17:42:49]

Please add ForTesting suffixes.

[Daniel Erat, 2014/08/28 23:27:57]

done (also renamed to SetAudioNodesForTesting and
SetAudioNodesAndNotifyObserversForTesting)

 
  private:
-  friend class CrasAudioHandlerTest;
-
   VolumeState volume_state_;
   AudioNodeList node_list_;
   uint64 active_input_node_id_;
   uint64 active_output_node_id_;
   ObserverList<Observer> observers_;
 
   DISALLOW_COPY_AND_ASSIGN(CrasAudioClientStubImpl);
 };
 
 }  // namespace chromeos
 
 #endif  // CHROMEOS_DBUS_CRAS_AUDIO_CLIENT_STUB_IMPL_H_