Fix failed capture handling in WebrtcVideoStream

remoting/protocol/connection_unittest.cc

 // Copyright 2015 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.
 
 #define _USE_MATH_DEFINES // For VC++ to get M_PI. This has to be first.
 
 #include <utility>
 
 #include "base/bind.h"
 #include "base/macros.h"
@@ skipping 617 matching lines @@
   Connect();
 
   std::unique_ptr<AudioStream> audio_stream =
       host_connection_->StartAudioStream(base::MakeUnique<TestAudioSource>());
 
   // Wait for 1 second worth of audio samples.
   client_audio_player_.WaitForSamples(kAudioSampleRate * 2);
   client_audio_player_.Verify();
 }
 
-#if !defined(MEMORY_SANITIZER)
-// Test fails under msan, https://crbug.com/697178
 TEST_P(ConnectionTest, FirstCaptureFailed) {
   Connect();
 
   base::TimeTicks event_timestamp = base::TimeTicks::FromInternalValue(42);
 
   scoped_refptr<InputEventTimestampsSourceImpl> input_event_timestamps_source =
       new InputEventTimestampsSourceImpl();
   input_event_timestamps_source->OnEventReceived(
       InputEventTimestamps{event_timestamp, base::TimeTicks::Now()});
 
@@ skipping 12 matching lines @@
     WaitFirstFrameStats();
 
     // Verify that the event timestamp received before the first frame gets used
     // for the second frame.
     const FrameStats& stats = client_video_renderer_.GetFrameStatsConsumer()
                                   ->received_stats()
                                   .front();
     EXPECT_EQ(event_timestamp, stats.host_stats.latest_event_timestamp);
   }
 }
-#endif
 
 TEST_P(ConnectionTest, SecondCaptureFailed) {
   Connect();
 
   auto capturer = base::MakeUnique<TestScreenCapturer>();
   capturer->FailNthFrame(1);
   auto video_stream = host_connection_->StartVideoStream(std::move(capturer));
 
   WaitNextVideoFrame();
   WaitNextVideoFrame();
 }
 
 }  // namespace protocol
 }  // namespace remoting