Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP
Browse files

Additional crashtest fix follow up from bug 399227 - catch errors thr…

…own by functions that now fail, so that the test completes properly
  • Loading branch information...
commit dea2d61f21f20225432707b2e726793658c15252 1 parent 5011ed1
Mark Banner authored
View
7 layout/xul/base/src/tree/src/crashtests/399227-1.xul
@@ -13,7 +13,12 @@
selection.select(0);
tree.parentNode.removeChild(tree);
- selection.rangedSelect(1, 1, false);
+
+ // This is expected to throw an error (it used to crash).
+ try {
+ selection.rangedSelect(1, 1, false);
+ }
+ catch (ex) {}
document.documentElement.removeAttribute("class");
}
View
9 layout/xul/base/src/tree/src/crashtests/399227-2.xul
@@ -2,7 +2,7 @@
<?xml-stylesheet href="chrome://global/skin/" type="text/css"?>
-<window xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul" class="reftest-wait" onload="setTimeout(boom, 30)">
+<window xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul" class="reftest-wait" onload="setTimeout(boom, 30);">
<script>
@@ -16,7 +16,12 @@
selection.select(0);
selection.currentColumn = treecolumn0;
tree.parentNode.removeChild(tree);
- selection.currentColumn = treecolumn1;
+
+ // This is expected to throw an error (it used to crash).
+ try {
+ selection.currentColumn = treecolumn1;
+ }
+ catch (ex) {}
document.documentElement.removeAttribute("class");
}
Please sign in to comment.
Something went wrong with that request. Please try again.