da1227e
# HG changeset patch
da1227e
# User ssadetsky
da1227e
# Date 1467220169 -3600
da1227e
#      Wed Jun 29 18:09:29 2016 +0100
da1227e
# Node ID 9fffaa63c49d93bff70b96af9a86b7635bce44be
da1227e
# Parent  d0462c26152fc8bd5f79db7f1670777e807ef2b3
da1227e
6260348, PR3066: GTK+ L&F JTextComponent not respecting desktop caret blink rate
da1227e
Reviewed-by: alexsch, azvegint
da1227e
da1227e
diff -r d0462c26152f -r 9fffaa63c49d src/share/classes/com/sun/java/swing/plaf/gtk/GTKEngine.java
da1227e
--- openjdk/jdk/src/share/classes/com/sun/java/swing/plaf/gtk/GTKEngine.java	Tue Jun 28 19:44:50 2016 +0100
da1227e
+++ openjdk/jdk/src/share/classes/com/sun/java/swing/plaf/gtk/GTKEngine.java	Wed Jun 29 18:09:29 2016 +0100
da1227e
@@ -1,5 +1,5 @@
da1227e
 /*
da1227e
- * Copyright (c) 2005, 2007, Oracle and/or its affiliates. All rights reserved.
da1227e
+ * Copyright (c) 2005, 2015, Oracle and/or its affiliates. All rights reserved.
da1227e
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
da1227e
  *
da1227e
  * This code is free software; you can redistribute it and/or modify it
da1227e
@@ -93,7 +93,9 @@
da1227e
      */
da1227e
     static enum Settings {
da1227e
         GTK_FONT_NAME,
da1227e
-        GTK_ICON_SIZES
da1227e
+        GTK_ICON_SIZES,
da1227e
+        GTK_CURSOR_BLINK,
da1227e
+        GTK_CURSOR_BLINK_TIME
da1227e
     }
da1227e
 
da1227e
     /* Custom regions are needed for representing regions that don't exist
da1227e
diff -r d0462c26152f -r 9fffaa63c49d src/share/classes/com/sun/java/swing/plaf/gtk/GTKLookAndFeel.java
da1227e
--- openjdk/jdk/src/share/classes/com/sun/java/swing/plaf/gtk/GTKLookAndFeel.java	Tue Jun 28 19:44:50 2016 +0100
da1227e
+++ openjdk/jdk/src/share/classes/com/sun/java/swing/plaf/gtk/GTKLookAndFeel.java	Wed Jun 29 18:09:29 2016 +0100
da1227e
@@ -1,5 +1,5 @@
da1227e
 /*
da1227e
- * Copyright (c) 2002, 2013, Oracle and/or its affiliates. All rights reserved.
da1227e
+ * Copyright (c) 2002, 2015, Oracle and/or its affiliates. All rights reserved.
da1227e
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
da1227e
  *
da1227e
  * This code is free software; you can redistribute it and/or modify it
da1227e
@@ -371,7 +371,17 @@
da1227e
         int vProgWidth  =  22 - (progXThickness * 2);
da1227e
         int vProgHeight =  80 - (progYThickness * 2);
da1227e
 
da1227e
-        Integer caretBlinkRate = Integer.valueOf(500);
da1227e
+        Integer caretBlinkRate;
da1227e
+        if (Boolean.FALSE.equals(GTKEngine.INSTANCE.getSetting(
da1227e
+                GTKEngine.Settings.GTK_CURSOR_BLINK))) {
da1227e
+            caretBlinkRate = Integer.valueOf(0);
da1227e
+        } else {
da1227e
+            caretBlinkRate = (Integer) GTKEngine.INSTANCE.getSetting(
da1227e
+                    GTKEngine.Settings.GTK_CURSOR_BLINK_TIME);
da1227e
+            if (caretBlinkRate == null) {
da1227e
+                caretBlinkRate = Integer.valueOf(500);
da1227e
+            }
da1227e
+        }
da1227e
         Insets zeroInsets = new InsetsUIResource(0, 0, 0, 0);
da1227e
 
da1227e
         Double defaultCaretAspectRatio = new Double(0.025);
da1227e
diff -r d0462c26152f -r 9fffaa63c49d src/solaris/native/sun/awt/gtk2_interface.c
da1227e
--- openjdk/jdk/src/solaris/native/sun/awt/gtk2_interface.c	Tue Jun 28 19:44:50 2016 +0100
da1227e
+++ openjdk/jdk/src/solaris/native/sun/awt/gtk2_interface.c	Wed Jun 29 18:09:29 2016 +0100
da1227e
@@ -1316,9 +1316,6 @@
da1227e
             {
da1227e
                 result = gtk2_widgets[_GTK_COMBO_BOX_TEXT_FIELD_TYPE] =
da1227e
                      (*fp_gtk_entry_new)();
da1227e
-
da1227e
-                GtkSettings* settings = fp_gtk_widget_get_settings(result);
da1227e
-                fp_g_object_set(settings, "gtk-cursor-blink", FALSE, NULL);
da1227e
             }
da1227e
             result = gtk2_widgets[_GTK_COMBO_BOX_TEXT_FIELD_TYPE];
da1227e
             break;
da1227e
@@ -1363,10 +1360,6 @@
da1227e
             {
da1227e
                 gtk2_widgets[_GTK_ENTRY_TYPE] =
da1227e
                     (*fp_gtk_entry_new)();
da1227e
-
da1227e
-                GtkSettings* settings =
da1227e
-                    fp_gtk_widget_get_settings(gtk2_widgets[_GTK_ENTRY_TYPE]);
da1227e
-                fp_g_object_set(settings, "gtk-cursor-blink", FALSE, NULL);
da1227e
             }
da1227e
             result = gtk2_widgets[_GTK_ENTRY_TYPE];
da1227e
             break;
da1227e
@@ -1558,9 +1551,6 @@
da1227e
             {
da1227e
                 result = gtk2_widgets[_GTK_SPIN_BUTTON_TYPE] =
da1227e
                     (*fp_gtk_spin_button_new)(NULL, 0, 0);
da1227e
-
da1227e
-                GtkSettings* settings = fp_gtk_widget_get_settings(result);
da1227e
-                fp_g_object_set(settings, "gtk-cursor-blink", FALSE, NULL);
da1227e
             }
da1227e
             result = gtk2_widgets[_GTK_SPIN_BUTTON_TYPE];
da1227e
             break;
da1227e
@@ -2510,14 +2500,20 @@
da1227e
 
da1227e
     return result;
da1227e
 }
da1227e
-/*
da1227e
+
da1227e
 jobject get_integer_property(JNIEnv *env, GtkSettings* settings, const gchar* key)
da1227e
 {
da1227e
-    gint    intval = NULL;
da1227e
-
da1227e
+    gint intval = NULL;
da1227e
     (*fp_g_object_get)(settings, key, &intval, NULL);
da1227e
     return create_Integer(env, intval);
da1227e
-}*/
da1227e
+}
da1227e
+
da1227e
+jobject get_boolean_property(JNIEnv *env, GtkSettings* settings, const gchar* key)
da1227e
+{
da1227e
+    gint intval = NULL;
da1227e
+    (*fp_g_object_get)(settings, key, &intval, NULL);
da1227e
+    return create_Boolean(env, intval);
da1227e
+}
da1227e
 
