Cast: Fix encoding event subscriber log packet size.

media/cast/logging/encoding_event_subscriber.cc

 // Copyright 2014 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.
 
 #include "media/cast/logging/encoding_event_subscriber.h"
 
 #include <cstring>
 #include <utility>
 
 #include "base/logging.h"
@@ skipping 160 matching lines @@
       base_packet_event_proto = GetNewBasePacketEvent(
           event_proto.get(), packet_event.packet_id, packet_event.size);
     }
   }
 
   base_packet_event_proto->add_event_type(
       ToProtoEventType(packet_event.type));
   base_packet_event_proto->add_event_timestamp_ms(
       (packet_event.timestamp - base::TimeTicks()).InMilliseconds());
 
+  // |base_packet_event_proto| could have been created with a receiver event
+  // which does not have the packet size and we would need to overwrite it when
+  // we see a sender event, which does have the packet size.
+  if (packet_event.size > 0) {
+    base_packet_event_proto->set_size(packet_event.size);
+  }
+
   if (packet_event_map_.size() > kMaxMapSize)
     TransferPacketEvents(kNumMapEntriesToTransfer);
 
   DCHECK(packet_event_map_.size() <= kMaxMapSize);
   DCHECK(packet_event_storage_.size() <= max_frames_);
 }
 
 void EncodingEventSubscriber::GetEventsAndReset(LogMetadata* metadata,
     FrameEventList* frame_events, PacketEventList* packet_events) {
   DCHECK(thread_checker_.CalledOnValidThread());
@@ skipping 82 matching lines @@
   frame_event_storage_index_ = 0;
   packet_event_map_.clear();
   packet_event_storage_.clear();
   packet_event_storage_index_ = 0;
   seen_first_rtp_timestamp_ = false;
   first_rtp_timestamp_ = 0u;
 }
 
 }  // namespace cast
 }  // namespace media