See http://test-results.appspot.com/dashboards/flakiness_dashboard.html#showExpectations=true&tests=fast%2Fdom%2Ferror-to-string-stack-overflow.html%2Csvg%2Fcustom%2Ftext-ctm.svg&builder=Webkit%20Win It looks like the v8 roll in http://codereview.chromium.org/9950056 caused this, so I'm going to roll it out. I apologize for the lack of notification; it looks like this has been failing all day but we didn't have a webkit gardener so no one noticed. -- Dirk
The difference between the expected(-) and the actual(+) in Linux 64-bit: -CONSOLE MESSAGE: Uncaught RangeError: Maximum call stack size exceeded +CONSOLE MESSAGE: line 14: Uncaught RangeError: Maximum call stack size exceeded Regression test for We started reporting the line number with V8 3.10.0.2. Should we adjust the test expectations for Linux 64-bit (platform/chromium-linux/)? Note that the test expectations for Linux 32-bit already have the correct line number: CONSOLE MESSAGE: line 14: Uncaught RangeError: Maximum call stack size exceeded
Created attachment 135295 [details] Adjust test expectation to include the line number of the exception. This can be landed after V8 3.10.0.3 roll.
Created attachment 135300 [details] Update LayoutTests/platform/chromium/test_expectations.txt to ignore the test Looks like the correct sequence is to land this patch with updated LayoutTests/platform/chromium/test_expectations.txt and then roll V8. Could someone please review and land this patch for me?
Comment on attachment 135300 [details] Update LayoutTests/platform/chromium/test_expectations.txt to ignore the test Looks good!
Comment on attachment 135300 [details] Update LayoutTests/platform/chromium/test_expectations.txt to ignore the test Rejecting attachment 135300 [details] from commit-queue. Failed to run "['/mnt/git/webkit-commit-queue/Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '-..." exit_code: 2 Last 500 characters of output: queue/Source/WebKit/chromium/third_party/sfntly/cpp/src --revision 128 --non-interactive --force --accept theirs-conflict --ignore-externals' in '/mnt/git/webkit-commit-queue/Source/WebKit/chromium' 42>At revision 128. ________ running '/usr/bin/python tools/clang/scripts/update.py --mac-only' in '/mnt/git/webkit-commit-queue/Source/WebKit/chromium' ________ running '/usr/bin/python gyp_webkit' in '/mnt/git/webkit-commit-queue/Source/WebKit/chromium' Updating webkit projects from gyp files... Full output: http://queues.webkit.org/results/12321126
Would you please rebase your patch with the latest WebKit trunk and then re-upload it, just in case?
Created attachment 135317 [details] Rebased Rebased to avoid merge conflict.
Comment on attachment 135317 [details] Rebased Clearing flags on attachment: 135317 Committed r113029: <http://trac.webkit.org/changeset/113029>
All reviewed patches have been landed. Closing bug.
Oh, I'm sorry I rolled this out just for a rebaseline :(. If I had been sharper last night I would've noticed. Glad it's fixed now, and sorry for the inconvenience!
(In reply to comment #10) > Oh, I'm sorry I rolled this out just for a rebaseline :(. If I had been sharper last night I would've noticed. Glad it's fixed now, and sorry for the inconvenience! No problem, we should have updated the expectations before rolling in.