<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>178592</bug_id>
          
          <creation_ts>2017-10-20 10:20:53 -0700</creation_ts>
          <short_desc>[Regression] test262.yaml/test262/test/language/statements/try/tco-catch.js.default-strict is failing on Test262 consistently.</short_desc>
          <delta_ts>2017-10-26 09:42:13 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>JavaScriptCore</component>
          <version>WebKit Nightly Build</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>DUPLICATE</resolution>
          <dup_id>176601</dup_id>
          <see_also>https://bugs.webkit.org/show_bug.cgi?id=176703</see_also>
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Matt Lewis">jlewis3</reporter>
          <assigned_to name="Robin Morisset">rmorisset</assigned_to>
          <cc>rmorisset</cc>
    
    <cc>ryanhaddad</cc>
    
    <cc>saam</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1362745</commentid>
    <comment_count>0</comment_count>
    <who name="Matt Lewis">jlewis3</who>
    <bug_when>2017-10-20 10:20:53 -0700</bug_when>
    <thetext>test262.yaml/test262/test/language/statements/try/tco-catch.js.default-strict is failing consistently on the Test262 tester according to the Archived runs.

https://build.webkit.org/builders/Apple%20El%20Capitan%20Debug%20Test262%20(Tests)?numbuilds=50

It started failing after https://trac.webkit.org/changeset/223691/webkit

failure:
test262.yaml/test262/test/language/statements/try/tco-catch.js.default-strict: ERROR: Unexpected exit code: 0
FAIL: test262.yaml/test262/test/language/statements/try/tco-catch.js.default-strict

log:
https://build.webkit.org/builders/Apple%20El%20Capitan%20Debug%20Test262%20%28Tests%29/builds/3943/steps/test262-test/logs/stdio</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1362752</commentid>
    <comment_count>1</comment_count>
    <who name="Robin Morisset">rmorisset</who>
    <bug_when>2017-10-20 10:28:00 -0700</bug_when>
    <thetext>Thanks for reporting it, I will look into it.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1363247</commentid>
    <comment_count>2</comment_count>
    <who name="Robin Morisset">rmorisset</who>
    <bug_when>2017-10-23 05:28:26 -0700</bug_when>
    <thetext>I spent some time trying to reproduce this bug and not understanding why it only triggered when using the testing harness and not when running the program by hand.

After looking at JSTests/test262.yaml in more details, I saw that this program was marked :fail, so visibly it was expected that the test would fail, and the testing harness is now complaining that it no longer does. This assumption was last changed in https://bugs.webkit.org/show_bug.cgi?id=177859.

Interestingly, increasing the number of iterations to 100000 makes the test fail with stack overflow. This looks possibly related to https://bugs.webkit.org/show_bug.cgi?id=176703.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1364269</commentid>
    <comment_count>3</comment_count>
    <who name="Ryan Haddad">ryanhaddad</who>
    <bug_when>2017-10-25 10:15:39 -0700</bug_when>
    <thetext>The bot has been red for days due to this issue. Should we go ahead and mark the test as passing?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1364300</commentid>
    <comment_count>4</comment_count>
    <who name="Ryan Haddad">ryanhaddad</who>
    <bug_when>2017-10-25 10:42:54 -0700</bug_when>
    <thetext>I confirmed with Robin that we should mark the test as passing.

Made the change in https://trac.webkit.org/changeset/223961/webkit</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1364868</commentid>
    <comment_count>5</comment_count>
    <who name="Ryan Haddad">ryanhaddad</who>
    <bug_when>2017-10-26 09:41:28 -0700</bug_when>
    <thetext>Reverted r223961 for reason:

The change that required this has been rolled out.

Committed r224022: &lt;https://trac.webkit.org/changeset/224022&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1364870</commentid>
    <comment_count>6</comment_count>
    <who name="Ryan Haddad">ryanhaddad</who>
    <bug_when>2017-10-26 09:42:13 -0700</bug_when>
    <thetext>Duping to the bug that caused this.

*** This bug has been marked as a duplicate of bug 176601 ***</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>