WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
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+
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
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
Committed
r146711
: <
http://trac.webkit.org/changeset/146711
>
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug