Send release events as well as press events in linux event mocking infrastruc...

Send release events as well as press events in linux event mocking infrastructure.

Also send events for the modifier keys. This matches Windows more closely. I needed this for an test I was writing which I decided to throw away as it was using the wrong approach.


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=24650

[sky, 2009-08-27 19:24:18]

LGTM

http://codereview.chromium.org/178002/diff/1/2
File chrome/browser/automation/ui_controls_linux.cc (right):

http://codereview.chromium.org/178002/diff/1/2#newcode32
Line 32: : task_(task), type_(type), count_(count) {
nit: the style guide says once you wrap, you need to put each member on its own
line.

http://codereview.chromium.org/178002/diff/1/2#newcode42
Line 42: if (event->type == type_) {
nit: I would combine this into a single line:
if (event->type == type && --count_ == 0) {

[Evan Stade, 2009-08-27 19:50:52]

done and done