[Win] Exception fuzz tests fail
authorpvollan@apple.com <pvollan@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 8 Sep 2016 07:45:27 +0000 (07:45 +0000)
committerpvollan@apple.com <pvollan@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 8 Sep 2016 07:45:27 +0000 (07:45 +0000)
https://bugs.webkit.org/show_bug.cgi?id=140928

Reviewed by Mark Lam.

JSTests:

Unskip tests.

* exceptionFuzz.yaml:

Source/JavaScriptCore:

Flush stdout when throwing the exception to make sure the output comes before output in the exception handler.
The tests depend on the output to stdout being written in the correct order.

* runtime/ExceptionFuzz.cpp:
(JSC::doExceptionFuzzing):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@205598 268f45cc-cd09-0410-ab3c-d52691b4dbfc

JSTests/ChangeLog
JSTests/exceptionFuzz.yaml
Source/JavaScriptCore/ChangeLog
Source/JavaScriptCore/runtime/ExceptionFuzz.cpp

index 1b992ce..02dc875 100644 (file)
@@ -1,3 +1,14 @@
+2016-09-08  Per Arne Vollan  <pvollan@apple.com>
+
+        [Win] Exception fuzz tests fail
+        https://bugs.webkit.org/show_bug.cgi?id=140928
+
+        Reviewed by Mark Lam.
+
+        Unskip tests.
+
+        * exceptionFuzz.yaml:
+
 2016-09-06  Saam Barati  <sbarati@apple.com>
 
         ProxyObject's structure should not have ObjectPrototype as its prototype and it should not have special behavior for intercepting "__proto__"
index 30e4c7c..fd53556 100644 (file)
@@ -22,9 +22,4 @@
 # THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 
 - path: exceptionFuzz
-  cmd: |
-      if ($hostOS == "windows")
-          skip
-      else
-          runExceptionFuzz
-      end
+  cmd: runExceptionFuzz
index 3905b53..d7b2022 100644 (file)
@@ -1,3 +1,16 @@
+2016-09-08  Per Arne Vollan  <pvollan@apple.com>
+
+        [Win] Exception fuzz tests fail
+        https://bugs.webkit.org/show_bug.cgi?id=140928
+
+        Reviewed by Mark Lam.
+
+        Flush stdout when throwing the exception to make sure the output comes before output in the exception handler.
+        The tests depend on the output to stdout being written in the correct order.
+
+        * runtime/ExceptionFuzz.cpp:
+        (JSC::doExceptionFuzzing):
+
 2016-09-07  Simon Fraser  <simon.fraser@apple.com>
 
         Enable the <meter> element on iOS
index 618354b..944c97a 100644 (file)
@@ -49,6 +49,7 @@ void doExceptionFuzzing(ExecState* exec, const char* where, void* returnPC)
     unsigned fireTarget = Options::fireExceptionFuzzAt();
     if (fireTarget == s_numberOfExceptionFuzzChecks) {
         printf("JSC EXCEPTION FUZZ: Throwing fuzz exception with call frame %p, seen in %s and return address %p.\n", exec, where, returnPC);
+        fflush(stdout);
         throwException(exec, scope, createError(exec, ASCIILiteral("Exception Fuzz")));
     }
 }