Bug 189491 – Update webkit-webgl-test-harness.js for more details on WebGL 2 conformance tests part 3

• Status: RESOLVED
• Resolution: FIXED
• Component: New Bugs

[Justin Fan]

Update webkit-webgl-test-harness.js for more details on WebGL 2 conformance tests part 3

[Radar WebKit Bug Importer]

<rdar://problem/44326194>

[Justin Fan]

Created attachment 349368 [details]
Patch

[Dean Jackson]

Comment on attachment 349368 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=349368&action=review

> LayoutTests/webgl/2.0.0/resources/webkit-webgl-test-harness.js:53
> +        list("[ " + resultNum + ": PASS ] " + msg, "green");

The modern JS way to do these strings is:

`[ ${resultNum}: PASS ] ${msg}`

[Justin Fan]

Created attachment 349428 [details]
Patch

[Justin Fan]

(In reply to Dean Jackson from comment #3)
> Comment on attachment 349368 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=349368&action=review
> 
> > LayoutTests/webgl/2.0.0/resources/webkit-webgl-test-harness.js:53
> > +        list("[ " + resultNum + ": PASS ] " + msg, "green");
> 
> The modern JS way to do these strings is:
> 
> `[ ${resultNum}: PASS ] ${msg}`

Cool, Thanks!

[Justin Fan]

Created attachment 349430 [details]
Patch

[WebKit Commit Bot]

Comment on attachment 349430 [details]
Patch

Clearing flags on attachment: 349430

Committed r235913: <https://trac.webkit.org/changeset/235913>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.