Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(70)

Unified Diff: chromeos/dbus/cras_audio_client_stub_impl.h

Issue 515573003: app_shell: Do simple audio initialization. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: add tests Created 6 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chromeos/dbus/cras_audio_client_stub_impl.h
diff --git a/chromeos/dbus/cras_audio_client_stub_impl.h b/chromeos/dbus/cras_audio_client_stub_impl.h
index e363e933a026f063edd1223e057b3caddf9fd62a..e6960c7207598892cbf23dc5775c59c22e5e3970 100644
--- a/chromeos/dbus/cras_audio_client_stub_impl.h
+++ b/chromeos/dbus/cras_audio_client_stub_impl.h
@@ -36,14 +36,11 @@ class CrasAudioClientStubImpl : public CrasAudioClient {
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
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
private:
- friend class CrasAudioHandlerTest;
-
VolumeState volume_state_;
AudioNodeList node_list_;
uint64 active_input_node_id_;

Powered by Google App Engine
This is Rietveld 408576698