Make more media APIs aware of |delay| and |delay_timestamp|

media/audio/audio_device_thread.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef MEDIA_AUDIO_AUDIO_DEVICE_THREAD_H_
 #define MEDIA_AUDIO_AUDIO_DEVICE_THREAD_H_
 
 #include <stdint.h>
 
 #include "base/macros.h"
@@ skipping 24 matching lines @@
              int total_segments);
 
     // One time initialization for the callback object on the audio thread.
     void InitializeOnAudioThread();
 
     // Derived implementations must call shared_memory_.Map appropriately
     // before Process can be called.
     virtual void MapSharedMemory() = 0;
 
     // Called whenever we receive notifications about pending input data.
-    virtual void Process(uint32_t pending_data) = 0;
+    virtual void Process(int64_t pending_data,

[chcunningham, 2016/10/21 18:19:47]

Can you use a TimeDelta in this signature (and calling it delay and
delay_timestamp like we do browser side).

[Mikhail, 2016/10/24 19:50:24]

The reason for the current naming is that AudioInputDevice is using it in a
different way (pls see AudioInputDevice::AudioThreadCallback::Process):
|pending_data| contains shmem segment there, so I had to keep naming generic.

+                         base::TimeTicks data_timestamp) = 0;
 
    protected:
     virtual ~Callback();
 
     // Protected so that derived classes can access directly.
     // The variables are 'const' since values are calculated/set in the
     // constructor and must never change.
     const AudioParameters audio_parameters_;
 
     base::SharedMemory shared_memory_;
@@ skipping 12 matching lines @@
 
   // Creates and automatically starts the audio thread.
   AudioDeviceThread(Callback* callback,
                     base::SyncSocket::Handle socket,
                     const char* thread_name);
 
   // This tells the audio thread to stop and clean up the data; this is a
   // synchronous process and the thread will stop before the method returns.
   ~AudioDeviceThread() override;
 
+  // Represents a data packet to be received via the socket.
+  struct Packet {
+    int64_t pending_data;

[chcunningham, 2016/10/21 18:19:47]

Can you name these like you do browser side? 

delay;
delay_timestamp;

[Mikhail, 2016/10/24 19:50:24]

Same motivation as in the comment above.

+    int64_t data_timestamp_us;
+  };
+
  private:
   void ThreadMain() final;
 
   Callback* const callback_;
   const char* thread_name_;
   base::CancelableSyncSocket socket_;
   base::PlatformThreadHandle thread_handle_;
 
   DISALLOW_COPY_AND_ASSIGN(AudioDeviceThread);
 };
 
 }  // namespace media.
 
 #endif  // MEDIA_AUDIO_AUDIO_DEVICE_THREAD_H_