Remove mouse events from Sky

Remove mouse events from Sky

We use pointer events now.

R=esprehn@chromium.org

Committed: https://chromium.googlesource.com/external/mojo/+/7e3ecedfcf3ed39a8bff3cbb67b14884bfb84f7a

[esprehn, 2015-01-23 20:15:56]

rs lgtm

[abarth-chromium, 2015-01-23 20:16:13]

Committed patchset #1 (id:1) manually as
7e3ecedfcf3ed39a8bff3cbb67b14884bfb84f7a (presubmit successful).

[ojan, 2015-01-23 22:23:06]

https://codereview.chromium.org/870073003/diff/1/sky/engine/core/rendering/Re...
File sky/engine/core/rendering/RenderLayerScrollableArea.cpp (left):

https://codereview.chromium.org/870073003/diff/1/sky/engine/core/rendering/Re...
sky/engine/core/rendering/RenderLayerScrollableArea.cpp:226:
frame->eventHandler().dispatchFakeMouseMoveEventSoonInQuad(quadForFakeMouseMoveEvent);
Have we removed :hover? If not, then this change makes is so that we don't
update hover state when you scroll (e.g. with the scroll wheel or keyboard).

[abarth-chromium, 2015-01-23 22:39:29]

https://codereview.chromium.org/870073003/diff/1/sky/engine/core/rendering/Re...
File sky/engine/core/rendering/RenderLayerScrollableArea.cpp (left):

https://codereview.chromium.org/870073003/diff/1/sky/engine/core/rendering/Re...
sky/engine/core/rendering/RenderLayerScrollableArea.cpp:226:
frame->eventHandler().dispatchFakeMouseMoveEventSoonInQuad(quadForFakeMouseMoveEvent);
On 2015/01/23 at 22:23:06, ojan wrote:
> Have we removed :hover? If not, then this change makes is so that we don't
update hover state when you scroll (e.g. with the scroll wheel or keyboard).

IMHO, we should remove hover for now.  We don't have any hardware that supports
it yet.

[ojan, 2015-01-24 05:54:21]

On 2015/01/23 at 22:39:29, abarth wrote:
>
https://codereview.chromium.org/870073003/diff/1/sky/engine/core/rendering/Re...
> File sky/engine/core/rendering/RenderLayerScrollableArea.cpp (left):
> 
>
https://codereview.chromium.org/870073003/diff/1/sky/engine/core/rendering/Re...
> sky/engine/core/rendering/RenderLayerScrollableArea.cpp:226:
frame->eventHandler().dispatchFakeMouseMoveEventSoonInQuad(quadForFakeMouseMoveEvent);
> On 2015/01/23 at 22:23:06, ojan wrote:
> > Have we removed :hover? If not, then this change makes is so that we don't
update hover state when you scroll (e.g. with the scroll wheel or keyboard).
> 
> IMHO, we should remove hover for now.  We don't have any hardware that
supports it yet.

Well, you can plug a mouse into an android device and that's an increasingly
common/important use case. But, removing it is fine with me. Adding it back in
when we're closer to shipping a product shouldn't be an enormous amount of work.
I thought we were leaving it in because we didn't want demos to be totally
broken if you plugged a mouse in. esprehn had told me that you had said to leave
it in for that reason. :) esprehn, removing hover OK with you?