Support multiple displays in EventGenerator

Support multiple displays in EventGenerator
- Introduces EventGeneratorDelegates interface that abstracts coordinates
- Added AshEventGeneratorDelegates that uses screen
   coordinates in ash environment.
- Updates several ash tests to use new EventGenerator

I'll convert other tests in separate CL.

BUG=139024
TEST=covered by tests.

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=176392

[Ben Goodger (Google), 2013-01-09 17:21:52]

lgtm

https://codereview.chromium.org/11691010/diff/35009/ash/extended_desktop_unit...
File ash/extended_desktop_unittest.cc (right):

https://codereview.chromium.org/11691010/diff/35009/ash/extended_desktop_unit...
ash/extended_desktop_unittest.cc:90: explicit
EventLocationRecordingEventHandler() {
not explicit

https://codereview.chromium.org/11691010/diff/35009/ash/extended_desktop_unit...
ash/extended_desktop_unittest.cc:112: void reset() {
Reset

https://codereview.chromium.org/11691010/diff/35009/ash/extended_desktop_unit...
ash/extended_desktop_unittest.cc:191: aura::test::EventGenerator&
event_generator(GetEventGenerator());
looks a little less weird as event_generator = GetEventGenerator();

I'm more used to seeing the parens used for ctors.

[oshima, 2013-01-10 06:13:05]

trybot failure turns out to be a real bug in aura code. I'll fix the bug first
and land this later.

https://codereview.chromium.org/11691010/diff/35009/ash/extended_desktop_unit...
File ash/extended_desktop_unittest.cc (right):

https://codereview.chromium.org/11691010/diff/35009/ash/extended_desktop_unit...
ash/extended_desktop_unittest.cc:90: explicit
EventLocationRecordingEventHandler() {
On 2013/01/09 17:21:52, Ben Goodger (Google) wrote:
> not explicit

Done.

https://codereview.chromium.org/11691010/diff/35009/ash/extended_desktop_unit...
ash/extended_desktop_unittest.cc:112: void reset() {
On 2013/01/09 17:21:52, Ben Goodger (Google) wrote:
> Reset

Done.

https://codereview.chromium.org/11691010/diff/35009/ash/extended_desktop_unit...
ash/extended_desktop_unittest.cc:191: aura::test::EventGenerator&
event_generator(GetEventGenerator());
On 2013/01/09 17:21:52, Ben Goodger (Google) wrote:
> looks a little less weird as event_generator = GetEventGenerator();
> 
> I'm more used to seeing the parens used for ctors.

Done.

[commit-bot: I haz the power, 2013-01-11 16:28:40]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/[email protected]/11691010/35009

[commit-bot: I haz the power, 2013-01-11 18:35:45]

Change committed as 176392