Bug 1146561 - Add back the visibility check for Mac plugins. r=jmuizelaar, a=sledru
authorJet Villegas <bugs@junglecode.net>
Fri, 24 Apr 2015 17:44:00 -0400
changeset 267351 8d0f31b22060d436b9946800f025807f2780a440
parent 267350 836f6b1688468729b7da9225a39cb058a37e41e9
child 267352 0b0ea2bcdc8473876b56ca1334cdf8aa9184ecdf
push id830
push userraliiev@mozilla.com
push dateFri, 19 Jun 2015 19:24:37 +0000
treeherdermozilla-release@932614382a68 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersjmuizelaar, sledru
bugs1146561
milestone39.0a2
Bug 1146561 - Add back the visibility check for Mac plugins. r=jmuizelaar, a=sledru
layout/generic/nsPluginFrame.cpp
--- a/layout/generic/nsPluginFrame.cpp
+++ b/layout/generic/nsPluginFrame.cpp
@@ -1030,17 +1030,17 @@ nsPluginFrame::NotifyPluginReflowObserve
 {
   nsContentUtils::AddScriptRunner(new PluginEventNotifier(NS_LITERAL_STRING("reflow")));
 }
 
 void
 nsPluginFrame::DidSetWidgetGeometry()
 {
 #if defined(XP_MACOSX)
-  if (mInstanceOwner) {
+  if (mInstanceOwner && !IsHidden()) {
     mInstanceOwner->FixUpPluginWindow(nsPluginInstanceOwner::ePluginPaintEnable);
   }
 #else
   if (!mWidget && mInstanceOwner) {
     // UpdateWindowVisibility will notify the plugin of position changes
     // by updating the NPWindow and calling NPP_SetWindow/AsyncSetWindow.
     // We treat windowless plugins inside popups as always visible, since
     // plugins inside popups don't get valid mNextConfigurationBounds