cros: Laser pointer fades out on release, do not cover palette.

ash/laser/laser_pointer_points_test_api.cc

 // Copyright 2016 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 "ash/laser/laser_pointer_points_test_api.h"
 
-#include "ash/laser/laser_pointer_points.h"
-
 namespace ash {
 
 LaserPointerPointsTestApi::LaserPointerPointsTestApi(
     LaserPointerPoints* instance)
-    : new_point_time_(base::Time::Now()), instance_(instance) {}
+    : instance_(instance) {}
 
 LaserPointerPointsTestApi::~LaserPointerPointsTestApi() {}
 
 int LaserPointerPointsTestApi::GetNumberOfPoints() const {
   return instance_->GetNumberOfPoints();
 }
 
 void LaserPointerPointsTestApi::MoveForwardInTime(
     const base::TimeDelta& delta) {
-  for (LaserPointerPoints::LaserPoint& point : instance_->points_)
-    point.creation_time -= delta;
+  base::Time new_time = instance_->collection_latest_time_ + delta;
 
   LaserPointerPoints::LaserPoint new_point;
-  new_point.creation_time = new_point_time_;
+  new_point.age = 0.0;
+  instance_->MoveForwardToTime(new_time);

[jdufault, 2016/10/05 21:36:05]

Keep new_point initialization together; move the MoveForwardToTime call above
the new_point declaration.

[sammiequon, 2016/10/06 00:19:16]

Done.

   instance_->points_.push_back(new_point);
-  instance_->ClearOldPoints();
 }
+
+LaserPointerPoints::LaserPoint LaserPointerPointsTestApi::GetPointAtIndex(
+    int index) {
+  DCHECK(index >= GetNumberOfPoints());

[jdufault, 2016/10/05 21:36:05]

Is this DCHECK right? It looks like it is inverted. Make sure you build chrome
with DCHECK enabled.

  DHCECK(index < GetNumberOfPoints())

[sammiequon, 2016/10/06 00:19:15]

Done.

+  return instance_->points_[index];
+}
+
 }  // namespace ash