Desktop: Forward multi-clicks (double/triple/etc.-clicks) to CEF correctly (#3224)

* fix multiclick

* Revert "fix multiclick"

This reverts commit c4130d1.

* try another way

* use copy

* Use struct initializer syntax fror click tracker (#3226)

---------

Co-authored-by: Dennis Kobert <[email protected]>

Author: timon-schelling

desktop/src/cef/input.rs

@@ -151,7 +151,7 @@ impl InputState {
 
 	fn mouse_input(&mut self, button: &MouseButton, state: &ElementState) -> ClickCount {
 		self.mouse_state.update(button, state);
-		self.mouse_click_tracker.input(button, state, &self.mouse_position)
+		self.mouse_click_tracker.input(button, state, self.mouse_position)
 	}
 
 	fn cef_modifiers(&self, location: &winit::keyboard::KeyLocation, is_repeat: bool) -> CefModifiers {
@@ -187,7 +187,7 @@ impl From<&mut InputState> for MouseEvent {
 	}
 }
 
-#[derive(Default, Clone)]
+#[derive(Default, Clone, Copy)]
 pub(crate) struct MousePosition {
 	x: usize,
 	y: usize,
@@ -233,26 +233,41 @@ struct ClickTracker {
 	right: Option<ClickRecord>,
 }
 impl ClickTracker {
-	fn input(&mut self, button: &MouseButton, state: &ElementState, position: &MousePosition) -> ClickCount {
+	fn input(&mut self, button: &MouseButton, state: &ElementState, position: MousePosition) -> ClickCount {
 		let record = match button {
 			MouseButton::Left => &mut self.left,
 			MouseButton::Right => &mut self.right,
 			MouseButton::Middle => &mut self.middle,
 			_ => return ClickCount::Single,
 		};
 
-		let now = Instant::now();
-
 		let Some(record) = record else {
-			*record = Some(ClickRecord {
-				time: now,
-				position: position.clone(),
-				down_count: ClickCount::Single,
-				up_count: ClickCount::Single,
-			});
+			*record = Some(ClickRecord { position, ..Default::default() });
 			return ClickCount::Single;
 		};
 
+		let now = Instant::now();
+		record.time = now;
+		record.position = position;
+
+		match state {
+			ElementState::Pressed if record.down_count == ClickCount::Double => {
+				*record = ClickRecord {
+					down_count: ClickCount::Single,
+					..*record
+				};
+				return ClickCount::Single;
+			}
+			ElementState::Released if record.up_count == ClickCount::Double => {
+				*record = ClickRecord {
+					up_count: ClickCount::Single,
+					..*record
+				};
+				return ClickCount::Single;
+			}
+			_ => {}
+		}
+
 		let dx = position.x.abs_diff(record.position.x);
 		let dy = position.y.abs_diff(record.position.y);
 		let within_dist = dx <= MULTICLICK_ALLOWED_TRAVEL && dy <= MULTICLICK_ALLOWED_TRAVEL;
@@ -261,33 +276,19 @@ impl ClickTracker {
 		let count = if within_time && within_dist { ClickCount::Double } else { ClickCount::Single };
 
 		*record = match state {
-			ElementState::Pressed => ClickRecord {
-				time: now,
-				position: position.clone(),
-				down_count: count.clone(),
-				up_count: record.up_count.clone(),
-			},
-			ElementState::Released => ClickRecord {
-				time: now,
-				position: position.clone(),
-				down_count: record.down_count.clone(),
-				up_count: count.clone(),
-			},
+			ElementState::Pressed => ClickRecord { down_count: count, ..*record },
+			ElementState::Released => ClickRecord { up_count: count, ..*record },
 		};
 		count
 	}
 }
 
-#[derive(Clone)]
+#[derive(Clone, Copy, PartialEq, Default)]
 enum ClickCount {
+	#[default]
 	Single,
 	Double,
 }
-impl Default for ClickCount {
-	fn default() -> Self {
-		Self::Single
-	}
-}
 impl From<ClickCount> for i32 {
 	fn from(count: ClickCount) -> i32 {
 		match count {
@@ -297,13 +298,25 @@ impl From<ClickCount> for i32 {
 	}
 }
 
+#[derive(Clone, Copy)]
 struct ClickRecord {
 	time: Instant,
 	position: MousePosition,
 	down_count: ClickCount,
 	up_count: ClickCount,
 }
 
+impl Default for ClickRecord {
+	fn default() -> Self {
+		Self {
+			time: Instant::now(),
+			position: Default::default(),
+			down_count: Default::default(),
+			up_count: Default::default(),
+		}
+	}
+}
+
 struct CefModifiers(u32);
 impl CefModifiers {
 	fn new(input_state: &InputState, location: &winit::keyboard::KeyLocation, is_repeat: bool) -> Self {