[Ozone-Drm] Notify cursor of channel established last

[Ozone-Drm] Notify cursor of channel established last

The cursor is special since movements don't need to be routed to
the UI thread, instead they are directly dispatched to the GPU
process from the IO-thread. During the recovery of a crashed GPU
process, the windowing state needs to be re-created on the new GPU. As
such we need to make sure that the windows are re-created on the GPU
before the cursor can send targeted events (to a specific window).

BUG=468485
Committed: https://crrev.com/d818104f773a0d49ba316e0aa8ab1629052d8c97
Cr-Commit-Position: refs/heads/master@{#321645}

[dnicoara, 2015-03-20 16:10:32]

dnicoara@chromium.org changed reviewers:
+ spang@chromium.org

[spang, 2015-03-20 16:51:45]

https://codereview.chromium.org/1024083002/diff/20001/ui/ozone/platform/drm/h...
File ui/ozone/platform/drm/host/drm_gpu_platform_support_host.h (right):

https://codereview.chromium.org/1024083002/diff/20001/ui/ozone/platform/drm/h...
ui/ozone/platform/drm/host/drm_gpu_platform_support_host.h:34: void
RegisterCursorHandler(GpuPlatformSupportHost* cursor_handler);
I think this should say DrmCursor*. It's clearer when things are concrete, and
this isn't actually a useful abstraction - the whole point is that the cursor
needs different treatment. You could also pass it into the constructor & drop
the registration functions.

[dnicoara, 2015-03-20 17:27:51]

https://codereview.chromium.org/1024083002/diff/20001/ui/ozone/platform/drm/h...
File ui/ozone/platform/drm/host/drm_gpu_platform_support_host.h (right):

https://codereview.chromium.org/1024083002/diff/20001/ui/ozone/platform/drm/h...
ui/ozone/platform/drm/host/drm_gpu_platform_support_host.h:34: void
RegisterCursorHandler(GpuPlatformSupportHost* cursor_handler);
On 2015/03/20 16:51:45, spang wrote:
> I think this should say DrmCursor*. It's clearer when things are concrete, and
> this isn't actually a useful abstraction - the whole point is that the cursor
> needs different treatment. You could also pass it into the constructor & drop
> the registration functions.

Done.

[spang, 2015-03-20 19:44:11]

https://codereview.chromium.org/1024083002/diff/40001/ui/ozone/platform/drm/h...
File ui/ozone/platform/drm/host/drm_cursor.h (left):

https://codereview.chromium.org/1024083002/diff/40001/ui/ozone/platform/drm/h...
ui/ozone/platform/drm/host/drm_cursor.h:32: class DrmCursor : public
CursorDelegateEvdev, public GpuPlatformSupportHost {
remove the "GpuPlatformSupportHost"

[spang, 2015-03-20 19:44:16]

lgtm

[dnicoara, 2015-03-20 21:32:33]

The CQ bit was checked by dnicoara@chromium.org

[commit-bot: I haz the power, 2015-03-20 21:33:11]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1024083002/40001

[commit-bot: I haz the power, 2015-03-20 22:02:47]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-03-20 22:03:21]

Patchset 3 (id:??) landed as
https://crrev.com/d818104f773a0d49ba316e0aa8ab1629052d8c97
Cr-Commit-Position: refs/heads/master@{#321645}