[Video] Fix DCHECK crash in PauseVideoIfHidden

media/blink/webmediaplayer_impl.cc

 // Copyright 2013 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/blink/webmediaplayer_impl.h"
 
 #include <algorithm>
 #include <cmath>
 #include <limits>
 #include <string>
@@ skipping 1387 matching lines @@
   if (video_weblayer_)
     video_weblayer_->layer()->SetContentsOpaque(opaque_);
 }
 
 void WebMediaPlayerImpl::OnFrameHidden() {
   DCHECK(main_task_runner_->BelongsToCurrentThread());
 
   if (watch_time_reporter_)
     watch_time_reporter_->OnHidden();
 
-  if (ShouldPauseVideoWhenHidden()) {
-    PauseVideoIfNeeded();
-    return;
-  } else {
-    DisableVideoTrackIfNeeded();
+  if (!delegate_->IsFrameClosed()) {

[sandersd (OOO until July 31), 2017/01/24 19:51:21]

Perhaps just call IsHidden()?

Also, rename IsHidden() to IsBackgrounded() to reduce confusion?

+    if (ShouldPauseVideoWhenHidden()) {
+      PauseVideoIfNeeded();
+      return;
+    } else {
+      DisableVideoTrackIfNeeded();
+    }
   }
 
   UpdatePlayState();
 
   // Schedule suspended playing media to be paused if the user doesn't come back
   // to it within some timeout period to avoid any autoplay surprises.
   ScheduleIdlePauseTimer();
 }
 
 void WebMediaPlayerImpl::OnFrameClosed() {
@@ skipping 830 matching lines @@
     UMA_HISTOGRAM_TIMES(
         "Media.Video.TimeFromForegroundToFirstFrame.DisableTrack",
         time_to_first_frame);
   } else {
     UMA_HISTOGRAM_TIMES("Media.Video.TimeFromForegroundToFirstFrame.Paused",
                         time_to_first_frame);
   }
 }
 
 }  // namespace media