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

content/browser/renderer_host/media/audio_input_sync_writer.cc

Index: content/browser/renderer_host/media/audio_input_sync_writer.cc
diff --git a/content/browser/renderer_host/media/audio_input_sync_writer.cc b/content/browser/renderer_host/media/audio_input_sync_writer.cc
index e706934dfbcd52e3a517def4bfb8e41bfacb0520..961de95242aa72d28df1a98447100f2605d2ec02 100644
--- a/content/browser/renderer_host/media/audio_input_sync_writer.cc
+++ b/content/browser/renderer_host/media/audio_input_sync_writer.cc
@@ -12,10 +12,12 @@
 #include "build/build_config.h"
 #include "content/browser/renderer_host/media/media_stream_manager.h"
 #include "content/public/browser/browser_thread.h"
+#include "media/audio/audio_device_thread.h"
 
 using media::AudioBus;
 using media::AudioInputBuffer;
 using media::AudioInputBufferParameters;
+using Packet = media::AudioDeviceThread::Packet;
 
 namespace content {
 
@@ -325,8 +327,9 @@ void AudioInputSyncWriter::WriteParametersToCurrentSegment(
 }
 
 bool AudioInputSyncWriter::SignalDataWrittenAndUpdateCounters() {
-  if (socket_->Send(&current_segment_id_, sizeof(current_segment_id_)) !=
-      sizeof(current_segment_id_)) {
+  Packet packet = {current_segment_id_,
+                   base::TimeTicks::Now().ToInternalValue()};

[miu, 2016/10/23 01:19:15]

Please don't use ToInternalValue() (per http://crbug.com/634507). Use
InMicroseconds() instead.

Note: base::TimeTicks::Now().ToInternalValue() can be written equivalently as:

  (base::TimeTicks::Now() - base::TimeTicks()).InMicroseconds()

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

Done. Even though IMO in this particular case using of 'ToInternalValue' is
quite appropriate as it is used for serialization.

off topic: Might be advisable to rename 'FromInternalValue/ToInternalValue' to
'Serialize/Deserialize'

[miu, 2016/10/24 22:45:52]

Your use was appropriate here. :)

As the base/time OWNER, I'm trying to eliminate these methods because developers
are repeatedly using them inappropriately to side-step the compile-time checking
of math operations. This has already led to user-facing bugs. Instead,
developers should use In/FromMicroseconds() since the naming makes it clear that
a data type conversion is occurring (with clear semantics), and also would tend
to allow code reviewers to catch problems more easily.

I could go into more detail, but I've probably already said too much here. ;)

+  if (socket_->Send(&packet, sizeof(Packet)) != sizeof(Packet)) {
     const std::string error_message = "AISW: No room in socket buffer.";
     LOG(WARNING) << error_message;
     AddToNativeLog(error_message);