WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
122286
Add a testcase for a bug that I had during the development. This code uses the asm.js style.
https://bugs.webkit.org/show_bug.cgi?id=122286
Summary
Add a testcase for a bug that I had during the development. This code uses th...
Nadav Rotem
Reported
2013-10-03 10:51:53 PDT
Add a testcase for a bug that I had during the development. This code uses the asm.js style.
Attachments
Patch
(2.83 KB, patch)
2013-10-03 10:52 PDT
,
Nadav Rotem
no flags
Details
Formatted Diff
Diff
Archive of layout-test-results from webkit-cq-02 for mac-mountainlion
(460.29 KB, application/zip)
2013-10-03 12:28 PDT
,
WebKit Commit Bot
no flags
Details
Archive of layout-test-results from webkit-ews-03 for mac-mountainlion
(647.08 KB, application/zip)
2013-10-03 12:31 PDT
,
Build Bot
no flags
Details
Archive of layout-test-results from webkit-ews-08 for mac-mountainlion
(478.46 KB, application/zip)
2013-10-03 13:29 PDT
,
Build Bot
no flags
Details
Archive of layout-test-results from webkit-ews-12 for mac-mountainlion-wk2
(487.15 KB, application/zip)
2013-10-03 14:22 PDT
,
Build Bot
no flags
Details
Patch
(2.82 KB, patch)
2013-10-03 14:32 PDT
,
Nadav Rotem
no flags
Details
Formatted Diff
Diff
Show Obsolete
(1)
View All
Add attachment
proposed patch, testcase, etc.
Nadav Rotem
Comment 1
2013-10-03 10:52:09 PDT
Created
attachment 213279
[details]
Patch
Nadav Rotem
Comment 2
2013-10-03 10:53:15 PDT
Adding a test case for a bug that I had during the development of the ValueToInt32 pattern. It looks like we don't have too many asm.js kind of programs.
Nadav Rotem
Comment 3
2013-10-03 11:23:19 PDT
To be clear Emscripten is available under 2 licenses: MIT and University of Illinois/NCSA Open Source Licenses. The strlen function is a part of the emscript library that is a part of all auto-generated files.
Filip Pizlo
Comment 4
2013-10-03 11:39:37 PDT
(In reply to
comment #3
)
> To be clear Emscripten is available under 2 licenses: MIT and University of Illinois/NCSA Open Source Licenses. The strlen function is a part of the emscript library that is a part of all auto-generated files.
I think we've already checked in emscripten code before so it should be fine! Also we tend to be more lax for the licenses for tests in LayoutTests since those don't get "shipped" in products. We have everything from Apache license to LGPL and probably multiple varieties of the BSD-style licenses in the different tests.
WebKit Commit Bot
Comment 5
2013-10-03 12:28:05 PDT
Comment on
attachment 213279
[details]
Patch Rejecting
attachment 213279
[details]
from commit-queue. New failing tests: js/regress/asmjs_bool_bug.html Full output:
http://webkit-queues.appspot.com/results/3152001
WebKit Commit Bot
Comment 6
2013-10-03 12:28:07 PDT
Created
attachment 213287
[details]
Archive of layout-test-results from webkit-cq-02 for mac-mountainlion The attached test failures were seen while running run-webkit-tests on the commit-queue. Bot: webkit-cq-02 Port: mac-mountainlion Platform: Mac OS X 10.8.4
Build Bot
Comment 7
2013-10-03 12:30:59 PDT
Comment on
attachment 213279
[details]
Patch
Attachment 213279
[details]
did not pass mac-ews (mac): Output:
http://webkit-queues.appspot.com/results/3144003
New failing tests: js/regress/asmjs_bool_bug.html svg/batik/filters/filterRegions.svg
Build Bot
Comment 8
2013-10-03 12:31:01 PDT
Created
attachment 213288
[details]
Archive of layout-test-results from webkit-ews-03 for mac-mountainlion The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: webkit-ews-03 Port: mac-mountainlion Platform: Mac OS X 10.8.5
Build Bot
Comment 9
2013-10-03 13:28:59 PDT
Comment on
attachment 213279
[details]
Patch
Attachment 213279
[details]
did not pass mac-ews (mac): Output:
http://webkit-queues.appspot.com/results/3151023
New failing tests: js/regress/asmjs_bool_bug.html
Build Bot
Comment 10
2013-10-03 13:29:01 PDT
Created
attachment 213290
[details]
Archive of layout-test-results from webkit-ews-08 for mac-mountainlion The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: webkit-ews-08 Port: mac-mountainlion Platform: Mac OS X 10.8.5
Build Bot
Comment 11
2013-10-03 14:22:16 PDT
Comment on
attachment 213279
[details]
Patch
Attachment 213279
[details]
did not pass mac-wk2-ews (mac-wk2): Output:
http://webkit-queues.appspot.com/results/3148006
New failing tests: js/regress/asmjs_bool_bug.html
Build Bot
Comment 12
2013-10-03 14:22:18 PDT
Created
attachment 213296
[details]
Archive of layout-test-results from webkit-ews-12 for mac-mountainlion-wk2 The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews. Bot: webkit-ews-12 Port: mac-mountainlion-wk2 Platform: Mac OS X 10.8.5
Nadav Rotem
Comment 13
2013-10-03 14:32:51 PDT
Created
attachment 213299
[details]
Patch
Nadav Rotem
Comment 14
2013-10-03 14:33:57 PDT
I fixed the test name in the expected.txt file.
WebKit Commit Bot
Comment 15
2013-10-03 16:17:16 PDT
Comment on
attachment 213299
[details]
Patch Clearing flags on attachment: 213299 Committed
r156865
: <
http://trac.webkit.org/changeset/156865
>
WebKit Commit Bot
Comment 16
2013-10-03 16:17:20 PDT
All reviewed patches have been landed. Closing bug.
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