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

Unified Diff: media/audio/sounds/audio_stream_handler.h

Issue 115693004: Added volume adjust sound behind the flag. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix, rebase. Created 6 years, 11 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
« no previous file with comments | « chromeos/chromeos_switches.cc ('k') | media/audio/sounds/audio_stream_handler.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/audio/sounds/audio_stream_handler.h
diff --git a/media/audio/sounds/audio_stream_handler.h b/media/audio/sounds/audio_stream_handler.h
index 7c63a24f0348f3fb4228ed1bacde363ad773a772..f814aaef5993b793052fd54bf80105f8c634d57e 100644
--- a/media/audio/sounds/audio_stream_handler.h
+++ b/media/audio/sounds/audio_stream_handler.h
@@ -42,10 +42,12 @@ class MEDIA_EXPORT AudioStreamHandler : public base::NonThreadSafe {
// Returns true iff AudioStreamHandler is correctly initialized;
bool IsInitialized() const;
- // Stops any previous playback if it's still not completed and
- // starts new playback. Volume level will be set according to
- // current settings and won't be changed during playback. Returns
- // true iff new playback was successfully started.
+ // Plays sound. Volume level will be set according to current settings
+ // and won't be changed during playback. Returns true iff new playback
+ // was successfully started.
+ //
+ // NOTE: if current playback isn't at end of stream, playback request
+ // is dropped, but true is returned.
bool Play();
// Stops current playback.
« no previous file with comments | « chromeos/chromeos_switches.cc ('k') | media/audio/sounds/audio_stream_handler.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698