87423 2012-05-24 13:42:06 -0700 [Chromium] [V8] Layout Test fast/js/string-replace-2.html failing following 11594 => 11648 v8 update 2012-05-28 09:06:59 -0700 1 1 1 Unclassified WebKit Tools / Tests 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 1 jsbell bulach bulach danakj danno jkummerow vsevik oldest_to_newest 633163 0 jsbell 2012-05-24 13:42:06 -0700 Layout Test is failing (assertion) on DEBUG Mac/Win/Linux fast/js/string-replace-2.html Flakiness dashboard: http://test-results.appspot.com/dashboards/flakiness_dashboard.html#showExpectations=true&tests=fast%2Fjs%2Fstring-replace-2.html Assert hit: STDERR: # Fatal error in /Volumes/data/b/build/slave/webkit-mac-latest-dbg/build/src/v8/tools/gyp/../../src/factory.cc, line 261 STDERR: # CHECK(begin > 0 || end < str->length()) failed 633176 1 jsbell 2012-05-24 13:50:41 -0700 Note that the test is currently "expected" to fail, see http://webkit.org/b/80665 But the crashing behavior is obviously new. 633189 2 jsbell 2012-05-24 13:55:59 -0700 Added expectation for this in WebKit r118412 but we should either roll back Chromium's V8 or fix this ASAP. Assuming the fix is straightforward and only debug is failing, letting this go for a bit. 634336 3 jsbell 2012-05-25 15:23:18 -0700 We can remove this as soon as http://codereview.chromium.org/10443033 lands in Chromium 635271 4 144375 bulach 2012-05-28 09:03:34 -0700 Created attachment 144375 Patch 635272 5 bulach 2012-05-28 09:05:53 -0700 Committed r118687: <http://trac.webkit.org/changeset/118687> 635273 6 bulach 2012-05-28 09:06:59 -0700 We rolled past http://codereview.chromium.org/10443033, resetting the expectation to TEXT. 144375 2012-05-28 09:03:34 -0700 2012-05-28 09:03:34 -0700 Patch bug-87423-20120528170333.patch text/plain 1580 bulach ZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL0NoYW5nZUxvZyBiL0xheW91dFRlc3RzL0NoYW5nZUxv ZwppbmRleCA3NmIwYzU4YjYyNGFmMjY2YWJlMWJmMTU1YzI5YzRhNTc1ZmY2NjY0Li5mN2I5MDky YjUyMDVlMmY5N2E4NzcyOTg4NWIwMDRhMzkzNjE2NGMxIDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0 cy9DaGFuZ2VMb2cKKysrIGIvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTQgQEAK KzIwMTItMDUtMjggIE1hcmN1cyBCdWxhY2ggIDxidWxhY2hAY2hyb21pdW0ub3JnPgorCisgICAg ICAgIFtDaHJvbWl1bV0gW1Y4XSBMYXlvdXQgVGVzdCBmYXN0L2pzL3N0cmluZy1yZXBsYWNlLTIu aHRtbCBmYWlsaW5nIGZvbGxvd2luZyAxMTU5NCA9PiAxMTY0OCB2OCB1cGRhdGUKKyAgICAgICAg aHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTg3NDIzCisKKyAgICAgICAg VW5yZXZpZXdlZCBnYXJkZW5pbmcuCisgICAgICAgIAorICAgICAgICBXZSBoYXZlIHJvbGxlZCB2 OCBhbmQgd2UgY2FuIG5vdyBzZXQgdGhlIGV4cGVjdGF0aW9uIGZvciBib3RoIGRlYnVnIGFuZCBy ZWxlYXNlIGJhY2sgdG8gVEVYVC4KKworICAgICAgICAqIHBsYXRmb3JtL2Nocm9taXVtL3Rlc3Rf ZXhwZWN0YXRpb25zLnR4dDoKKwogMjAxMi0wNS0yOCAgUGV0ZXIgUnliaW4gIDxwZXRlci5yeWJp bkBnbWFpbC5jb20+CiAKICAgICAgICAgV2ViIEluc3BlY3RvcjogRXhwb3NlIGZ1bmN0aW9uIChj bG9zdXJlKSBzY29wZXMgaW4gcmVtb3RlIHByb3RvY29sCmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0 cy9wbGF0Zm9ybS9jaHJvbWl1bS90ZXN0X2V4cGVjdGF0aW9ucy50eHQgYi9MYXlvdXRUZXN0cy9w bGF0Zm9ybS9jaHJvbWl1bS90ZXN0X2V4cGVjdGF0aW9ucy50eHQKaW5kZXggNzk5OTgxOGQ1Yjc0 YzU3ZDM2MmY3Yzc2YjllNWJhNDBkYzU5NGJjNi4uYmNkMjNkNDJjNzJjZWIzMDJlMzQ2NDQ1YmFi YTRhYzIzNzVhMDQxZCAxMDA2NDQKLS0tIGEvTGF5b3V0VGVzdHMvcGxhdGZvcm0vY2hyb21pdW0v dGVzdF9leHBlY3RhdGlvbnMudHh0CisrKyBiL0xheW91dFRlc3RzL3BsYXRmb3JtL2Nocm9taXVt L3Rlc3RfZXhwZWN0YXRpb25zLnR4dApAQCAtMzIyMCw4ICszMjIwLDcgQEAgQlVHV0s4Mzk1NCBN QUMgREVCVUcgOiBmYXN0L3RleHQvdGV4dEl0ZXJhdG9yTmlsUmVuZGVyZXIuaHRtbCA9IElNQUdF IFBBU1MKIEJVR1dLODA1MzEgOiBmYXN0L3JlcGFpbnQvc2VhcmNoLWZpZWxkLWNhbmNlbC5odG1s ID0gVEVYVCBJTUFHRStURVhUIElNQUdFCiAKIEJVR1dLODA2NjUgOiBmYXN0L2pzL3N0cmluZy1t YXRjaC5odG1sID0gVEVYVAotQlVHV0s4MDY2NSBSRUxFQVNFIDogZmFzdC9qcy9zdHJpbmctcmVw bGFjZS0yLmh0bWwgPSBURVhUCi1CVUdXSzg3NDIzIERFQlVHIDogZmFzdC9qcy9zdHJpbmctcmVw bGFjZS0yLmh0bWwgPSBDUkFTSAorQlVHV0s4MDY2NSA6IGZhc3QvanMvc3RyaW5nLXJlcGxhY2Ut Mi5odG1sID0gVEVYVAogCiBCVUdBQkFSVEggU05PV0xFT1BBUkQgOiBmYXN0L2V2ZW50cy9pbnB1 dC1pbWFnZS1zY3JvbGxlZC14LXkuaHRtbCA9IFBBU1MgQ1JBU0gKIAo=