Use display::DisplayMode in ozone/drm/gpu

Use display::DisplayMode in the ozone/drm DrmThread boundary to simplify
implementing a Mojo interface with the DrmThread interface.

BUG=620927
Review-Url: https://codereview.chromium.org/2894523007
Cr-Commit-Position: refs/heads/master@{#474139}
Committed: https://chromium.googlesource.com/chromium/src/+/54fee5d86ef41e706cb794d1c587a5269ecee9ec

[rjkroege, 2017-05-19 20:02:16]

Description was changed from

==========
Relocate display_snapshot_mojo

Keeping DisplaySnapshotMojo in display/types enables tighter use of
DEPS in ozone/platform/drm so relocate it appropriately.

BUG=620927
==========

to

==========
Use display::DisplayMode in the ozone/drm DrmThread boundary to simplify
implementing a Mojo interface with the DrmThread interface.

BUG=620927
==========

[rjkroege, 2017-05-19 20:02:17]

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

[rjkroege, 2017-05-23 18:24:21]

ptal

[dnicoara, 2017-05-23 18:48:20]

lgtm % nits

https://codereview.chromium.org/2894523007/diff/1/ui/ozone/platform/drm/commo...
File ui/ozone/platform/drm/common/drm_util_unittest.cc (right):

https://codereview.chromium.org/2894523007/diff/1/ui/ozone/platform/drm/commo...
ui/ozone/platform/drm/common/drm_util_unittest.cc:5: #include
"ui/ozone/platform/drm/common/drm_util.h"
Out of order include.

https://codereview.chromium.org/2894523007/diff/1/ui/ozone/platform/drm/gpu/d...
File ui/ozone/platform/drm/gpu/drm_thread_message_proxy.cc (right):

https://codereview.chromium.org/2894523007/diff/1/ui/ozone/platform/drm/gpu/d...
ui/ozone/platform/drm/gpu/drm_thread_message_proxy.cc:11: #include
"ui/display/types/display_snapshot_mojo.h"
Is this required?

https://codereview.chromium.org/2894523007/diff/1/ui/ozone/platform/drm/gpu/s...
File ui/ozone/platform/drm/gpu/screen_manager.cc (right):

https://codereview.chromium.org/2894523007/diff/1/ui/ozone/platform/drm/gpu/s...
ui/ozone/platform/drm/gpu/screen_manager.cc:13: #include
"ui/display/types/display_snapshot_mojo.h"
Is this required?

https://codereview.chromium.org/2894523007/diff/1/ui/ozone/platform/drm/mus_t...
File ui/ozone/platform/drm/mus_thread_proxy.cc (right):

https://codereview.chromium.org/2894523007/diff/1/ui/ozone/platform/drm/mus_t...
ui/ozone/platform/drm/mus_thread_proxy.cc:11: #include
"ui/display/types/display_snapshot_mojo.h"
Is this include required?

[rjkroege, 2017-05-23 22:08:18]

https://codereview.chromium.org/2894523007/diff/1/ui/ozone/platform/drm/commo...
File ui/ozone/platform/drm/common/drm_util_unittest.cc (right):

https://codereview.chromium.org/2894523007/diff/1/ui/ozone/platform/drm/commo...
ui/ozone/platform/drm/common/drm_util_unittest.cc:5: #include
"ui/ozone/platform/drm/common/drm_util.h"
On 2017/05/23 18:48:20, dnicoara wrote:
> Out of order include.

Weird that presubmit didn't catch. Fixed.

https://codereview.chromium.org/2894523007/diff/1/ui/ozone/platform/drm/gpu/d...
File ui/ozone/platform/drm/gpu/drm_thread_message_proxy.cc (right):

https://codereview.chromium.org/2894523007/diff/1/ui/ozone/platform/drm/gpu/d...
ui/ozone/platform/drm/gpu/drm_thread_message_proxy.cc:11: #include
"ui/display/types/display_snapshot_mojo.h"
On 2017/05/23 18:48:20, dnicoara wrote:
> Is this required?

nope. part of the other CL that's not done yet. Removed here and elsewhere.

[rjkroege, 2017-05-23 22:08:27]

The CQ bit was checked by rjkroege@chromium.org

[rjkroege, 2017-05-23 22:08:28]

The patchset sent to the CQ was uploaded after l-g-t-m from
dnicoara@chromium.org
Link to the patchset: https://codereview.chromium.org/2894523007/#ps20001
(title: "review nits")

[commit-bot: I haz the power, 2017-05-23 22:09:32]

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, 2017-05-24 02:50:29]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1495577307690160,
"parent_rev": "25c3ff2f544c82f66cc75087c63ddac145a6e984", "commit_rev":
"54fee5d86ef41e706cb794d1c587a5269ecee9ec"}

[commit-bot: I haz the power, 2017-05-24 02:50:42]

Description was changed from

==========
Use display::DisplayMode in the ozone/drm DrmThread boundary to simplify
implementing a Mojo interface with the DrmThread interface.

BUG=620927
==========

to

==========
Use display::DisplayMode in the ozone/drm DrmThread boundary to simplify
implementing a Mojo interface with the DrmThread interface.

BUG=620927

Review-Url: https://codereview.chromium.org/2894523007
Cr-Commit-Position: refs/heads/master@{#474139}
Committed:
https://chromium.googlesource.com/chromium/src/+/54fee5d86ef41e706cb794d1c587...
==========

[commit-bot: I haz the power, 2017-05-24 02:50:43]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/54fee5d86ef41e706cb794d1c587...