Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(766)

Side by Side Diff: media/cast/logging/encoding_event_subscriber_unittest.cc

Issue 241833002: Cast: Limit number of events/packets in EncodingEventSubscriber protos. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "base/memory/ref_counted.h" 5 #include "base/memory/ref_counted.h"
6 #include "base/memory/scoped_ptr.h" 6 #include "base/memory/scoped_ptr.h"
7 #include "base/test/simple_test_tick_clock.h" 7 #include "base/test/simple_test_tick_clock.h"
8 #include "base/time/tick_clock.h" 8 #include "base/time/tick_clock.h"
9 #include "media/cast/cast_environment.h" 9 #include "media/cast/cast_environment.h"
10 #include "media/cast/logging/encoding_event_subscriber.h" 10 #include "media/cast/logging/encoding_event_subscriber.h"
(...skipping 550 matching lines...) Expand 10 before | Expand all | Expand 10 after
561 561
562 GetEventsAndReset(); 562 GetEventsAndReset();
563 563
564 FrameEventMap::iterator it = frame_events_.find(0); 564 FrameEventMap::iterator it = frame_events_.find(0);
565 ASSERT_NE(frame_events_.end(), it); 565 ASSERT_NE(frame_events_.end(), it);
566 566
567 it = frame_events_.find(30); 567 it = frame_events_.find(30);
568 ASSERT_NE(frame_events_.end(), it); 568 ASSERT_NE(frame_events_.end(), it);
569 } 569 }
570 570
571 TEST_F(EncodingEventSubscriberTest, MaxEventsPerProto) {
572 Init(VIDEO_EVENT);
573 RtpTimestamp rtp_timestamp = 100;
574 for (int i = 0; i < kMaxEventsPerProto + 1; i++) {
575 cast_environment_->Logging()->InsertFrameEvent(testing_clock_->NowTicks(),
576 kVideoAckReceived,
577 rtp_timestamp,
578 /*frame_id*/ 0);
579 testing_clock_->Advance(base::TimeDelta::FromMilliseconds(30));
580 }
581
582 GetEventsAndReset();
583
584 FrameEventMap::iterator frame_it = frame_events_.find(0);
585 ASSERT_TRUE(frame_it != frame_events_.end());
586
587 linked_ptr<AggregatedFrameEvent> frame_event = frame_it->second;
588
589 EXPECT_EQ(kMaxEventsPerProto, frame_event->event_type_size());
590
591 for (int i = 0; i < kMaxPacketsPerFrame + 1; i++) {
592 for (int j = 0; j < kMaxEventsPerProto + 1; j++) {
593 cast_environment_->Logging()->InsertPacketEvent(
594 testing_clock_->NowTicks(),
595 kVideoPacketRetransmitted,
596 rtp_timestamp,
597 /*frame_id*/ 0,
598 i,
599 kMaxPacketsPerFrame,
600 123);
601 testing_clock_->Advance(base::TimeDelta::FromMilliseconds(30));
602 }
603 }
604
605 GetEventsAndReset();
606
607 PacketEventMap::iterator packet_it = packet_events_.find(0);
608 ASSERT_TRUE(packet_it != packet_events_.end());
609
610 linked_ptr<AggregatedPacketEvent> packet_event = packet_it->second;
611
612 EXPECT_EQ(kMaxPacketsPerFrame, packet_event->base_packet_event_size());
613 for (int i = 0; i < packet_event->base_packet_event_size(); i++) {
614 EXPECT_EQ(kMaxEventsPerProto,
615 packet_event->base_packet_event(i).event_type_size());
616 }
617 }
618
571 } // namespace cast 619 } // namespace cast
572 } // namespace media 620 } // namespace media
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698