Make media controls not depend on the webkitfullscreenchange event

third_party/WebKit/Source/core/html/HTMLMediaElement.cpp

 /*
  * Copyright (C) 2007, 2008, 2009, 2010, 2011, 2012, 2013 Apple Inc. All rights
  * reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 3371 matching lines @@
   Fullscreen::requestFullscreen(*this, Fullscreen::PrefixedRequest);
 }
 
 void HTMLMediaElement::exitFullscreen() {
   BLINK_MEDIA_LOG << "exitFullscreen(" << (void*)this << ")";
 
   Fullscreen::exitFullscreen(document());
 }
 
 void HTMLMediaElement::didEnterFullscreen() {
+  configureMediaControls();
   if (mediaControls())
     mediaControls()->enteredFullscreen();
   // FIXME: There is no embedder-side handling in layout test mode.
   if (webMediaPlayer() && !LayoutTestSupport::isRunningLayoutTest())
     webMediaPlayer()->enteredFullscreen();
   // Cache this in case the player is destroyed before leaving fullscreen.
   m_inOverlayFullscreenVideo = usesOverlayFullscreenVideo();
   if (m_inOverlayFullscreenVideo)
     document().layoutViewItem().compositor()->setNeedsCompositingUpdate(
         CompositingUpdateRebuildTree);
 }
 
 void HTMLMediaElement::didExitFullscreen() {
+  configureMediaControls();
   if (mediaControls())
     mediaControls()->exitedFullscreen();
   if (webMediaPlayer())
     webMediaPlayer()->exitedFullscreen();
   if (m_inOverlayFullscreenVideo)
     document().layoutViewItem().compositor()->setNeedsCompositingUpdate(
         CompositingUpdateRebuildTree);
   m_inOverlayFullscreenVideo = false;
 }
 
@@ skipping 220 matching lines @@
 
   // Note: The "time marches on" algorithm (updateActiveCues) runs the "rules
   // for updating the text track rendering" (updateTextTrackDisplay) only for
   // "affected tracks", i.e. tracks where the the active cues have changed.
   // This misses cues in tracks that changed mode between hidden and showing.
   // This appears to be a spec bug, which we work around here:
   // https://www.w3.org/Bugs/Public/show_bug.cgi?id=28236
   updateTextTrackDisplay();
 }
 
-EventDispatchHandlingState* HTMLMediaElement::preDispatchEventHandler(
-    Event* event) {
-  if (event && event->type() == EventTypeNames::webkitfullscreenchange)
-    configureMediaControls();
-
-  return nullptr;
-}
-
 // TODO(srirama.m): Merge it to resetMediaElement if possible and remove it.
 void HTMLMediaElement::resetMediaPlayerAndMediaSource() {
   closeMediaSource();
 
   {
     AudioSourceProviderClientLockScope scope(*this);
     clearMediaPlayerAndAudioSourceProviderClientWithoutLocking();
   }
 
   // We haven't yet found out if any remote routes are available.
@@ skipping 398 matching lines @@
       kMostlyFillViewportBecomeStableSeconds, BLINK_FROM_HERE);
 }
 
 void HTMLMediaElement::viewportFillDebouncerTimerFired(TimerBase*) {
   m_mostlyFillingViewport = true;
   if (m_webMediaPlayer)
     m_webMediaPlayer->becameDominantVisibleContent(m_mostlyFillingViewport);
 }
 
 }  // namespace blink