compositor_unittests target is unimplmented on Mac

ui/gfx/compositor/layer_animation_element.cc

 // Copyright (c) 2011 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 "ui/gfx/compositor/layer_animation_element.h"
 
 #include "base/compiler_specific.h"
 #include "ui/base/animation/tween.h"
 #include "ui/gfx/compositor/layer_animation_delegate.h"
 
@@ skipping 28 matching lines @@
         target_(target) {
   }
   virtual ~TransformTransition() {}
 
  protected:
   virtual void OnStart(LayerAnimationDelegate* delegate) OVERRIDE {
     start_ = delegate->GetTransformForAnimation();
   }
 
   virtual void OnProgress(double t, LayerAnimationDelegate* delegate) OVERRIDE {
+#if !defined(OS_MACOSX)

[piman, 2011/11/16 20:55:59]

Why? I don't see what's mac-adverse here.

[dhollowa, 2011/11/16 23:28:59]

ui/base/animation/tween.h:41 is #ifdef'd out on Mac.  My plan is to investigate
further and get this working in a subsequent CL.

On 2011/11/16 20:55:59, piman wrote:

     delegate->SetTransformFromAnimation(
         Tween::ValueBetween(t, start_, target_));
+#endif
   }
 
   virtual void OnGetTarget(TargetValue* target) const OVERRIDE {
     target->transform = target_;
   }
 
   virtual void OnAbort() OVERRIDE {}
 
  private:
   static const AnimatableProperties& GetProperties() {
@@ skipping 149 matching lines @@
   return new OpacityTransition(opacity, duration);
 }
 
 // static
 LayerAnimationElement* LayerAnimationElement::CreatePauseElement(
     const AnimatableProperties& properties, base::TimeDelta duration) {
   return new Pause(properties, duration);
 }
 
 }  // namespace ui