try to tame more of cef badness

minigui.d

@@ -8243,6 +8243,7 @@ class Window : Widget {
 		};
 
 		win.onFocusChange = (bool getting) {
+			// sdpyPrintDebugString("onFocusChange ", getting, " ", this.toString);
 			if(this.focusedWidget) {
 				if(getting) {
 					this.focusedWidget.emit!FocusEvent();
@@ -8463,6 +8464,12 @@ class Window : Widget {
 		// writefln("input proxy: 0x%0x", inputProxy);
 		this.inputProxy = new SimpleWindow(inputProxy);
 
+		/+
+		this.inputProxy.onFocusChange = (bool getting) {
+			sdpyPrintDebugString("input proxy focus change ", getting);
+		};
+		+/
+
 		XEvent lastEvent;
 		this.inputProxy.handleNativeEvent = (XEvent ev) {
 			lastEvent = ev;

minigui_addons/webview.d

@@ -543,6 +543,8 @@ class WebViewWidget_CEF : WebViewWidgetBase {
 			ev.mode = NotifyModes.NotifyNormal;
 			ev.detail = NotifyDetail.NotifyVirtual;
 
+			// sdpyPrintDebugString("Sending FocusIn");
+
 			trapXErrors( {
 				XSendEvent(XDisplayConnection.get, ozone, false, 0, cast(XEvent*) &ev);
 			});
@@ -560,6 +562,8 @@ class WebViewWidget_CEF : WebViewWidgetBase {
 			ev.mode = NotifyModes.NotifyNormal;
 			ev.detail = NotifyDetail.NotifyNonlinearVirtual;
 
+			// sdpyPrintDebugString("Sending FocusOut");
+
 			trapXErrors( {
 				XSendEvent(XDisplayConnection.get, ozone, false, 0, cast(XEvent*) &ev);
 			});
@@ -1284,6 +1288,7 @@ version(cef) {
 
 	class MiniguiFocusHandler : CEF!cef_focus_handler_t {
 		override void on_take_focus(RC!(cef_browser_t) browser, int next) nothrow {
+			// sdpyPrintDebugString("taking");
 			browser.runOnWebView(delegate(wv) {
 				Widget f;
 				if(next) {
@@ -1307,6 +1312,39 @@ version(cef) {
 			+/
 			// sdpyPrintDebugString("setting");
 
+			// if either the parent window or the ozone window has the focus, we
+			// can redirect it to the input focus. CEF calls this method sometimes
+			// before setting the focus (where return 1 can override) and sometimes
+			// after... which is totally inappropriate for it to do but it does anyway
+			// and we want to undo the damage of this.
+			browser.runOnWebView((ev) {
+				arsd.simpledisplay.Window focus_window;
+				int revert_to_return;
+				XGetInputFocus(XDisplayConnection.get, &focus_window, &revert_to_return);
+				if(focus_window is ev.parentWindow.win.impl.window || focus_window is ev.ozone) {
+					// refocus our correct input focus
+					ev.parentWindow.win.focus();
+					XSync(XDisplayConnection.get, 0);
+
+					// and then tell the chromium thing it still has it
+					// so it will think it got it, lost it, then got it again
+					// and hopefully not try to get it again
+					XFocusChangeEvent eve;
+					eve.type = arsd.simpledisplay.EventType.FocusIn;
+					eve.display = XDisplayConnection.get;
+					eve.window = ev.ozone;
+					eve.mode = NotifyModes.NotifyNormal;
+					eve.detail = NotifyDetail.NotifyVirtual;
+
+					// sdpyPrintDebugString("Sending FocusIn hack here");
+
+					trapXErrors( {
+						XSendEvent(XDisplayConnection.get, ev.ozone, false, 0, cast(XEvent*) &eve);
+					});
+
+				}
+			});
+
 			return 1; // otherwise, cancel because this bullshit tends to steal focus from other applications and i never, ever, ever want that to happen.
 			// seems to happen because of race condition in it getting a focus event and then stealing the focus from the parent
 			// even though things work fine if i always cancel except
@@ -1314,10 +1352,13 @@ version(cef) {
 			// it also breaks its own pop up menus and drop down boxes to allow this! wtf
 		}
 		override void on_got_focus(RC!(cef_browser_t) browser) nothrow {
+			// sdpyPrintDebugString("got");
 			browser.runOnWebView((ev) {
 				// this sometimes steals from the app too but it is relatively acceptable
 				// steals when i mouse in from the side of the window quickly, but still
 				// i want the minigui state to match so i'll allow it
+
+				//if(ev.parentWindow) ev.parentWindow.focus();
 				ev.focus();
 			});
 		}

simpledisplay.d

@@ -2406,6 +2406,8 @@ class SimpleWindow : CapableOfHandlingNativeEvent, CapableOfBeingDrawnUpon {
 						setTo = setRequestedInputFocus();
 					if(setTo is null)
 						setTo = this;
+
+					// sdpyPrintDebugString("grabInput() ", setTo.impl.window;
 					XSetInputFocus(XDisplayConnection.get, setTo.impl.window, RevertToParent, CurrentTime);
 				}
 			}
@@ -2575,6 +2577,7 @@ class SimpleWindow : CapableOfHandlingNativeEvent, CapableOfBeingDrawnUpon {
 				setTo = setRequestedInputFocus();
 			if(setTo is null)
 				setTo = this;
+			// sdpyPrintDebugString("sdpy.focus() ", setTo.impl.window);
 			XSetInputFocus(XDisplayConnection.get, setTo.impl.window, RevertToParent, CurrentTime);
 		} else version(Windows) {
 			SetFocus(this.impl.hwnd);
@@ -16400,6 +16403,7 @@ version(X11) {
 
 						// FIXME: so this is actually supposed to focus to a relevant child window if appropriate
 
+						// sdpyPrintDebugString("WM_TAKE_FOCUS ", setTo.impl.window);
 						XSetInputFocus(display, setTo.impl.window, RevertToParent, e.xclient.data.l[1]);
 					}
 				} else if(e.xclient.message_type == GetAtom!"MANAGER"(e.xany.display)) {