Bug 827407 - Back out Flash fullscreen video bug 809055 to test whether FlashPaintSurface crashes are correlated. r=snorp a=lsblakk
authorChris Peterson <cpeterson@mozilla.com>
Tue, 11 Jun 2013 10:09:57 -0700
changeset 143008 f863495cdd416c6db1ce8216986a1d709d43b89c
parent 143003 480f6c51f45a191d71709ab3cb7c453eaff4d6ce
child 143009 e0540178e3106e79c4a8ba09245d0484f7f384e5
push id2602
push usercpeterson@mozilla.com
push dateMon, 01 Jul 2013 22:53:27 +0000
treeherdermozilla-beta@f863495cdd41 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerssnorp, lsblakk
bugs827407, 809055
milestone23.0
Bug 827407 - Back out Flash fullscreen video bug 809055 to test whether FlashPaintSurface crashes are correlated. r=snorp a=lsblakk
dom/plugins/base/nsPluginInstanceOwner.cpp
--- a/dom/plugins/base/nsPluginInstanceOwner.cpp
+++ b/dom/plugins/base/nsPluginInstanceOwner.cpp
@@ -3293,18 +3293,16 @@ nsPluginInstanceOwner::UpdateDocumentAct
 {
   mPluginDocumentActiveState = aIsActive;
   UpdateWindowPositionAndClipRect(true);
 
 #ifdef MOZ_WIDGET_ANDROID
   if (mInstance) {
     if (!mPluginDocumentActiveState)
       RemovePluginView();
-    else if (mPluginDocumentActiveState && mFullScreen)
-      AddPluginView();
 
     mInstance->NotifyOnScreen(mPluginDocumentActiveState);
 
     // This is, perhaps, incorrect. It is supposed to be sent
     // when "the webview has paused or resumed". The side effect
     // is that Flash video players pause or resume (if they were
     // playing before) based on the value here. I personally think
     // we want that on Android when switching to another tab, so