Changeset 1233 for trunk/autoquest-htmlmonitor/src/main/java/de/ugoe

Timestamp:

06/27/13 17:12:41 (11 years ago)

Author:

pharms

Message:

• improved logging
• corrected handling of onchange events

File:

• trunk/autoquest-htmlmonitor/src/main/java/de/ugoe/cs/autoquest/htmlmonitor/HtmlMonitorServlet.java (modified) (3 diffs)

trunk/autoquest-htmlmonitor/src/main/java/de/ugoe/cs/autoquest/htmlmonitor/HtmlMonitorServlet.java

@@ -182 +182 @@
                 ("could not parse incoming data --> discarding it (" + e.toString() + ")");
         }
+        
+        // this must be done to prevent firefox from notifying the "element not found" error
+        response.setContentType("text/plain");
+        response.getOutputStream().write(' ');
     }
 
@@ -223 +227 @@
                 else {
                     messageListener.handleMessage(clientInfos, guiStructure, events);
+                    Console.println
+                        ("handled message of " + clientInfos.getClientId() + " (" +
+                         clientInfos.getWebAppId() + "; " + events.length + " events)");
                 }
             }
@@ -516 +523 @@
         }
         else if ("onchange".equals(eventType)) {
-            // "input_password" don't have a selectedValue
-            if (domPath.contains("input_password")) {
-                result =
-                    (coordinates == null) && (key == null) &&
-                    (scrollPosition == null) && (selectedValue == null);
-            }
-            else {
-                result =
-                    (coordinates == null) && (key == null) &&
-                    (scrollPosition == null) && (selectedValue != null);
-            }
-
+            // onchange events may have and may also not have a selected value. In the second
+            // case it means, that null was selected
+            result = (coordinates == null) && (key == null) && (scrollPosition == null);
         }
         else if ("onkeypress".equals(eventType) || "onkeydown".equals(eventType) ||