Fixing invalid visibility states with layers (opacity==0, visibility==true)

ash/wm/system_modal_container_layout_manager.cc

Index: ash/wm/system_modal_container_layout_manager.cc
diff --git a/ash/wm/system_modal_container_layout_manager.cc b/ash/wm/system_modal_container_layout_manager.cc
index faba106d863eaed627d13ad62fc8440a61badbba..a1432c6b9cfb3acfe3075d068d70895a725f87fa 100644
--- a/ash/wm/system_modal_container_layout_manager.cc
+++ b/ash/wm/system_modal_container_layout_manager.cc
@@ -165,8 +165,10 @@ void SystemModalContainerLayoutManager::CreateModalBackground() {
 
   ui::ScopedLayerAnimationSettings settings(
       modal_background_->GetNativeView()->layer()->GetAnimator());
-  modal_background_->Show();
+  // Show should not be called with a target opacity of 0. We therefore start
+  // the fade to show animation before Show() is called.
   modal_background_->GetNativeView()->layer()->SetOpacity(0.5f);
+  modal_background_->Show();
   container_->StackChildAtTop(modal_background_->GetNativeView());
 }