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

Side by Side 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 unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "ash/laser/laser_pointer_points_test_api.h" 5 #include "ash/laser/laser_pointer_points_test_api.h"
6 6
7 #include "ash/laser/laser_pointer_points.h" 7 #include "ash/laser/laser_pointer_points.h"
8 8
9 namespace ash { 9 namespace ash {
10 10
11 LaserPointerPointsTestApi::LaserPointerPointsTestApi( 11 LaserPointerPointsTestApi::LaserPointerPointsTestApi(
12 LaserPointerPoints* instance) 12 LaserPointerPoints* instance)
13 : new_point_time_(base::Time::Now()), instance_(instance) {} 13 : instance_(instance) {}
14 14
15 LaserPointerPointsTestApi::~LaserPointerPointsTestApi() {} 15 LaserPointerPointsTestApi::~LaserPointerPointsTestApi() {}
16 16
17 int LaserPointerPointsTestApi::GetNumberOfPoints() const { 17 int LaserPointerPointsTestApi::GetNumberOfPoints() const {
18 return instance_->GetNumberOfPoints(); 18 return instance_->GetNumberOfPoints();
19 } 19 }
20 20
21 void LaserPointerPointsTestApi::MoveForwardInTime( 21 void LaserPointerPointsTestApi::MoveForwardInTime(
22 const base::TimeDelta& delta) { 22 const base::TimeDelta& delta) {
23 for (LaserPointerPoints::LaserPoint& point : instance_->points_) 23 base::Time new_time = instance_->collection_latest_time_ + delta;
24 point.creation_time -= delta;
25 24
26 LaserPointerPoints::LaserPoint new_point; 25 LaserPointerPoints::LaserPoint new_point;
27 new_point.creation_time = new_point_time_; 26 new_point.creation_time = new_time;
28 instance_->points_.push_back(new_point); 27 instance_->points_.push_back(new_point);
28 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.
29 instance_->ClearOldPoints(); 29 instance_->ClearOldPoints();
30 } 30 }
31 } // namespace ash 31 } // namespace ash
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698