Switch browser side audio capture path to use base time primitives.

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 6ab50aa952e88e5ca97c3d903d40da530d59e104..9744df018422abdc66da469e4860d24535cf1ee4 100644
--- a/media/audio/audio_io.h
+++ b/media/audio/audio_io.h
@@ -119,18 +119,11 @@ class MEDIA_EXPORT AudioInputStream {
     // Called by the audio recorder when a full packet of audio data is
     // available. This is called from a special audio thread and the
     // implementation should return as soon as possible.
-    // TODO(henrika): should be pure virtual when old OnData() is phased out.
     virtual void OnData(AudioInputStream* stream,
                         const AudioBus* source,
-                        uint32_t hardware_delay_bytes,
-                        double volume) {}
-
-    // TODO(henrika): don't use; to be removed.
-    virtual void OnData(AudioInputStream* stream,
-                        const uint8_t* src,
-                        uint32_t size,
-                        uint32_t hardware_delay_bytes,
-                        double volume) {}
+                        base::TimeDelta delay,

[o1ka, 2017/02/10 13:28:50]

Are we expecting that TimeTicks::IsHighResolution() is true on all the
platforms? Should we have a check somewhere for that?

+                        base::TimeTicks delay_timestamp,

[o1ka, 2017/02/10 13:28:50]

It's hard for me to interpret what "delay timestamp" means. What is it counted
from? Is it a "buffer timestamp"?

How is |delay| calculated? Is it a time interval between the moment the physical
signal is received and the moment OnData() is called? How the user should
interpret it?

Is  |delay_timestamp| - |delay| equal to the physical time of the signal?


Would be nice to have comments explaining the contract, and more descriptive
naming (maybe |latency| and |timestamp| will do?)

+                        double volume) = 0;
 
     // There was an error while recording audio. The audio sink cannot be
     // destroyed yet. No direct action needed by the AudioInputStream, but it