NEW 113125
testLeakingPrototypesAcrossContexts added in r146682 doesn't compile on Win and fails on Mac
https://bugs.webkit.org/show_bug.cgi?id=113125
Summary testLeakingPrototypesAcrossContexts added in r146682 doesn't compile on Win a...
Ryosuke Niwa
Reported 2013-03-22 20:17:25 PDT
http://build.webkit.org/builders/Apple%20Win%20Release%20%28Build%29/builds/46467/steps/compile-webkit/logs/stdio 8>####### COMPILING 1 FILES USING AT MOST 8 PARALLEL INSTANCES OF cl.exe ########### 8>testapi.c 8>..\..\API\tests\testapi.c(146) : error C2332: 'class' : missing tag name 8>..\..\API\tests\testapi.c(146) : error C2236: unexpected 'class' '<unnamed-tag>'. Did you forget a ';'? 8>..\..\API\tests\testapi.c(149) : error C2332: 'class' : missing tag name 8>..\..\API\tests\testapi.c(149) : error C2226: syntax error : unexpected type '<unnamed-tag>' 8>..\..\API\tests\testapi.c(156) : error C2332: 'class' : missing tag name 8>..\..\API\tests\testapi.c(156) : error C2226: syntax error : unexpected type '<unnamed-tag>' 8>..\..\API\tests\testapi.c(169) : error C2664: 'nestedAllocateObject' : cannot convert parameter 2 from 'JSClassRef' to '' 8> No constructor could take the source type, or constructor overload resolution was ambiguous I've temporarily if-defed the test in http://trac.webkit.org/changeset/146707.
Attachments
Removes the test for now (3.27 KB, patch)
2013-03-22 21:16 PDT, Ryosuke Niwa
mhahnenberg: review+
Ryosuke Niwa
Comment 1 2013-03-22 21:14:29 PDT
It seems like this test also fails on Mac: http://build.webkit.org/builders/Apple%20MountainLion%20Release%20WK2%20%28Tests%29/builds/7233 FAIL: Failed to finalize the original object after the first GC. PASS: Infinite prototype chain does not occur. PASS: A cycle in a prototype chain can't be created. FAIL: Some tests failed.
Ryosuke Niwa
Comment 2 2013-03-22 21:16:38 PDT
Created attachment 194689 [details] Removes the test for now
Mark Hahnenberg
Comment 3 2013-03-22 21:17:20 PDT
Comment on attachment 194689 [details] Removes the test for now r=me
Ryosuke Niwa
Comment 4 2013-03-22 21:18:58 PDT
Thanks. Let me make sure it builds and land it.
Ryosuke Niwa
Comment 5 2013-03-22 21:22:39 PDT
Note You need to log in before you can comment on or make changes to this bug.