Simplify full-screen-test.js's runWithKeyDown()

Simplify full-screen-test.js's runWithKeyDown()

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=180373

[philipj_slow, 2014-08-14 08:40:23]

PTAL. I was mystified by this recently, before I understood most of it was dead
code.

[falken, 2014-08-15 13:21:35]

lgtm! yep looks like that was obsolete code

https://codereview.chromium.org/469203002/diff/1/LayoutTests/fullscreen/full-...
File LayoutTests/fullscreen/full-screen-test.js (right):

https://codereview.chromium.org/469203002/diff/1/LayoutTests/fullscreen/full-...
LayoutTests/fullscreen/full-screen-test.js:23: eventSender.keyDown(" ", []);
let's make this ' ' or the others " " for consistency

[philipj_slow, 2014-08-15 13:40:17]

https://codereview.chromium.org/469203002/diff/1/LayoutTests/fullscreen/full-...
File LayoutTests/fullscreen/full-screen-test.js (right):

https://codereview.chromium.org/469203002/diff/1/LayoutTests/fullscreen/full-...
LayoutTests/fullscreen/full-screen-test.js:23: eventSender.keyDown(" ", []);
On 2014/08/15 13:21:35, falken wrote:
> let's make this ' ' or the others " " for consistency

This file seems to use double quotes slightly more, so I'll go with that.

[philipj_slow, 2014-08-15 13:40:31]

quotes

[philipj_slow, 2014-08-15 13:40:52]

The CQ bit was checked by philipj@opera.com

[commit-bot: I haz the power, 2014-08-15 13:41:58]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/philipj@opera.com/469203002/20001

[commit-bot: I haz the power, 2014-08-15 15:41:37]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  linux_blink_dbg on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_dbg/builds/2...)
  win_blink_compile_dbg on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_compile_dbg/bu...)

[commit-bot: I haz the power, 2014-08-15 16:46:27]

Committed patchset #2 (20001) as 180373