Don't draw the playhead so often.

2 files changed, 15 insertions, 4 deletions

diff --git a/src/wx/content_timeline.cc b/src/wx/content_timeline.cc
index bba2b433c..537877635 100644
--- a/src/wx/content_timeline.cc
+++ b/src/wx/content_timeline.cc
@@ -179,7 +179,9 @@ ContentTimeline::mouse_wheel_turned(wxMouseEvent& event)
 void
 ContentTimeline::update_playhead()
 {
-	Refresh();
+	if (!_last_drawn_playhead || playhead_position() != *_last_drawn_playhead) {
+		Refresh();
+	}
 }
 
 
@@ -265,13 +267,20 @@ ContentTimeline::paint_main()
 
 	gc->SetPen(*wxRED_PEN);
 	auto path = gc->CreatePath();
-	double const ph = _viewer.position().seconds() * pixels_per_second().get_value_or(0);
-	path.MoveToPoint(ph, 0);
-	path.AddLineToPoint(ph, pixels_per_track() * _tracks + 32);
+	_last_drawn_playhead = playhead_position();
+	path.MoveToPoint(*_last_drawn_playhead, 0);
+	path.AddLineToPoint(*_last_drawn_playhead, pixels_per_track() * _tracks + 32);
 	gc->StrokePath(path);
 }
 
 
+int
+ContentTimeline::playhead_position() const
+{
+	return std::round(_viewer.position().seconds() * pixels_per_second().get_value_or(0));
+}
+
+
 void
 ContentTimeline::film_change(ChangeType type, FilmProperty p)
 {
diff --git a/src/wx/content_timeline.h b/src/wx/content_timeline.h
index 5e8c658db..48dd986cc 100644
--- a/src/wx/content_timeline.h
+++ b/src/wx/content_timeline.h
@@ -111,6 +111,7 @@ private:
 	void zoom_all();
 	void update_playhead();
 	void mouse_wheel_turned(wxMouseEvent& event);
+	int playhead_position() const;
 
 	std::shared_ptr<ContentTimelineView> event_to_view(wxMouseEvent &);
 	TimelineContentViewList selected_views() const;
@@ -144,6 +145,7 @@ private:
 	wxTimer _timer;
 	boost::optional<int> _last_mouse_wheel_x;
 	boost::optional<double> _last_mouse_wheel_time;
+	boost::optional<int> _last_drawn_playhead;
 
 	static int const _minimum_pixels_per_track;