Fix is_visible()

Author: Thiago M Sayao

modules/javafx.graphics/src/main/native-glass/gtk/GlassApplication.cpp

@@ -152,6 +152,9 @@ static gboolean x11_event_source_dispatch(GSource* source, GSourceFunc callback,
 //                case ConfigureRequest:
 //                    g_print("============> X11 Configure Request\n");
 //                    break;
+                case VisibilityNotify:
+                    ctx->process_visibility(&xevent.xvisibility);
+                    break;
                 case ConfigureNotify:
                     g_print("X11 Configure\n");
                     ctx->process_configure(&xevent.xconfigure);

modules/javafx.graphics/src/main/native-glass/gtk/glass_window.cpp

@@ -140,6 +140,10 @@ void WindowContextBase::process_state(GdkEventWindowState* event) {
     }
 }
 
+void WindowContextBase::process_visibility(XVisibilityEvent* event) {
+    visibility_state = event->state;
+}
+
 void WindowContextBase::process_focus(GdkEventFocus* event) {
 }
 
@@ -611,13 +615,16 @@ void WindowContextBase::set_visible(bool visible) {
 }
 
 bool WindowContextBase::is_visible() {
-    XWindowAttributes xattr;
-    if (XGetWindowAttributes(display, xwindow, &xattr)) {
-        return !(xattr.map_state == IsUnmapped);
-    }
+//    XWindowAttributes xattr;
+//    if (XGetWindowAttributes(display, xwindow, &xattr)) {
+//        return !(xattr.map_state == IsUnmapped);
+//    }
 //    return gtk_widget_get_visible(gtk_widget);
+//    if (!map_received) {
+//        return false;
+//    }
 
-    return false;
+    return (visibility_state != VisibilityFullyObscured);
 }
 
 bool WindowContextBase::set_view(jobject view) {
@@ -1140,10 +1147,10 @@ void WindowContextTop::process_configure(XConfigureEvent* event) {
             updateWindowConstraints = true;
             if (!frame_extents_initialized && !is_null_extents()) {
                 frame_extents_initialized = true;
-                set_bounds(0, 0, false, false,
-                    requested_bounds.width, requested_bounds.height,
-                    requested_bounds.client_width, requested_bounds.client_height
-                );
+//                set_bounds(0, 0, false, false,
+//                    requested_bounds.width, requested_bounds.height,
+//                    requested_bounds.client_width, requested_bounds.client_height
+//                );
             }
         }
     } else {

modules/javafx.graphics/src/main/native-glass/gtk/glass_window.h

@@ -180,6 +180,7 @@ class WindowContext {
     virtual void process_mouse_cross(XCrossingEvent*) = 0;
     virtual void process_key(GdkEventKey*) = 0;
     virtual void process_state(GdkEventWindowState*) = 0;
+    virtual void process_visibility(XVisibilityEvent*) = 0;
 
     virtual void notify_state(jint) = 0;
     virtual void notify_on_top(bool) {}
@@ -225,6 +226,7 @@ class WindowContextBase: public WindowContext {
 
     GdkWMFunction gdk_windowManagerFunctions;
 
+    int visibility_state;
     bool is_iconified;
     bool is_maximized;
     bool is_mouse_entered;
@@ -290,7 +292,7 @@ class WindowContextBase: public WindowContext {
     void process_mouse_cross(XCrossingEvent*);
     void process_key(GdkEventKey*);
     void process_state(GdkEventWindowState*);
-
+    void process_visibility(XVisibilityEvent*);
     void notify_state(jint);
 
     void increment_events_counter();