[gtk] Add fade-commit to instant suggestion for predictive instant.

app/multi_animation.cc

 // Copyright (c) 2010 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 "app/multi_animation.h"
 
 #include "app/animation_delegate.h"
 #include "base/logging.h"
 
 // Default interval, in ms.
@@ skipping 13 matching lines @@
       parts_(parts),
       cycle_time_ms_(TotalTime(parts)),
       current_value_(0),
       current_part_index_(0),
       continuous_(true) {
   DCHECK(!parts_.empty());
 }
 
 MultiAnimation::~MultiAnimation() {}
 
+void MultiAnimation::Reset() {
+  Stop();
+  current_value_ = 0;

[sky, 2010/12/06 17:38:12]

current_value and current_parent_index should be set in SetStartTime.
SetStartTime should probably be renamed to indicate its the place for this sort
of resetting. Don't feel you need to rename SetStartTime as part of this patch.

[Evan Stade, 2010/12/06 22:15:48]

I think I've done what you intended

+  current_part_index_ = 0;
+}
+
 void MultiAnimation::Step(base::TimeTicks time_now) {
   double last_value = current_value_;
   size_t last_index = current_part_index_;
 
   int delta = static_cast<int>((time_now - start_time()).InMilliseconds());
   if (delta >= cycle_time_ms_ && !continuous_) {
     current_part_index_ = parts_.size() - 1;
     current_value_ = Tween::CalculateValue(parts_[current_part_index_].type, 1);
     Stop();
     return;
@@ skipping 21 matching lines @@
       return parts_[i];
     }
 
     *time_ms -= parts_[i].time_ms;
   }
   NOTREACHED();
   *time_ms = 0;
   *part_index = 0;
   return parts_[0];
 }