Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(130)

Side by Side Diff: cc/layer_unittest.cc

Issue 11186039: Move CC switches to cc/switches.h. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Switch ui/compositor back to using compositorSupport() Created 8 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « cc/layer_tree_host_impl_unittest.cc ('k') | cc/settings.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2011 The Chromium Authors. All rights reserved. 1 // Copyright 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "config.h" 5 #include "config.h"
6 6
7 #include "cc/layer.h" 7 #include "cc/layer.h"
8 8
9 #include "CCKeyframedAnimationCurve.h" 9 #include "CCKeyframedAnimationCurve.h"
10 #include "CCLayerImpl.h" 10 #include "CCLayerImpl.h"
11 #include "CCLayerTreeHost.h" 11 #include "CCLayerTreeHost.h"
12 #include "CCSingleThreadProxy.h" 12 #include "CCSingleThreadProxy.h"
13 #include "cc/layer_painter.h" 13 #include "cc/layer_painter.h"
14 #include "cc/test/fake_layer_tree_host_client.h" 14 #include "cc/test/fake_layer_tree_host_client.h"
15 #include "cc/test/geometry_test_utils.h" 15 #include "cc/test/geometry_test_utils.h"
16 #include "cc/test/web_compositor_initializer.h" 16 #include "cc/test/web_compositor_initializer.h"
17 #include "cc/test/test_common.h"
18 #include "cc/settings.h"
17 #include "testing/gmock/include/gmock/gmock.h" 19 #include "testing/gmock/include/gmock/gmock.h"
18 #include "testing/gtest/include/gtest/gtest.h" 20 #include "testing/gtest/include/gtest/gtest.h"
19 #include <public/WebTransformationMatrix.h> 21 #include <public/WebTransformationMatrix.h>
20 22
21 using namespace cc; 23 using namespace cc;
22 using namespace WebKitTests; 24 using namespace WebKitTests;
23 using WebKit::WebTransformationMatrix; 25 using WebKit::WebTransformationMatrix;
24 using ::testing::Mock; 26 using ::testing::Mock;
25 using ::testing::_; 27 using ::testing::_;
26 using ::testing::AtLeast; 28 using ::testing::AtLeast;
(...skipping 772 matching lines...) Expand 10 before | Expand all | Expand 10 after
799 return layer->addAnimation(animation.Pass()); 801 return layer->addAnimation(animation.Pass());
800 } 802 }
801 803
802 TEST(LayerChromiumLayerTreeHostTest, shouldNotAddAnimationWithoutLayerTreeHost) 804 TEST(LayerChromiumLayerTreeHostTest, shouldNotAddAnimationWithoutLayerTreeHost)
803 { 805 {
804 // Currently, WebCore assumes that animations will be started immediately / very soon 806 // Currently, WebCore assumes that animations will be started immediately / very soon
805 // if a composited layer's addAnimation() returns true. However, without a l ayerTreeHost, 807 // if a composited layer's addAnimation() returns true. However, without a l ayerTreeHost,
806 // layers cannot actually animate yet. So, to prevent violating this WebCore assumption, 808 // layers cannot actually animate yet. So, to prevent violating this WebCore assumption,
807 // the animation should not be accepted if the layer doesn't already have a layerTreeHost. 809 // the animation should not be accepted if the layer doesn't already have a layerTreeHost.
808 810
809 WebKit::Platform::current()->compositorSupport()->setAcceleratedAnimationEna bled(true); 811 CCScopedSettings scopedSettings;
812 Settings::setAcceleratedAnimationEnabled(true);
810 813
811 WebCompositorInitializer compositorInitializer(0); 814 WebCompositorInitializer compositorInitializer(0);
812 scoped_refptr<LayerChromium> layer = LayerChromium::create(); 815 scoped_refptr<LayerChromium> layer = LayerChromium::create();
813 816
814 // Case 1: without a layerTreeHost, the animation should not be accepted. 817 // Case 1: without a layerTreeHost, the animation should not be accepted.
815 EXPECT_FALSE(addTestAnimation(layer.get())); 818 EXPECT_FALSE(addTestAnimation(layer.get()));
816 819
817 scoped_ptr<FakeCCLayerTreeHost> layerTreeHost(FakeCCLayerTreeHost::create()) ; 820 scoped_ptr<FakeCCLayerTreeHost> layerTreeHost(FakeCCLayerTreeHost::create()) ;
818 layerTreeHost->setRootLayer(layer.get()); 821 layerTreeHost->setRootLayer(layer.get());
819 layer->setLayerTreeHost(layerTreeHost.get()); 822 layer->setLayerTreeHost(layerTreeHost.get());
(...skipping 18 matching lines...) Expand all
838 scoped_refptr<MockLayerChromium> layer(new MockLayerChromium); 841 scoped_refptr<MockLayerChromium> layer(new MockLayerChromium);
839 EXPECT_FALSE(layer->needsDisplay()); 842 EXPECT_FALSE(layer->needsDisplay());
840 layer->setBounds(IntSize(0, 10)); 843 layer->setBounds(IntSize(0, 10));
841 EXPECT_FALSE(layer->needsDisplay()); 844 EXPECT_FALSE(layer->needsDisplay());
842 layer->setBounds(IntSize(10, 10)); 845 layer->setBounds(IntSize(10, 10));
843 EXPECT_TRUE(layer->needsDisplay()); 846 EXPECT_TRUE(layer->needsDisplay());
844 } 847 }
845 848
846 849
847 } // namespace 850 } // namespace
OLDNEW
« no previous file with comments | « cc/layer_tree_host_impl_unittest.cc ('k') | cc/settings.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698