window_builder hook no longer exposes winit WindowBuilder

[LunNova]

The window_builder hook was initially implemented in #3390 to allow directly configuring the winit windowbuilder.

#3172 removed this feature, and instead implemented specific support for only one particular winit NativeExt use for macos:

egui/crates/eframe/src/epi/mod.rs

Lines 41 to 47 in 83aa310

	/// Hook into the building of a the native window.
	///
	/// You can configure any platform specific details required on top of the default configuration
	/// done by `eframe`.
	#[cfg(not(target_arch = "wasm32"))]
	#[cfg(any(feature = "glow", feature = "wgpu"))]
	pub type WindowBuilderHook = Box<dyn FnOnce(egui::ViewportBuilder) -> egui::ViewportBuilder>;

egui/crates/egui-winit/src/lib.rs

Lines 1565 to 1570 in d319489

	use winit::platform::macos::WindowBuilderExtMacOS as _;
	window_builder = window_builder
	.with_title_hidden(!_title_shown.unwrap_or(true))
	.with_titlebar_buttons_hidden(!_titlebar_buttons_shown.unwrap_or(true))
	.with_titlebar_transparent(!_titlebar_shown.unwrap_or(true))
	.with_fullsize_content_view(_fullsize_content_view.unwrap_or(false));

The window_builder hook no longer matches its documentation, and now modifies the Viewportbuilder which was already available as the viewport field of NativeOptions.

window_builder should be reverted to take a winit WindowBuilder as its argument so native winit WindowBuilder extension can be used. It doesn't make sense for it to take ViewportBuilder as an argument.