78045 2012-02-07 14:51:41 -0800 Crash on http://www.rickshawbags.com/ 2012-02-07 15:46:19 -0800 1 1 1 Unclassified WebKit JavaScriptCore 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED http://www.rickshawbags.com/customize/custom-bag#!thl=rickshaw/bag() InRadar P2 Normal --- 1 barraclough barraclough oldest_to_newest 551419 0 barraclough 2012-02-07 14:51:41 -0800 This is a bug introduced by https://bugs.webkit.org/show_bug.cgi?id=71933, isVariableObject() checks were excluding StaticScopeObjects, this patch inadvertently changed them to be included. 551425 1 barraclough 2012-02-07 14:53:06 -0800 <rdar://problem/10697118> 551428 2 125938 barraclough 2012-02-07 14:56:58 -0800 Created attachment 125938 Fix 551446 3 125938 darin 2012-02-07 15:13:15 -0800 Comment on attachment 125938 Fix View in context: https://bugs.webkit.org/attachment.cgi?id=125938&action=review > Source/JavaScriptCore/ChangeLog:8 > + Problem URL is: http://www.rickshawbags.com/customize/custom-bag#!thl=rickshaw/bag() Can we make an automated regression test out of this? 551486 4 barraclough 2012-02-07 15:46:19 -0800 (In reply to comment #3) > Can we make an automated regression test out of this? Good question – I don't yet know. This patch just reverts an undesired change in behavior, I'm still investigating exactly why this change resulted in the bug that it did. Fixed in r106999. 125938 2012-02-07 14:56:58 -0800 2012-02-07 15:13:15 -0800 Fix 78045.patch text/plain 1863 barraclough SW5kZXg6IFNvdXJjZS9KYXZhU2NyaXB0Q29yZS9DaGFuZ2VMb2cKPT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291 cmNlL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkocmV2aXNpb24gMTA2OTkyKQorKysgU291cmNl L0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDIwIEBA CisyMDEyLTAyLTA3ICBHYXZpbiBCYXJyYWNsb3VnaCAgPGJhcnJhY2xvdWdoQGFwcGxlLmNvbT4K KworICAgICAgICBDcmFzaCBvbiBodHRwOi8vd3d3LnJpY2tzaGF3YmFncy5jb20vCisgICAgICAg IGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD03ODA0NQorCisgICAgICAg IFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIFByb2JsZW0gVVJMIGlzOiBo dHRwOi8vd3d3LnJpY2tzaGF3YmFncy5jb20vY3VzdG9taXplL2N1c3RvbS1iYWcjIXRobD1yaWNr c2hhdy9iYWcoKQorICAgICAgICAKKyAgICAgICAgVGhpcyBpcyBhIGJ1ZyBpbnRyb2R1Y2VkIGJ5 IGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD03MTkzMywKKyAgICAgICAg aXNWYXJpYWJsZU9iamVjdCgpIGNoZWNrcyB3ZXJlIGV4Y2x1ZGluZyBTdGF0aWNTY29wZU9iamVj dHMsIHRoaXMgcGF0Y2gKKyAgICAgICAgaW5hZHZlcnRlbnRseSBjaGFuZ2VkIHRoZW0gdG8gYmUg aW5jbHVkZWQuCisKKyAgICAgICAgKiBydW50aW1lL0pTVHlwZS5oOgorICAgICAgICAgICAgLSBz b3J0IEpTVHlwZSBlbnVtIHN1Y2ggdGhhdCBTdGF0aWNTY29wZU9iamVjdFR5cGUgY29tZXMgYmVm b3JlIFZhcmlhYmxlT2JqZWN0VHlwZSwKKyAgICAgICAgICAgICAgYW5kIHRodXMgaXMgZXhjbHVk ZWQgZnJvbSBpc1ZhcmlhYmxlT2JqZWN0KCkgY2hlY2tzLgorCiAyMDEyLTAyLTA2ICBKZXIgTm9i bGUgIDxqZXIubm9ibGVAYXBwbGUuY29tPgogCiAgICAgICAgIFVzZSBDTUNsb2NrIGFzIGEgdGlt aW5nIHNvdXJjZSBmb3IgUGxhdGZvcm1DbG9jayB3aGVyZSBhdmFpbGFibGUuCkluZGV4OiBTb3Vy Y2UvSmF2YVNjcmlwdENvcmUvcnVudGltZS9KU1R5cGUuaAo9PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2Uv SmF2YVNjcmlwdENvcmUvcnVudGltZS9KU1R5cGUuaAkocmV2aXNpb24gMTA2ODYzKQorKysgU291 cmNlL0phdmFTY3JpcHRDb3JlL3J1bnRpbWUvSlNUeXBlLmgJKHdvcmtpbmcgY29weSkKQEAgLTQ5 LDExICs0OSwxMiBAQCBlbnVtIEpTVHlwZSB7CiAgICAgRXJyb3JJbnN0YW5jZVR5cGUsCiAgICAg R2xvYmFsVGhpc1R5cGUsCiAKLSAgICAvLyBWYXJpYWJsZU9iamVjdFR5cGUgbXVzdCBjb21lIGJl Zm9yZSBhbGwgb2YgdGhlIHR5cGVzIG9mIGl0cyBzdWJjbGFzc2VzIGFuZCBvbmx5IGl0cyBzdWJj bGFzc2VzLgorICAgIFN0YXRpY1Njb3BlT2JqZWN0VHlwZSwKKyAgICAvLyBWYXJpYWJsZU9iamVj dFR5cGUgbXVzdCBiZSBsZXNzIHRoYW4gTU9TVCBvZiB0aGUgdHlwZXMgb2YgaXRzIHN1YmNsYXNz ZXMgYW5kIG9ubHkgaXRzIHN1YmNsYXNzZXMuCisgICAgLy8gV2UgdXNlID49VmFyaWFibGVPYmpl Y3RUeXBlIGNoZWNrcyB0byB0ZXN0IGZvciBHbG9iYWwgJiBBY3RpdmF0aW9uIG9iamVjdHMsIGJ1 dCBleGNsdWRlIFN0YXRpY1Njb3Blcy4KICAgICBWYXJpYWJsZU9iamVjdFR5cGUsCiAgICAgR2xv YmFsT2JqZWN0VHlwZSwKICAgICBBY3RpdmF0aW9uT2JqZWN0VHlwZSwKLSAgICBTdGF0aWNTY29w ZU9iamVjdFR5cGUsCiB9OwogCiB9IC8vIG5hbWVzcGFjZSBKU0MK