From 387363c426e097c358bd41367e0a8f4339fc8dc3 Mon Sep 17 00:00:00 2001
From: Chris Wren <cwren@android.com>
Date: Wed, 17 Oct 2012 16:04:15 -0400
Subject: tune re-deal behavior on photo table.

Bug: 7669353
Change-Id: I17aaf289067cbd688dba5cd317541b98bb4ee3dd
---
 res/values-sw600dp/config.xml                     |  3 +++
 res/values-sw800dp/config.xml                     |  4 ++--
 res/values/config.xml                             |  4 ++--
 src/com/android/dreams/phototable/PhotoTable.java | 23 +++++++++--------------
 4 files changed, 16 insertions(+), 18 deletions(-)

diff --git a/res/values-sw600dp/config.xml b/res/values-sw600dp/config.xml
index 4a8a67f..e1ad849 100644
--- a/res/values-sw600dp/config.xml
+++ b/res/values-sw600dp/config.xml
@@ -16,5 +16,8 @@
 <resources>
   <!-- Parts per million ratio between image size and screen size. -->
   <integer name="image_ratio">750000</integer>
+
+  <!-- Parts per million ratio between image size on the table and screen size. -->
+  <integer name="table_ratio">300000</integer>
 </resources>
 
diff --git a/res/values-sw800dp/config.xml b/res/values-sw800dp/config.xml
index eeefc3a..699e15a 100644
--- a/res/values-sw800dp/config.xml
+++ b/res/values-sw800dp/config.xml
@@ -15,10 +15,10 @@
 -->
 <resources>
   <!-- Milliseconds between drops. -->
-  <integer name="table_drop_period">90000</integer>
+  <integer name="table_drop_period">75000</integer>
 
   <!-- Milliseconds to wait before the next fast drop.-->
-  <integer name="fast_drop">10000</integer>
+  <integer name="fast_drop">5000</integer>
 
   <!-- Parts per million ratio between image size and screen size. -->
   <integer name="image_ratio">500000</integer>
diff --git a/res/values/config.xml b/res/values/config.xml
index d8d745d..36835c9 100644
--- a/res/values/config.xml
+++ b/res/values/config.xml
@@ -18,7 +18,7 @@
   <integer name="table_drop_period">45000</integer>
 
   <!-- Milliseconds to wait before the next fast drop.-->
-  <integer name="fast_drop">4000</integer>
+  <integer name="fast_drop">3000</integer>
 
   <!-- Milliseconds between drops. -->
   <integer name="carousel_drop_period">10000</integer>
@@ -33,7 +33,7 @@
   <integer name="table_capacity">10</integer>
 
   <!-- Number of images to discard at a time.-->
-  <integer name="redeal_count">3</integer>
+  <integer name="redeal_count">5</integer>
 
   <!-- Parts per million ratio between image size and screen size. -->
   <integer name="image_ratio">500000</integer>
diff --git a/src/com/android/dreams/phototable/PhotoTable.java b/src/com/android/dreams/phototable/PhotoTable.java
index 50212c9..36b274c 100644
--- a/src/com/android/dreams/phototable/PhotoTable.java
+++ b/src/com/android/dreams/phototable/PhotoTable.java
@@ -119,7 +119,7 @@ public class PhotoTable extends FrameLayout {
         mStarted = false;
     }
 
-    
+
     public void setDream(DreamService dream) {
         mDream = dream;
     }
@@ -297,6 +297,13 @@ public class PhotoTable extends FrameLayout {
                 log("drop it");
                 table.throwOnTable(photo);
 
+                if (mOnTable.size() > mTableCapacity) {
+                    int targetSize = Math.max(0, mOnTable.size() - mRedealCount);
+                    while (mOnTable.size() > targetSize) {
+                        fadeAway(mOnTable.poll(), false);
+                    }
+                }
+
                 if(table.mOnTable.size() < table.mTableCapacity) {
                     table.scheduleNext(table.mFastDropPeriod);
                 }
@@ -399,19 +406,7 @@ public class PhotoTable extends FrameLayout {
                 .x(x)
                 .y(y)
                 .setDuration(duration)
-                .setInterpolator(interpolator)
-                .withEndAction(new Runnable() {
-                        @Override
-                            public void run() {
-                            if (mOnTable.size() > mTableCapacity) {
-                                while (mOnTable.size() > (mTableCapacity - mRedealCount)) {
-                                    fadeAway(mOnTable.poll(), false);
-                                }
-                                // zero delay because we already waited duration ms
-                                scheduleNext(0);
-                            }
-                        }
-                    });
+                .setInterpolator(interpolator);
     }
 
     /** wrap all orientations to the interval [-180, 180). */
-- 
cgit v1.2.3