MediaRecorder: handle libwebm::Segment::AddFrame() errors and wire up to Blink

content/renderer/media/media_recorder_handler.cc

Index: content/renderer/media/media_recorder_handler.cc
diff --git a/content/renderer/media/media_recorder_handler.cc b/content/renderer/media/media_recorder_handler.cc
index cea6710b159b6332dcdd7feea700c524ba27d134..9fde39595793b388853cc6d4f8143ebd2a35a7ea 100644
--- a/content/renderer/media/media_recorder_handler.cc
+++ b/content/renderer/media/media_recorder_handler.cc
@@ -266,8 +266,11 @@ void MediaRecorderHandler::OnEncodedVideo(
   DCHECK(main_render_thread_checker_.CalledOnValidThread());
   if (!webm_muxer_)
     return;
-  webm_muxer_->OnEncodedVideo(params, std::move(encoded_data), timestamp,
-                              is_key_frame);
+  if (!webm_muxer_->OnEncodedVideo(params, std::move(encoded_data), timestamp,
+                                   is_key_frame)) {
+    DLOG(ERROR) << "Error muxing video data";
+    client_->onError("Error muxing video data");
+  }
 }
 
 void MediaRecorderHandler::OnEncodedAudio(
@@ -275,8 +278,13 @@ void MediaRecorderHandler::OnEncodedAudio(
     std::unique_ptr<std::string> encoded_data,
     base::TimeTicks timestamp) {
   DCHECK(main_render_thread_checker_.CalledOnValidThread());
-  if (webm_muxer_)
-    webm_muxer_->OnEncodedAudio(params, std::move(encoded_data), timestamp);
+  if (!webm_muxer_)
+    return;
+  if (!webm_muxer_->OnEncodedAudio(params, std::move(encoded_data),
+                                   timestamp)) {
+    DLOG(ERROR) << "Error muxing audio data";
+    client_->onError("Error muxing audio data");
+  }
 }
 
 void MediaRecorderHandler::WriteData(base::StringPiece data) {