42802 2010-07-21 19:29:16 -0700 Temporarily disable the layout test for the bug 41175 2010-07-27 17:02:33 -0700 1 1 1 Unclassified WebKit SVG 528+ (Nightly build) Mac OS X 10.5 RESOLVED FIXED P3 Normal --- 0 ctruta webkit-unassigned abarth commit-queue eric krit ojan zimmermann oldest_to_newest 254632 0 ctruta 2010-07-21 19:29:16 -0700 The fix for the bug 41175 is innocuous; however, the layout test added is failing in debug-mode WebKit on 10.5. Add an exclusion for that layout test, until a proper fix is developed. 254633 1 62255 ctruta 2010-07-21 19:44:40 -0700 Created attachment 62255 Add an exclusion for the layout test for 41175 under OS X 10.5, until the real problem is fixed 254635 2 62255 eric 2010-07-21 19:59:11 -0700 Comment on attachment 62255 Add an exclusion for the layout test for 41175 under OS X 10.5, until the real problem is fixed Every change requires a ChangeLog entry. :( 254912 3 62325 ctruta 2010-07-22 11:57:36 -0700 Created attachment 62325 Excluding the layout test for 41175; including the ChangeLog entry 254915 4 62325 eric 2010-07-22 12:05:19 -0700 Comment on attachment 62325 Excluding the layout test for 41175; including the ChangeLog entry https://bugs.webkit.org/show_bug.cgi?id=41808 does not explain the bug at all. It talks about Chromium, yet this is disabling a test on Mac. I do not understand. Please file a bug about the ASSERT and link to it from your bug and skipped list. Or re-purpose https://bugs.webkit.org/show_bug.cgi?id=41808 to be about the ASSERT itself (with information about what assert and a reduced test case, etc.) This patch is fine, but the course forward is not yet. 254934 5 62325 darin 2010-07-22 13:03:16 -0700 Comment on attachment 62325 Excluding the layout test for 41175; including the ChangeLog entry This will disable the test for both Leopard and Tiger. But I suppose it’s better than having the bot fail for days. 255129 6 62325 commit-queue 2010-07-22 18:24:19 -0700 Comment on attachment 62325 Excluding the layout test for 41175; including the ChangeLog entry Clearing flags on attachment: 62325 Committed r63935: <http://trac.webkit.org/changeset/63935> 255130 7 commit-queue 2010-07-22 18:24:25 -0700 All reviewed patches have been landed. Closing bug. 256943 8 ctruta 2010-07-27 17:02:33 -0700 The bug 43069 has been opened. That will address the issue of filtering zero-sized <g> elements at a higher, platform-independent level, as opposed to using platform-specific fixes and workarounds. That should hopefully resolve this issue as well. 62255 2010-07-21 19:44:40 -0700 2010-07-22 11:57:36 -0700 Add an exclusion for the layout test for 41175 under OS X 10.5, until the real problem is fixed wk42802.submit text/plain 575 ctruta ZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL3BsYXRmb3JtL21hYy1sZW9wYXJkL1NraXBwZWQgYi9M YXlvdXRUZXN0cy9wbGF0Zm9ybS9tYWMtbGVvcGFyZC9Ta2lwcGVkCmluZGV4IDcxMDU1ZDUuLmMw MDJmMzMgMTAwNjQ0Ci0tLSBhL0xheW91dFRlc3RzL3BsYXRmb3JtL21hYy1sZW9wYXJkL1NraXBw ZWQKKysrIGIvTGF5b3V0VGVzdHMvcGxhdGZvcm0vbWFjLWxlb3BhcmQvU2tpcHBlZApAQCAtNzcs MyArNzcsNyBAQCBzdG9yYWdlL2luZGV4ZWRkYgogIyBEaXNhYmxlIHRoZSB0ZXN0cyB1bnRpbCB0 aGUgbWVzYSByZW5kZXJlciBpcyBpbnRlZ3JhdGVkLgogIyBodHRwczovL2J1Z3Mud2Via2l0Lm9y Zy9zaG93X2J1Zy5jZ2k/aWQ9NDE1NDUKIGZhc3QvY2FudmFzL3dlYmdsL3RleHR1cmUtbnBvdC5o dG1sCisKKyMgRGlzYWJsZSBhbiBTVkcgdGVzdCB0aGF0IGNhdXNlcyBhbiBhc3NlcnRpb24gZmFp bHVyZSBpbiBkZWJ1Zy1tb2RlIFdlYktpdCB1bmRlciAxMC41LgorIyBodHRwczovL2J1Z3Mud2Vi a2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NDE4MDgKK3N2Zy9maWx0ZXJzL2ZpbHRlci1lbXB0eS1n LnN2Zwo= 62325 2010-07-22 11:57:36 -0700 2010-07-22 18:24:19 -0700 Excluding the layout test for 41175; including the ChangeLog entry wk42802.submit text/plain 1131 ctruta ZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL0NoYW5nZUxvZyBiL0xheW91dFRlc3RzL0NoYW5nZUxv ZwppbmRleCA0M2NhYjFkLi5kYWYxYWQ0IDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0cy9DaGFuZ2VM b2cKKysrIGIvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTIgQEAKKzIwMTAtMDct MjIgIENvc21pbiBUcnV0YSAgPGN0cnV0YUBjaHJvbWl1bS5vcmc+CisKKyAgICAgICAgUmV2aWV3 ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgU1ZHOiBUZW1wb3JhcmlseSBkaXNhYmxl IHRoZSBsYXlvdXQgdGVzdCBmb3IgdGhlIGJ1ZyA0MTE3NSBvbiBMZW9wYXJkLgorICAgICAgICBo dHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NDI4MDIKKworICAgICAgICAq IHBsYXRmb3JtL21hYy1sZW9wYXJkL1NraXBwZWQ6IHN2Zy9maWx0ZXJzL2ZpbHRlci1lbXB0eS1n LnN2ZyBza2lwcGVkIHVudGlsIGZpeC4KKwogMjAxMC0wNy0yMiAgQ3NhYmEgT3N6dHJvZ29uw6Fj ICA8b3NzeUB3ZWJraXQub3JnPgogCiAgICAgICAgIFVucmV2aWV3ZWQuCmRpZmYgLS1naXQgYS9M YXlvdXRUZXN0cy9wbGF0Zm9ybS9tYWMtbGVvcGFyZC9Ta2lwcGVkIGIvTGF5b3V0VGVzdHMvcGxh dGZvcm0vbWFjLWxlb3BhcmQvU2tpcHBlZAppbmRleCA3MTA1NWQ1Li5jMDAyZjMzIDEwMDY0NAot LS0gYS9MYXlvdXRUZXN0cy9wbGF0Zm9ybS9tYWMtbGVvcGFyZC9Ta2lwcGVkCisrKyBiL0xheW91 dFRlc3RzL3BsYXRmb3JtL21hYy1sZW9wYXJkL1NraXBwZWQKQEAgLTc3LDMgKzc3LDcgQEAgc3Rv cmFnZS9pbmRleGVkZGIKICMgRGlzYWJsZSB0aGUgdGVzdHMgdW50aWwgdGhlIG1lc2EgcmVuZGVy ZXIgaXMgaW50ZWdyYXRlZC4KICMgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dp P2lkPTQxNTQ1CiBmYXN0L2NhbnZhcy93ZWJnbC90ZXh0dXJlLW5wb3QuaHRtbAorCisjIERpc2Fi bGUgYW4gU1ZHIHRlc3QgdGhhdCBjYXVzZXMgYW4gYXNzZXJ0aW9uIGZhaWx1cmUgaW4gZGVidWct bW9kZSBXZWJLaXQgdW5kZXIgMTAuNS4KKyMgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19i dWcuY2dpP2lkPTQxODA4CitzdmcvZmlsdGVycy9maWx0ZXItZW1wdHktZy5zdmcK