da1227e
 jobject gtk2_get_setting(JNIEnv *env, Setting property)
da1227e
 {
da1227e
@@ -2529,6 +2525,10 @@
da1227e
             return get_string_property(env, settings, "gtk-font-name");
da1227e
         case GTK_ICON_SIZES:
da1227e
             return get_string_property(env, settings, "gtk-icon-sizes");
da1227e
+        case GTK_CURSOR_BLINK:
da1227e
+            return get_boolean_property(env, settings, "gtk-cursor-blink");
da1227e
+        case GTK_CURSOR_BLINK_TIME:
da1227e
+            return get_integer_property(env, settings, "gtk-cursor-blink-time");
da1227e
     }
da1227e
 
da1227e
     return NULL;
da1227e
diff -r d0462c26152f -r 9fffaa63c49d src/solaris/native/sun/awt/gtk2_interface.h
da1227e
--- openjdk/jdk/src/solaris/native/sun/awt/gtk2_interface.h	Tue Jun 28 19:44:50 2016 +0100
da1227e
+++ openjdk/jdk/src/solaris/native/sun/awt/gtk2_interface.h	Wed Jun 29 18:09:29 2016 +0100
da1227e
@@ -1,5 +1,5 @@
da1227e
 /*
da1227e
- * Copyright (c) 2005, 2013, Oracle and/or its affiliates. All rights reserved.
da1227e
+ * Copyright (c) 2005, 2015, Oracle and/or its affiliates. All rights reserved.
da1227e
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
da1227e
  *
da1227e
  * This code is free software; you can redistribute it and/or modify it
da1227e
@@ -140,7 +140,9 @@
da1227e
 typedef enum _Setting
da1227e
 {
da1227e
     GTK_FONT_NAME,
da1227e
-    GTK_ICON_SIZES
da1227e
+    GTK_ICON_SIZES,
da1227e
+    GTK_CURSOR_BLINK,
da1227e
+    GTK_CURSOR_BLINK_TIME
da1227e
 } Setting;
da1227e
 
da1227e
 /* GTK types, here to eliminate need for GTK headers at compile time */