Skip to content

Commit a36b7ab

Browse files
Jeff BrownAndroid (Google) Code Review
authored andcommitted
Merge "Work around build break in layoutlib." into jb-dev
2 parents ffd6ea4 + 96ff709 commit a36b7ab

File tree

1 file changed

+7
-5
lines changed

1 file changed

+7
-5
lines changed

tools/layoutlib/bridge/src/android/animation/AnimationThread.java

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -84,20 +84,22 @@ public AnimationThread(RenderSessionImpl scene, String threadName,
8484
public void run() {
8585
Bridge.prepareThread();
8686
try {
87+
/* FIXME: The ANIMATION_FRAME message no longer exists. Instead, the
88+
* animation timing loop is completely based on a Choreographer objects
89+
* that schedules animation and drawing frames. The animation handler is
90+
* no longer even a handler; it is just a Runnable enqueued on the Choreographer.
8791
Handler_Delegate.setCallback(new IHandlerCallback() {
8892
@Override
8993
public void sendMessageAtTime(Handler handler, Message msg, long uptimeMillis) {
90-
if (msg.what == ValueAnimator.ANIMATION_START /*||
91-
FIXME: The ANIMATION_FRAME message no longer exists. Instead,
92-
the animation timing loop is based on a Choreographer object
93-
that schedules animation and drawing frames.
94-
msg.what == ValueAnimator.ANIMATION_FRAME*/) {
94+
if (msg.what == ValueAnimator.ANIMATION_START ||
95+
msg.what == ValueAnimator.ANIMATION_FRAME) {
9596
mQueue.add(new MessageBundle(handler, msg, uptimeMillis));
9697
} else {
9798
// just ignore.
9899
}
99100
}
100101
});
102+
*/
101103

102104
// call out to the pre-animation work, which should start an animation or more.
103105
Result result = preAnimation();

0 commit comments

Comments
 (0)