[HtmlUnit] SVN: [14641] trunk/htmlunit/src/main/java/com/gargoylesoftware/htmlunit/ javascript/JavaScriptEngine.java

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[HtmlUnit] SVN: [14641] trunk/htmlunit/src/main/java/com/gargoylesoftware/htmlunit/ javascript/JavaScriptEngine.java

HtmlUnit - Dev mailing list
Revision: 14641
          http://sourceforge.net/p/htmlunit/code/14641
Author:   rbri
Date:     2017-07-03 17:33:52 +0000 (Mon, 03 Jul 2017)
Log Message:
-----------
as workaround log/ignore the exception for now
Issue 1897

Modified Paths:
--------------
    trunk/htmlunit/src/main/java/com/gargoylesoftware/htmlunit/javascript/JavaScriptEngine.java

Modified: trunk/htmlunit/src/main/java/com/gargoylesoftware/htmlunit/javascript/JavaScriptEngine.java
===================================================================
--- trunk/htmlunit/src/main/java/com/gargoylesoftware/htmlunit/javascript/JavaScriptEngine.java 2017-07-02 14:36:57 UTC (rev 14640)
+++ trunk/htmlunit/src/main/java/com/gargoylesoftware/htmlunit/javascript/JavaScriptEngine.java 2017-07-03 17:33:52 UTC (rev 14641)
@@ -338,7 +338,18 @@
                         final Object prototypeProperty = ScriptableObject.getProperty(window, prototype.getClassName());
 
                         if (function instanceof FunctionObject) {
-                            ((FunctionObject) function).addAsConstructor(window, prototype);
+                            try {
+                                ((FunctionObject) function).addAsConstructor(window, prototype);
+                            }
+                            catch (final Exception e) {
+                                // TODO see issue #1897
+                                if (LOG.isWarnEnabled()) {
+                                    final String newline = System.lineSeparator();
+                                    LOG.warn("Error during JavaScriptEngine.init(WebWindow, Context)" + newline
+                                            + e.getMessage() + newline
+                                            + "prototype: " + prototype.getClassName());
+                                }
+                            }
                         }
 
                         ScriptableObject.defineProperty(window, hostClassSimpleName, function,
@@ -358,7 +369,18 @@
                     }
                     else {
                         if (function instanceof FunctionObject) {
-                            ((FunctionObject) function).addAsConstructor(window, prototype);
+                            try {
+                                ((FunctionObject) function).addAsConstructor(window, prototype);
+                            }
+                            catch (final Exception e) {
+                                // TODO see issue #1897
+                                if (LOG.isWarnEnabled()) {
+                                    final String newline = System.lineSeparator();
+                                    LOG.warn("Error during JavaScriptEngine.init(WebWindow, Context)" + newline
+                                            + e.getMessage() + newline
+                                            + "prototype: " + prototype.getClassName());
+                                }
+                            }
                         }
                     }
 


------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
HtmlUnit-develop mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/htmlunit-develop