Skip to content

Commit

Permalink
Trap window.onerror by default.
Browse files Browse the repository at this point in the history
Changes gwt.uncaughtexceptionhander.windowonerror
to REPORT_IF_NO_HANDLER.
This is a good default for most GWT applications.
see: https://gwt-review.googlesource.com/#/c/18540/


Change-Id: I2a1cb3e89aacfd14315841835aed26927e1ff944
  • Loading branch information
dankurka committed Jul 18, 2017
1 parent a3a3e69 commit e1f1eb0
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 20 deletions.
2 changes: 1 addition & 1 deletion user/src/com/google/gwt/core/Core.gwt.xml
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,7 @@
- Do not trap anything (legacy behavior) (IGNORE)
-->
<define-property name="gwt.uncaughtexceptionhander.windowonerror" values="IGNORE, REPORT, REPORT_IF_NO_HANDLER" />
<set-property name="gwt.uncaughtexceptionhander.windowonerror" value="IGNORE"/>
<set-property name="gwt.uncaughtexceptionhander.windowonerror" value="REPORT_IF_NO_HANDLER"/>

<!-- To support legacy logging flags -->
<inherits name="com.google.gwt.logging.LogImpl"/>
Expand Down
18 changes: 0 additions & 18 deletions user/test/com/google/gwt/core/WindowOnError.gwt.xml

This file was deleted.

Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ public class WindowOnErrorTest extends GWTTestCase {

@Override
public String getModuleName() {
return "com.google.gwt.core.WindowOnError";
return "com.google.gwt.core.Core";
}

@Override
Expand Down

0 comments on commit e1f1eb0

Please sign in to comment.