Bug 161156

Summary: REGRESSION: js/dom/prototype-chain-caching-with-impure-get-own-property-slot-traps-5.html has a flaky failure
Product: WebKit Reporter: Ryan Haddad <ryanhaddad>
Component: New BugsAssignee: Filip Pizlo <fpizlo>
Status: RESOLVED FIXED    
Severity: Normal CC: ap, benjamin, csaavedra, fpizlo, ggaren, jfbastien, keith_miller, mark.lam, msaboff, rniwa, saam, sam, webkit-bug-importer, ysuzuki
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
the patch saam: review+

Description Ryan Haddad 2016-08-24 12:50:07 PDT
LayoutTest js/dom/prototype-chain-caching-with-impure-get-own-property-slot-traps-5.html is a flaky failure

https://build.webkit.org/builders/Apple%20Yosemite%20Release%20WK2%20(Tests)/builds/17320

https://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=js%2Fdom%2Fprototype-chain-caching-with-impure-get-own-property-slot-traps-5.html

--- /Volumes/Data/slave/yosemite-release-tests-wk2/build/layout-test-results/js/dom/prototype-chain-caching-with-impure-get-own-property-slot-traps-5-expected.txt
+++ /Volumes/Data/slave/yosemite-release-tests-wk2/build/layout-test-results/js/dom/prototype-chain-caching-with-impure-get-own-property-slot-traps-5-actual.txt
@@ -48,8 +48,8 @@
 PASS typeof f() is "function"
 PASS typeof f() is "function"
 PASS typeof f() is "function"
-PASS typeof f() is "object"
-PASS typeof f() is "object"
+FAIL typeof f() should be object. Was function.
+FAIL typeof f() should be object. Was function.
 PASS typeof f() is "object"
 PASS typeof f() is "object"
 PASS typeof f() is "object"
Comment 1 Ryan Haddad 2016-08-30 12:44:29 PDT
Marked test as flaky on mac in http://trac.webkit.org/projects/webkit/changeset/205196
Comment 2 Ryosuke Niwa 2016-08-30 15:02:06 PDT
Oh this is a real regression. We're not clearing the inline cache in time sometimes!
Comment 3 Radar WebKit Bug Importer 2016-09-01 09:04:33 PDT
<rdar://problem/28116283>
Comment 4 Filip Pizlo 2017-06-08 12:43:17 PDT
Created attachment 312334 [details]
the patch
Comment 5 Filip Pizlo 2017-06-08 13:08:50 PDT
Landed in https://trac.webkit.org/changeset/217944/webkit
Comment 6 Claudio Saavedra 2017-06-09 04:27:15 PDT
The test is missing an expectation file. Is this intentional?
Comment 7 Yusuke Suzuki 2017-06-09 04:29:49 PDT
(In reply to Claudio Saavedra from comment #6)
> The test is missing an expectation file. Is this intentional?

I don't think so. We should add expectation file.
Comment 8 Ryan Haddad 2017-06-09 14:40:30 PDT
(In reply to Yusuke Suzuki from comment #7)
> (In reply to Claudio Saavedra from comment #6)
> > The test is missing an expectation file. Is this intentional?
> 
> I don't think so. We should add expectation file.

I went ahead and added one in https://trac.webkit.org/changeset/218013/webkit
Comment 9 Sam Weinig 2017-06-10 19:04:02 PDT
platform/Mac/TestExpectations still has js/dom/prototype-chain-caching-with-impure-get-own-property-slot-traps-5.html listed as [ Pass Failure ], pointing to this bug. Can we remove it from the list now?
Comment 10 Saam Barati 2017-06-12 12:00:42 PDT
(In reply to Sam Weinig from comment #9)
> platform/Mac/TestExpectations still has
> js/dom/prototype-chain-caching-with-impure-get-own-property-slot-traps-5.
> html listed as [ Pass Failure ], pointing to this bug. Can we remove it from
> the list now?

I went ahead and did this:
https://trac.webkit.org/changeset/218122/webkit
Comment 11 Sam Weinig 2017-06-12 15:14:23 PDT
(In reply to Saam Barati from comment #10)
> (In reply to Sam Weinig from comment #9)
> > platform/Mac/TestExpectations still has
> > js/dom/prototype-chain-caching-with-impure-get-own-property-slot-traps-5.
> > html listed as [ Pass Failure ], pointing to this bug. Can we remove it from
> > the list now?
> 
> I went ahead and did this:
> https://trac.webkit.org/changeset/218122/webkit

Cool cool.