Patch forthcoming.
Created attachment 282318 [details] the patch
Comment on attachment 282318 [details] the patch View in context: https://bugs.webkit.org/attachment.cgi?id=282318&action=review > PerformanceTests/ES6SampleBench/driver.js:152 > + magicFrame.contentDocument.write( > + "<!DOCTYPE html><head><title>benchmark payload</title></head><body><script>\n" + > + "window.onerror = top." + this._key + ".reportError;\n" + > + "function reportResult()\n" + > + "{\n" + > + " var driver = top." + this._key + ";\n" + > + " driver.reportResult.apply(driver, arguments);\n" + > + "}\n" + > + "</script>\n" + > + this._benchmark.code + "</body></html>"); Nit: This would be easier to read as a template literal
(In reply to comment #2) > Comment on attachment 282318 [details] > the patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=282318&action=review > > > PerformanceTests/ES6SampleBench/driver.js:152 > > + magicFrame.contentDocument.write( > > + "<!DOCTYPE html><head><title>benchmark payload</title></head><body><script>\n" + > > + "window.onerror = top." + this._key + ".reportError;\n" + > > + "function reportResult()\n" + > > + "{\n" + > > + " var driver = top." + this._key + ";\n" + > > + " driver.reportResult.apply(driver, arguments);\n" + > > + "}\n" + > > + "</script>\n" + > > + this._benchmark.code + "</body></html>"); > > Nit: This would be easier to read as a template literal Good idea! Fixed.
Landed in https://trac.webkit.org/changeset/202613