Refactor common Pointer Lock test code to a harness.
Created attachment 151301 [details] Patch
Comment on attachment 151301 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=151301&action=review > LayoutTests/pointer-lock/resources/pointer-lock-test-harness.js:5 > + doNextStepButton.innerText = "doNextStep button for manual testing. Use keyboard to select and press (TAB, then SPACE) when pointer locked."; What about only adding the button or the text if there isn't a window.testRunner, so that this text doesn't get baked into every expectation file?
Created attachment 151311 [details] Patch
Comment on attachment 151301 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=151301&action=review >> LayoutTests/pointer-lock/resources/pointer-lock-test-harness.js:5 >> + doNextStepButton.innerText = "doNextStep button for manual testing. Use keyboard to select and press (TAB, then SPACE) when pointer locked."; > > What about only adding the button or the text if there isn't a window.testRunner, so that this text doesn't get baked into every expectation file? Done.
Comment on attachment 151311 [details] Patch R=me.
Committed r122158: <http://trac.webkit.org/changeset/122158>