[Ozone-Dri] Allocate cursor buffers on the correct DRM device

[Ozone-Dri] Allocate cursor buffers on the correct DRM device

Need to keep track of changing displays and update the cursor buffers
when the window moves between DRM devices.

BUG=461105
Committed: https://crrev.com/f95b436fae524522a65bdbcd8bd0defaae691290
Cr-Commit-Position: refs/heads/master@{#318252}

[dnicoara, 2015-02-26 15:03:29]

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

[dnicoara, 2015-02-26 15:03:29]

PTAL

[alexst (slow to review), 2015-02-26 15:28:21]

https://codereview.chromium.org/963453003/diff/20001/ui/ozone/platform/dri/dr...
File ui/ozone/platform/dri/dri_window_delegate_impl.cc (right):

https://codereview.chromium.org/963453003/diff/20001/ui/ozone/platform/dri/dr...
ui/ozone/platform/dri/dri_window_delegate_impl.cc:130: bool
should_allocate_cursor_buffers = controller_ != controller;
Why not just have if (controller_ != controller) below?

[dnicoara, 2015-02-26 15:31:30]

https://codereview.chromium.org/963453003/diff/20001/ui/ozone/platform/dri/dr...
File ui/ozone/platform/dri/dri_window_delegate_impl.cc (right):

https://codereview.chromium.org/963453003/diff/20001/ui/ozone/platform/dri/dr...
ui/ozone/platform/dri/dri_window_delegate_impl.cc:130: bool
should_allocate_cursor_buffers = controller_ != controller;
On 2015/02/26 15:28:21, alexst wrote:
> Why not just have if (controller_ != controller) below?

Because |controller_| may be set to |controller| bellow.

[alexst (slow to review), 2015-02-26 15:35:46]

lgtm

https://codereview.chromium.org/963453003/diff/20001/ui/ozone/platform/dri/dr...
File ui/ozone/platform/dri/dri_window_delegate_impl.cc (right):

https://codereview.chromium.org/963453003/diff/20001/ui/ozone/platform/dri/dr...
ui/ozone/platform/dri/dri_window_delegate_impl.cc:130: bool
should_allocate_cursor_buffers = controller_ != controller;
On 2015/02/26 15:31:29, dnicoara wrote:
> On 2015/02/26 15:28:21, alexst wrote:
> > Why not just have if (controller_ != controller) below?
> 
> Because |controller_| may be set to |controller| bellow.

Acknowledged.

[dnicoara, 2015-02-26 17:23:51]

The CQ bit was checked by dnicoara@chromium.org

[commit-bot: I haz the power, 2015-02-26 17:24:22]

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

[commit-bot: I haz the power, 2015-02-26 17:34:08]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-02-26 17:34:57]

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