Cast: Make vp8 3-buffer mode work

media/cast/sender/video_encoder_impl_unittest.cc

Index: media/cast/sender/video_encoder_impl_unittest.cc
diff --git a/media/cast/sender/video_encoder_impl_unittest.cc b/media/cast/sender/video_encoder_impl_unittest.cc
index 010798ed676864b507693ff26c8642067eb78eb0..b4c54540e602b38bb4ef8f822256b115aa820781 100644
--- a/media/cast/sender/video_encoder_impl_unittest.cc
+++ b/media/cast/sender/video_encoder_impl_unittest.cc
@@ -36,15 +36,14 @@ class TestVideoEncoderCallback
 
   void DeliverEncodedVideoFrame(
       scoped_ptr<EncodedFrame> encoded_frame) {
-    if (expected_frame_id_ == expected_last_referenced_frame_id_) {
-      EXPECT_EQ(EncodedFrame::KEY, encoded_frame->dependency);
-    } else {
+    if (expected_frame_id_ != expected_last_referenced_frame_id_) {

[miu, 2014/09/02 19:22:03]

If this assumption is no longer true, should we be introducing a new
EncodedFrame enum?

[hubbe, 2014/09/02 21:39:48]

I think I'm going to try to make the dependency into a bool again. (In a
separate CL.)

       EXPECT_EQ(EncodedFrame::DEPENDENT,
                 encoded_frame->dependency);
     }
     EXPECT_EQ(expected_frame_id_, encoded_frame->frame_id);
     EXPECT_EQ(expected_last_referenced_frame_id_,
-              encoded_frame->referenced_frame_id);
+              encoded_frame->referenced_frame_id)
+        << "frame id: " << expected_frame_id_;
     EXPECT_EQ(expected_capture_time_, encoded_frame->reference_time);
   }