update CEF to 142

Author: timon-schelling

.nix/deps/cef.nix

@@ -1,28 +1,24 @@
 { pkgs, inputs, ... }:
 
 let
-  libcef = pkgs.libcef.overrideAttrs (
-    _: _: {
-      postInstall = ''
-        strip $out/lib/*
-      '';
-    }
-  );
-  cefPath = pkgs.runCommand "cef-path" { } ''
+  cef = pkgs.cef-binary.overrideAttrs (_: _: {
+    postInstall = ''
+      strip $out/Release/*.so*
+    '';
+  });
+
+  cefPath = pkgs.runCommand "cef-path" {} ''
     mkdir -p $out
 
-    ln -s ${libcef}/include $out/include
-    find ${libcef}/lib -type f -name "*" -exec ln -s {} $out/ \;
-    find ${libcef}/libexec -type f -name "*" -exec ln -s {} $out/ \;
-    cp -r ${libcef}/share/cef/* $out/
+    ln -s ${cef}/include $out/include
+    find ${cef}/Release -name "*" -type f -exec ln -s {} $out/ \;
+    find ${cef}/Resources -name "*" -maxdepth 1 -exec ln -s {} $out/ \;
 
-    echo '${
-      builtins.toJSON {
-        type = "minimal";
-        name = builtins.baseNameOf libcef.src.url;
-        sha1 = "";
-      }
-    }' > $out/archive.json
+    echo '${builtins.toJSON {
+      type = "minimal";
+      name = builtins.baseNameOf cef.src.url;
+      sha1 = "";
+    }}' > $out/archive.json
   '';
 in
 {

.nix/flake.lock

@@ -227,9 +227,8 @@ iai-callgrind = { version = "0.16" }
 ndarray = "0.16"
 strum = { version = "0.27", features = ["derive"] }
 dirs = "6.0"
-# TODO: remove fork usage when https://github.com/tauri-apps/cef-rs/pull/272 is merged and published
-cef = { git = "https://github.com/timon-schelling/cef-rs.git", rev = "98493a182928f1ff8d5bf8b9eea61483235df75d" }
-cef-dll-sys = { git = "https://github.com/timon-schelling/cef-rs.git", rev = "98493a182928f1ff8d5bf8b9eea61483235df75d" }
+cef = "142"
+cef-dll-sys = "142"
 include_dir = "0.7"
 tracing-subscriber = { version = "0.3", features = ["env-filter"] }
 tracing = "0.1"
@@ -270,4 +269,3 @@ debug = true
 [patch.crates-io]
 # Force cargo to use only one version of the dpi crate (vendoring breaks without this)
 dpi = { git = "https://github.com/rust-windowing/winit.git" }
-

Cargo.lock

@@ -79,7 +79,7 @@ pub(crate) fn handle_window_event(browser: &Browser, input_state: &mut InputStat
 				..Default::default()
 			};
 
-			key_event.modifiers = input_state.cef_modifiers(&event.location, event.repeat).raw();
+			key_event.modifiers = input_state.cef_modifiers(&event.location, event.repeat).0;
 
 			key_event.windows_key_code = match &event.logical_key {
 				winit::keyboard::Key::Named(named) => named.to_vk_bits(),
@@ -130,8 +130,8 @@ pub(crate) fn handle_window_event(browser: &Browser, input_state: &mut InputStat
 			let Some(host) = browser.host() else { return };
 
 			let mut mouse_event: MouseEvent = input_state.into();
-			mouse_event.modifiers |= cef_event_flags_t::EVENTFLAG_CONTROL_DOWN as u32;
-			mouse_event.modifiers |= cef_event_flags_t::EVENTFLAG_PRECISION_SCROLLING_DELTA as u32;
+			mouse_event.modifiers |= cef_event_flags_t::EVENTFLAG_CONTROL_DOWN.0;
+			mouse_event.modifiers |= cef_event_flags_t::EVENTFLAG_PRECISION_SCROLLING_DELTA.0;
 
 			let delta = (delta * PINCH_ZOOM_SPEED).round() as i32;
 

Cargo.toml

@@ -1,7 +1,9 @@
 use cef::MouseEvent;
+use cef::sys::cef_event_flags_t;
 use std::time::Instant;
 use winit::dpi::PhysicalPosition;
 use winit::event::{ElementState, MouseButton};
+use winit::keyboard::KeyLocation;
 
 use crate::cef::consts::{MULTICLICK_ALLOWED_TRAVEL, MULTICLICK_TIMEOUT};
 
@@ -31,26 +33,57 @@ impl InputState {
 		self.mouse_click_tracker.input(button, state, self.mouse_position)
 	}
 
-	pub(crate) fn cef_modifiers(&self, location: &winit::keyboard::KeyLocation, is_repeat: bool) -> CefModifiers {
-		CefModifiers::new(self, location, is_repeat)
+	pub(crate) fn cef_modifiers(&self, location: &winit::keyboard::KeyLocation, is_repeat: bool) -> cef_event_flags_t {
+		let mut flags = cef_event_flags_t::EVENTFLAG_NONE;
+
+		if self.modifiers.shift_key() {
+			flags |= cef_event_flags_t::EVENTFLAG_SHIFT_DOWN;
+		}
+		if self.modifiers.control_key() {
+			flags |= cef_event_flags_t::EVENTFLAG_CONTROL_DOWN;
+		}
+		if self.modifiers.alt_key() {
+			flags |= cef_event_flags_t::EVENTFLAG_ALT_DOWN;
+		}
+		if self.modifiers.meta_key() {
+			flags |= cef_event_flags_t::EVENTFLAG_COMMAND_DOWN;
+		}
+
+		if self.mouse_state.left {
+			flags |= cef_event_flags_t::EVENTFLAG_LEFT_MOUSE_BUTTON;
+		}
+		if self.mouse_state.right {
+			flags |= cef_event_flags_t::EVENTFLAG_RIGHT_MOUSE_BUTTON;
+		}
+		if self.mouse_state.middle {
+			flags |= cef_event_flags_t::EVENTFLAG_MIDDLE_MOUSE_BUTTON;
+		}
+
+		if is_repeat {
+			flags |= cef_event_flags_t::EVENTFLAG_IS_REPEAT;
+		}
+
+		flags |= match location {
+			KeyLocation::Left => cef_event_flags_t::EVENTFLAG_IS_LEFT,
+			KeyLocation::Right => cef_event_flags_t::EVENTFLAG_IS_RIGHT,
+			KeyLocation::Numpad => cef_event_flags_t::EVENTFLAG_IS_KEY_PAD,
+			KeyLocation::Standard => cef_event_flags_t::EVENTFLAG_NONE,
+		};
+
+		flags
 	}
 
-	pub(crate) fn cef_mouse_modifiers(&self) -> CefModifiers {
+	pub(crate) fn cef_mouse_modifiers(&self) -> cef_event_flags_t {
 		self.cef_modifiers(&winit::keyboard::KeyLocation::Standard, false)
 	}
 }
 
-impl From<InputState> for CefModifiers {
-	fn from(val: InputState) -> Self {
-		CefModifiers::new(&val, &winit::keyboard::KeyLocation::Standard, false)
-	}
-}
 impl From<&InputState> for MouseEvent {
 	fn from(val: &InputState) -> Self {
 		MouseEvent {
 			x: val.mouse_position.x as i32,
 			y: val.mouse_position.y as i32,
-			modifiers: val.cef_mouse_modifiers().raw(),
+			modifiers: val.cef_mouse_modifiers().0,
 		}
 	}
 }
@@ -59,7 +92,7 @@ impl From<&mut InputState> for MouseEvent {
 		MouseEvent {
 			x: val.mouse_position.x as i32,
 			y: val.mouse_position.y as i32,
-			modifiers: val.cef_mouse_modifiers().raw(),
+			modifiers: val.cef_mouse_modifiers().0,
 		}
 	}
 }
@@ -196,52 +229,3 @@ impl Default for ClickRecord {
 		}
 	}
 }
-
-pub(crate) struct CefModifiers(u32);
-impl CefModifiers {
-	fn new(input_state: &InputState, location: &winit::keyboard::KeyLocation, is_repeat: bool) -> Self {
-		use cef::sys::cef_event_flags_t;
-
-		let mut inner = 0;
-
-		if input_state.modifiers.shift_key() {
-			inner |= cef_event_flags_t::EVENTFLAG_SHIFT_DOWN as u32;
-		}
-		if input_state.modifiers.control_key() {
-			inner |= cef_event_flags_t::EVENTFLAG_CONTROL_DOWN as u32;
-		}
-		if input_state.modifiers.alt_key() {
-			inner |= cef_event_flags_t::EVENTFLAG_ALT_DOWN as u32;
-		}
-		if input_state.modifiers.meta_key() {
-			inner |= cef_event_flags_t::EVENTFLAG_COMMAND_DOWN as u32;
-		}
-
-		if input_state.mouse_state.left {
-			inner |= cef_event_flags_t::EVENTFLAG_LEFT_MOUSE_BUTTON as u32;
-		}
-		if input_state.mouse_state.right {
-			inner |= cef_event_flags_t::EVENTFLAG_RIGHT_MOUSE_BUTTON as u32;
-		}
-		if input_state.mouse_state.middle {
-			inner |= cef_event_flags_t::EVENTFLAG_MIDDLE_MOUSE_BUTTON as u32;
-		}
-
-		if is_repeat {
-			inner |= cef_event_flags_t::EVENTFLAG_IS_REPEAT as u32;
-		}
-
-		inner |= match location {
-			winit::keyboard::KeyLocation::Left => cef_event_flags_t::EVENTFLAG_IS_LEFT as u32,
-			winit::keyboard::KeyLocation::Right => cef_event_flags_t::EVENTFLAG_IS_RIGHT as u32,
-			winit::keyboard::KeyLocation::Numpad => cef_event_flags_t::EVENTFLAG_IS_KEY_PAD as u32,
-			winit::keyboard::KeyLocation::Standard => 0,
-		};
-
-		Self(inner)
-	}
-
-	pub(crate) fn raw(&self) -> u32 {
-		self.0
-	}
-}