Implementation of 'AudioContext.getOutputTimestamp' method

media/audio/audio_io.h

Index: media/audio/audio_io.h
diff --git a/media/audio/audio_io.h b/media/audio/audio_io.h
index 66ff3f2dc98cf7af2324fa1cb58944e29c003342..e9f8168b67f17a1d6012354a4a165563e51545c3 100644
--- a/media/audio/audio_io.h
+++ b/media/audio/audio_io.h
@@ -49,6 +49,17 @@
 
 namespace media {
 
+struct MEDIA_EXPORT AudioTimestamp {
+  AudioTimestamp() = default;
+  int64_t frames;

[miu, 2016/07/19 01:22:36]

Why do we need this? Client code could track this by keeping a running-sum of
AudioBus::frames(), if it's needed for a particular use case.

+  int64_t ticks;  // Obtained from base::TimeTicks::ToInternalValue().

[miu, 2016/07/19 01:22:36]

Don't use ToInternalValue(). As the header comments in base/time/time.h explain,
this is only for wire serialization. Client code should never attempt to
interpret the meaning of the TimeTicks's internal value.

naming nit: This should probably be called |reference_time| for clarity.

+};
+
+// Used in unittests.
+inline bool operator==(const AudioTimestamp& one, const AudioTimestamp& other) {
+  return one.ticks == other.ticks && one.frames == other.frames;
+}
+
 class MEDIA_EXPORT AudioOutputStream {
  public:
   // Audio sources must implement AudioSourceCallback. This interface will be
@@ -63,9 +74,13 @@ class MEDIA_EXPORT AudioOutputStream {
     // the number of frames it filled.  |total_bytes_delay| contains current
     // number of bytes of delay buffered by the AudioOutputStream.
     // |frames_skipped| contains the number of frames skipped by the consumer.
-    virtual int OnMoreData(AudioBus* dest,
-                           uint32_t total_bytes_delay,
-                           uint32_t frames_skipped) = 0;
+    // |timestamp| if provided, contains timestamp of the currently audible
+    // signal.
+    virtual int OnMoreData(
+        AudioBus* dest,
+        uint32_t total_bytes_delay,
+        uint32_t frames_skipped,

[miu, 2016/07/19 01:22:36]

This is in conflict with this change:
https://codereview.chromium.org/2101303004/

You guys need to chat and come to a consensus here. ;-)

(I've also stated some of my opinions on what this API should become, in that
code review.)

[jameswest, 2016/07/20 04:19:18]

We're in favor of changing the API to the following, which was suggested by
Yuri:

virtual int OnMoreData(
    base::TimeTicks target_playout_time,
    int prior_frames_skipped,
    AudioBus* dest);

To get |target_playout_time|, you would convert |total_bytes_delay| to time
duration and add it to base::TimeTicks::Now().

Does that work for you?

+        const AudioTimestamp& output_timestamp = AudioTimestamp()) = 0;
 
     // There was an error while playing a buffer. Audio source cannot be
     // destroyed yet. No direct action needed by the AudioStream, but it is