Corner Passthrough for Accessibility

ash/accessibility_delegate.h

Index: ash/accessibility_delegate.h
diff --git a/ash/accessibility_delegate.h b/ash/accessibility_delegate.h
index 7dc2826e352796f6687366304c3b4260c48abfa3..1252068b121a1f0b51ec17ce077c1f0c71f3bf1d 100644
--- a/ash/accessibility_delegate.h
+++ b/ash/accessibility_delegate.h
@@ -8,6 +8,7 @@
 #include "ash/ash_export.h"
 #include "ash/magnifier/magnifier_constants.h"
 #include "base/time/time.h"
+#include "media/audio/sounds/sounds_manager.h"
 
 namespace ash {
 
@@ -92,6 +93,10 @@ class ASH_EXPORT AccessibilityDelegate {
   // Gets the last accessibility alert that was triggered.
   virtual AccessibilityAlert GetLastAccessibilityAlert() = 0;
 
+  // Plays an earcon. Earcons are brief and distinctive sounds that indicate
+  // when their mapped event has occurred.
+  virtual void PlayEarcon(media::SoundsManager::SoundKey sound_key) = 0;

[James Cook, 2014/07/29 04:27:24]

I would prefer you keep this an "int". I prefer interface definitions to use
plain-old-data types as much as possible.  In particular, interfaces are often
widely #included. Including sounds_manager.h means that every class that uses
this interface recursively includes hash_table.h, string_piece.h, etc.

Let me know if one of your other reviewers disagrees.

[dmazzoni, 2014/07/29 05:47:49]

That's a good reason. How about a comment here specifying the file where the
enums you're supposed to use are defined, then?

[lisayin, 2014/07/30 15:32:51]

Done.

+
   // Initiates play of shutdown sound and returns it's duration.
   virtual base::TimeDelta PlayShutdownSound() const = 0;
 };