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

Side by Side Diff: sky/engine/core/animation/AnimationTimelineTest.cpp

Issue 772673002: Fix Animations, Remove Compostior Animations. (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: CompositorPendingAnimations -> PendingAnimations Created 6 years 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
OLDNEW
1 /* 1 /*
2 * Copyright (c) 2013, Google Inc. All rights reserved. 2 * Copyright (c) 2013, Google Inc. All rights reserved.
3 * 3 *
4 * Redistribution and use in source and binary forms, with or without 4 * Redistribution and use in source and binary forms, with or without
5 * modification, are permitted provided that the following conditions are 5 * modification, are permitted provided that the following conditions are
6 * met: 6 * met:
7 * 7 *
8 * * Redistributions of source code must retain the above copyright 8 * * Redistributions of source code must retain the above copyright
9 * notice, this list of conditions and the following disclaimer. 9 * notice, this list of conditions and the following disclaimer.
10 * * Redistributions in binary form must reproduce the above 10 * * Redistributions in binary form must reproduce the above
(...skipping 83 matching lines...) Expand 10 before | Expand all | Expand 10 after
94 virtual void TearDown() 94 virtual void TearDown()
95 { 95 {
96 document.release(); 96 document.release();
97 element.release(); 97 element.release();
98 timeline.release(); 98 timeline.release();
99 } 99 }
100 100
101 void updateClockAndService(double time) 101 void updateClockAndService(double time)
102 { 102 {
103 document->animationClock().updateTime(time); 103 document->animationClock().updateTime(time);
104 document->compositorPendingAnimations().update(false); 104 document->pendingAnimations().update(false);
105 timeline->serviceAnimations(TimingUpdateForAnimationFrame); 105 timeline->serviceAnimations(TimingUpdateForAnimationFrame);
106 } 106 }
107 107
108 RefPtr<Document> document; 108 RefPtr<Document> document;
109 RefPtr<Element> element; 109 RefPtr<Element> element;
110 RefPtr<AnimationTimeline> timeline; 110 RefPtr<AnimationTimeline> timeline;
111 Timing timing; 111 Timing timing;
112 MockPlatformTiming* platformTiming; 112 MockPlatformTiming* platformTiming;
113 113
114 void wake() 114 void wake()
(...skipping 116 matching lines...) Expand 10 before | Expand all | Expand 10 after
231 231
232 TEST_F(AnimationAnimationTimelineTest, UseAnimationPlayerAfterTimelineDeref) 232 TEST_F(AnimationAnimationTimelineTest, UseAnimationPlayerAfterTimelineDeref)
233 { 233 {
234 RefPtr<AnimationPlayer> player = timeline->createAnimationPlayer(0); 234 RefPtr<AnimationPlayer> player = timeline->createAnimationPlayer(0);
235 timeline.clear(); 235 timeline.clear();
236 // Test passes if this does not crash. 236 // Test passes if this does not crash.
237 player->setStartTime(0); 237 player->setStartTime(0);
238 } 238 }
239 239
240 } 240 }
OLDNEW
« no previous file with comments | « sky/engine/core/animation/AnimationStackTest.cpp ('k') | sky/engine/core/animation/CompositorAnimations.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698