Don't animate the default/background layer if there is no background.

chrome/browser/chromeos/login/base_login_display_host.cc

 // Copyright (c) 2012 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 "chrome/browser/chromeos/login/base_login_display_host.h"
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/file_util.h"
 #include "base/logging.h"
@@ skipping 311 matching lines @@
   // Fade out transition for sign in screen.
   // BaseLoginDisplayHost will be deleted this animation is ended.
   ui::Layer* layer = GetLayer(GetWidget());
   layer->GetAnimator()->AddObserver(this);
   ui::LayerAnimator::ScopedSettings signin_animation(layer->GetAnimator());
   signin_animation.SetTransitionDuration(
       base::TimeDelta::FromMilliseconds(kLoginFadeoutTransitionDurationMs));
   layer->SetOpacity(0.0f);
 
   // Background animation.
   if (kEnableBackgroundAnimation) {

[James Cook, 2012/01/13 00:42:12]

Should this also check IsWindowModeCompact()?  Compact window mode doesn't have
a background to animate either.  (This probably works right now because
kEnableBackgroundAnimation is always false.)

     ui::Transform background_transform;
     background_transform.SetScale(kBackgroundScale, kBackgroundScale);
     background_transform.SetTranslateX(kBackgroundTranslate);
     background_transform.SetTranslateY(kBackgroundTranslate);
     scoped_ptr<ui::LayerAnimationElement>
         background_transform_animation_initial(
             ui::LayerAnimationElement::CreateTransformElement(
                 background_transform,
                 base::TimeDelta()));
     ui::LayerAnimationElement::AnimatableProperties background_pause_properties;
@@ skipping 13 matching lines @@
     background_transition->AddElement(background_pause.release());
     background_transition->AddElement(background_transform_animation.release());
     ui::Layer* background_layer =
         ash::Shell::GetInstance()->GetContainer(
             ash::internal::kShellWindowId_DesktopBackgroundContainer)->
             layer();
     background_layer->GetAnimator()->StartAnimation(
         background_transition.release());
   }
 
-  // Browser windows layer opacity animation.
-  if (kEnableBrowserWindowsOpacityAnimation) {
+  // Browser windows layer opacity animation.  Don't fade in the
+  // browser in compact mode because there is nothing behind it.
+  if (kEnableBrowserWindowsOpacityAnimation &&
+      !ash::Shell::GetInstance()->IsWindowModeCompact()) {
     scoped_ptr<ui::LayerAnimationElement> browser_opacity_animation_initial(
         ui::LayerAnimationElement::CreateOpacityElement(
             0.0f,
             base::TimeDelta()));
     ui::LayerAnimationElement::AnimatableProperties browser_pause_properties;
     browser_pause_properties.insert(ui::LayerAnimationElement::OPACITY);
     scoped_ptr<ui::LayerAnimationElement> browser_pause_animation(
         ui::LayerAnimationElement::CreatePauseElement(
             browser_pause_properties,
             base::TimeDelta::FromMilliseconds(kBrowserTransitionPauseMs)));
@@ skipping 208 matching lines @@
   // user has changed to during OOBE.
   if (!timezone_name.empty()) {
     icu::TimeZone* timezone = icu::TimeZone::createTimeZone(
         icu::UnicodeString::fromUTF8(timezone_name));
     CHECK(timezone) << "Timezone could not be set for " << timezone_name;
     chromeos::system::TimezoneSettings::GetInstance()->SetTimezone(*timezone);
   }
 }
 
 }  // namespace browser