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

Unified Diff: ash/laser/laser_pointer_points_test_api.cc

Issue 2362063002: cros: Laser pointer fades out on release, do not cover palette. (Closed)
Patch Set: Created 4 years, 3 months 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 side-by-side diff with in-line comments
Download patch
Index: ash/laser/laser_pointer_points_test_api.cc
diff --git a/ash/laser/laser_pointer_points_test_api.cc b/ash/laser/laser_pointer_points_test_api.cc
index 5180731128d1e4e18a4c0622ffcd3b7c241b252f..3fe27d4895356bb1399d3a4e5bdff2e27b656906 100644
--- a/ash/laser/laser_pointer_points_test_api.cc
+++ b/ash/laser/laser_pointer_points_test_api.cc
@@ -10,7 +10,7 @@ namespace ash {
LaserPointerPointsTestApi::LaserPointerPointsTestApi(
LaserPointerPoints* instance)
- : new_point_time_(base::Time::Now()), instance_(instance) {}
+ : instance_(instance) {}
LaserPointerPointsTestApi::~LaserPointerPointsTestApi() {}
@@ -20,12 +20,12 @@ int LaserPointerPointsTestApi::GetNumberOfPoints() const {
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.creation_time = new_time;
instance_->points_.push_back(new_point);
+ instance_->collection_latest_time_ = new_time;
jdufault 2016/09/23 23:59:20 Should this be calling SetLatestTime(new_time)?
sammiequon 2016/09/26 19:30:38 Done.
instance_->ClearOldPoints();
}
} // namespace ash

Powered by Google App Engine
This is Rietveld 408576698