Move DisplayLayoutStore from ash to ui.

Move DisplayLayoutStore from ash to ui.

This CL moves DisplayLayoutStore from src/ash/display/ to
src/ui/display/manager. This is so that DisplayLayoutStore can be used
with services/ui/display/ code. Update references and move some utility
functions and flags used by DisplayLayoutStore.

BUG=625876
Committed: https://crrev.com/7eacccaaa5e5cb14cafbbb25cba892adf839fb5a
Cr-Commit-Position: refs/heads/master@{#413148}

[kylechar, 2016-08-09 17:48:56]

Description was changed from

==========
Move DisplayLayoutStore from ash to ui.

This CL moves DisplayLayoutStore from src/ash/display/ to
src/ui/display/manager. This is so that DisplayLayoutStore can be used
with services/ui/display/ code. Update references and move some
utility/flags.

BUG=TBD
==========

to

==========
Move DisplayLayoutStore from ash to ui.

This CL moves DisplayLayoutStore from src/ash/display/ to
src/ui/display/manager. This is so that DisplayLayoutStore can be used
with services/ui/display/ code. Update references and move some utility
functions and flags used by DisplayLayoutStore.

BUG=625876
==========

[kylechar, 2016-08-09 18:06:05]

kylechar@chromium.org changed reviewers:
+ oshima@chromium.org

[kylechar, 2016-08-09 18:06:06]

There are two utility functions I've copied into display_layout_store.cc
temporarily. Right now the functions are in a util file for ash. It seems a
common place for utility functions somewhere under src/ui/display/ would make
sense but there are already two sets of display_util.cc/h files under
src/ui/display/.

src/ui/display/util/display_util.cc/h: This is part of the //ui/display/util
target and it doesn't include //ui/display as a dep. That means utility
functions that depend on display::Display can't go in here.

src/ui/display/chromeos/display_util.cc/h: This is part of //ui/display so it
can use display::Display but we need somewhere for utility functions that aren't
Chrome OS specific.

Any thoughts on how to handle it? I could add something like
src/ui/display/display_util.cc/h as part of //ui/display but that seems rife for
confusion.

[kylechar, 2016-08-17 16:22:00]

Think you'll have time to take a look this week oshima?

[oshima, 2016-08-17 23:57:10]

On 2016/08/17 16:22:00, kylechar wrote:
> Think you'll have time to take a look this week oshima?

Sorry the CL was buried in other CLs. I'll review tomorrow.

[oshima, 2016-08-18 18:28:22]

lgtm

[kylechar, 2016-08-19 13:59:25]

Thanks oshima!

[kylechar, 2016-08-19 13:59:39]

The CQ bit was checked by kylechar@chromium.org

[kylechar, 2016-08-19 13:59:40]

The patchset sent to the CQ was uploaded after l-g-t-m from oshima@chromium.org
Link to the patchset: https://codereview.chromium.org/2223063003/#ps40001
(title: "Rebase/add TODO.")

[commit-bot: I haz the power, 2016-08-19 13:59:54]

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-08-19 15:16:28]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-08-19 15:18:38]

Description was changed from

==========
Move DisplayLayoutStore from ash to ui.

This CL moves DisplayLayoutStore from src/ash/display/ to
src/ui/display/manager. This is so that DisplayLayoutStore can be used
with services/ui/display/ code. Update references and move some utility
functions and flags used by DisplayLayoutStore.

BUG=625876
==========

to

==========
Move DisplayLayoutStore from ash to ui.

This CL moves DisplayLayoutStore from src/ash/display/ to
src/ui/display/manager. This is so that DisplayLayoutStore can be used
with services/ui/display/ code. Update references and move some utility
functions and flags used by DisplayLayoutStore.

BUG=625876

Committed: https://crrev.com/7eacccaaa5e5cb14cafbbb25cba892adf839fb5a
Cr-Commit-Position: refs/heads/master@{#413148}
==========

[commit-bot: I haz the power, 2016-08-19 15:18:39]

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