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

Unified Diff: ash/system/audio/tray_audio.h

Issue 163953007: Refactor the TrayAudio code so that it can be used by other platforms. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 10 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: ash/system/audio/tray_audio.h
diff --git a/ash/system/chromeos/audio/tray_audio.h b/ash/system/audio/tray_audio.h
similarity index 71%
rename from ash/system/chromeos/audio/tray_audio.h
rename to ash/system/audio/tray_audio.h
index eb6e117a2c4b8805e31e818a812cd2b38cbe65f6..14fb6c9321d0d69ac44d4e918e3ae1038a832649 100644
--- a/ash/system/chromeos/audio/tray_audio.h
+++ b/ash/system/audio/tray_audio.h
@@ -2,26 +2,41 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#ifndef ASH_SYSTEM_CHROMEOS_AUDIO_TRAY_AUDIO_H_
-#define ASH_SYSTEM_CHROMEOS_AUDIO_TRAY_AUDIO_H_
+#ifndef ASH_SYSTEM_AUDIO_TRAY_AUDIO_H_
+#define ASH_SYSTEM_AUDIO_TRAY_AUDIO_H_
+#include "ash/system/audio/audio_observer.h"
#include "ash/system/tray/tray_image_item.h"
-#include "chromeos/audio/cras_audio_handler.h"
namespace ash {
+
+namespace system {
+class TrayAudioDelegate;
+}
+
namespace internal {
namespace tray {
class VolumeView;
-class AudioDetailedView;
}
class TrayAudio : public TrayImageItem,
- public chromeos::CrasAudioHandler::AudioObserver {
+ public AudioObserver {
public:
- explicit TrayAudio(SystemTray* system_tray);
+ explicit TrayAudio(SystemTray* system_tray,
+ system::TrayAudioDelegate* audio_delegate);
jennyz 2014/02/14 21:36:43 Remove explicit since the constructor now takes 2
zturner 2014/02/15 00:57:45 Done.
virtual ~TrayAudio();
+ protected:
+ virtual void Update();
+
+ system::TrayAudioDelegate* audio_delegate_;
+ tray::VolumeView* volume_view_;
+
+ // True if VolumeView should be created for accelerator pop up;
+ // Otherwise, it should be created for detailed view in ash tray bubble.
+ bool pop_up_volume_view_;
+
private:
// Overridden from TrayImageItem.
virtual bool GetInitialVisibility() OVERRIDE;
@@ -34,28 +49,17 @@ class TrayAudio : public TrayImageItem,
virtual bool ShouldHideArrow() const OVERRIDE;
virtual bool ShouldShowShelf() const OVERRIDE;
- // Overridden from chromeos::CrasAudioHandler::AudioObserver.
+ // Overridden from AudioObserver.
virtual void OnOutputVolumeChanged() OVERRIDE;
virtual void OnOutputMuteChanged() OVERRIDE;
- virtual void OnInputGainChanged() OVERRIDE;
- virtual void OnInputMuteChanged() OVERRIDE;
virtual void OnAudioNodesChanged() OVERRIDE;
virtual void OnActiveOutputNodeChanged() OVERRIDE;
virtual void OnActiveInputNodeChanged() OVERRIDE;
- void Update();
-
- tray::VolumeView* volume_view_;
- tray::AudioDetailedView* audio_detail_;
-
- // True if VolumeView should be created for accelerator pop up;
- // Otherwise, it should be created for detailed view in ash tray bubble.
- bool pop_up_volume_view_;
-
DISALLOW_COPY_AND_ASSIGN(TrayAudio);
};
} // namespace internal
} // namespace ash
-#endif // ASH_SYSTEM_CHROMEOS_AUDIO_TRAY_AUDIO_H_
+#endif // ASH_SYSTEM_AUDIO_TRAY_AUDIO_H_

Powered by Google App Engine
This is Rietveld 408576698