mash: Fix system tray bubble not closing for clicks outside its bounds

mash: Fix system tray bubble not closing for clicks outside its bounds

* Event observers need to convert pointer down events to mouse down events,
just like they do for normal window input events.
* Convert the event observer unit tests to simulate pointer events being sent
from the window server, which is a better representation of how the system
works in production.

BUG=631126
TEST=mus_public_unittests

[James Cook, 2016-07-25 18:06:48]

jamescook@chromium.org changed reviewers:
+ moshayedi@chromium.org

[James Cook, 2016-07-25 18:06:48]

moshayedi, please take a look.

[Hadi, 2016-07-25 19:01:42]

https://codereview.chromium.org/2179083002/diff/1/services/ui/public/cpp/lib/...
File services/ui/public/cpp/lib/window_tree_client.cc (right):

https://codereview.chromium.org/2179083002/diff/1/services/ui/public/cpp/lib/...
services/ui/public/cpp/lib/window_tree_client.cc:602: nullptr /* target */);
Why are we ignoring the |target| function parameter and passing nullptr as
target?

Same question for other branches in this function.

[James Cook, 2016-07-25 20:23:37]

moshayedi, please take another look.

https://codereview.chromium.org/2179083002/diff/1/services/ui/public/cpp/lib/...
File services/ui/public/cpp/lib/window_tree_client.cc (right):

https://codereview.chromium.org/2179083002/diff/1/services/ui/public/cpp/lib/...
services/ui/public/cpp/lib/window_tree_client.cc:602: nullptr /* target */);
On 2016/07/25 19:01:42, Hadi wrote:
> Why are we ignoring the |target| function parameter and passing nullptr as
> target?
> 
> Same question for other branches in this function.

Good catch - that was a mistake. Fixed it and extended the tests to check for
target window.

[James Cook, 2016-07-25 20:23:48]

The CQ bit was checked by jamescook@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-07-25 20:24:31]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-25 21:43:45]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-07-25 21:43:46]

Dry run: This issue passed the CQ dry run.

[Hadi, 2016-07-26 14:00:30]

lgtm

[James Cook, 2016-07-26 16:02:47]

jamescook@chromium.org changed reviewers:
+ msw@chromium.org

[James Cook, 2016-07-26 16:02:48]

msw, can I get OWNERS?

[James Cook, 2016-07-26 16:04:52]

The CQ bit was checked by jamescook@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-07-26 16:05:10]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-26 16:45:26]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-07-26 16:45:27]

Dry run: This issue passed the CQ dry run.

[James Cook, 2016-07-26 16:48:23]

On 2016/07/26 16:02:48, James Cook wrote:
> msw, can I get OWNERS?

Nevermind. Abandoning this patch per discussion in
https://bugs.chromium.org/p/chromium/issues/detail?id=631126 -- PointerWatchers
should get pointer events.