Missing director.js dependencies must be added back in.
Created attachment 310361 [details] Patch
Comment on attachment 310361 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=310361&action=review r=me, but consider using director.min.js. > PerformanceTests/Speedometer/resources/todomvc/architecture-examples/react/index.html:17 > + <script src="node_modules/director/director.js"></script> Don't you want to use director.min.js here to match other resources?
Created attachment 310401 [details] Patch
Comment on attachment 310401 [details] Patch Missing the added files, so the ChangeLog entry is wrong.
(In reply to David Kilzer (:ddkilzer) from comment #4) > Comment on attachment 310401 [details] > Patch > > Missing the added files, so the ChangeLog entry is wrong. And I assume you want to add the files still?
Yes. Please excuse the missing file - unsure what is dropping `build` directories locally (checked gitignores). I'm going to update this and double check the dependency is definitely in there.
Created attachment 310582 [details] Patch
Comment on attachment 310582 [details] Patch r=me.
Comment on attachment 310582 [details] Patch Clearing flags on attachment: 310582 Committed r217088: <http://trac.webkit.org/changeset/217088>
All reviewed patches have been landed. Closing bug.