Makes sure bounds and visibilty are updated correctly for new displays

Makes sure bounds and visibilty are updated correctly for new displays

When WindowTreeClient is told about a new display it was not
fully configuring the WindowTreeHost. This patch makes the bounds and
visibility match.

BUG=664625
TEST=none
R=ben@chromium.org

Committed: https://crrev.com/189712a4cd41d68fc08c4b054d3ed2d6a6b1e3a1
Cr-Commit-Position: refs/heads/master@{#435384}

[sky, 2016-11-29 23:15:44]

The CQ bit was checked by sky@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-29 23:16:37]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[sadrul, 2016-11-29 23:29:27]

sadrul@chromium.org changed reviewers:
+ sadrul@chromium.org

[sadrul, 2016-11-29 23:29:28]

lgtm

https://codereview.chromium.org/2535983004/diff/1/ui/aura/mus/window_tree_cli...
File ui/aura/mus/window_tree_client.cc (right):

https://codereview.chromium.org/2535983004/diff/1/ui/aura/mus/window_tree_cli...
ui/aura/mus/window_tree_client.cc:382: window_data->bounds);
Should this set bounds regardless of visibility?

[sky, 2016-11-29 23:35:00]

https://codereview.chromium.org/2535983004/diff/1/ui/aura/mus/window_tree_cli...
File ui/aura/mus/window_tree_client.cc (right):

https://codereview.chromium.org/2535983004/diff/1/ui/aura/mus/window_tree_cli...
ui/aura/mus/window_tree_client.cc:382: window_data->bounds);
On 2016/11/29 23:29:28, sadrul wrote:
> Should this set bounds regardless of visibility?

Good catch! done.

[sky, 2016-11-29 23:35:07]

The CQ bit was checked by sky@chromium.org

[sky, 2016-11-29 23:35:07]

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

[commit-bot: I haz the power, 2016-11-29 23:35:56]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Ben Goodger (Google), 2016-11-30 00:24:18]

lgtm

[commit-bot: I haz the power, 2016-11-30 01:38:51]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-11-30 01:38:52]

Try jobs failed on following builders:
  blimp_linux_dbg on master.tryserver.chromium.linux (JOB_TIMED_OUT, no build
URL)
  chromeos_daisy_chromium_compile_only_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, no build URL)
  chromeos_x86-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_TIMED_OUT, no build URL)
  chromium_presubmit on master.tryserver.chromium.linux (JOB_TIMED_OUT, no build
URL)
  linux_chromium_chromeos_compile_dbg_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, no build URL)
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, no build URL)
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, no build URL)
  linux_chromium_clobber_rel_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, no build URL)
  linux_chromium_compile_dbg_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, no build URL)

[sky, 2016-11-30 03:01:13]

The CQ bit was checked by sky@chromium.org

[commit-bot: I haz the power, 2016-11-30 03:01:40]

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-11-30 05:03:53]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-11-30 05:03:54]

Try jobs failed on following builders:
  blimp_linux_dbg on master.tryserver.chromium.linux (JOB_TIMED_OUT, no build
URL)
  chromeos_x86-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_TIMED_OUT, no build URL)
  chromium_presubmit on master.tryserver.chromium.linux (JOB_TIMED_OUT, no build
URL)
  linux_chromium_chromeos_compile_dbg_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, no build URL)
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, no build URL)
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, no build URL)
  linux_chromium_clobber_rel_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, no build URL)
  linux_chromium_compile_dbg_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, no build URL)
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_TIMED_OUT, no
build URL)

[sky, 2016-11-30 16:11:06]

The CQ bit was checked by sky@chromium.org

[commit-bot: I haz the power, 2016-11-30 16:11:45]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[sky, 2016-11-30 18:52:26]

The CQ bit was checked by sky@chromium.org

[sky, 2016-11-30 18:52:26]

The patchset sent to the CQ was uploaded after l-g-t-m from sadrul@chromium.org,
ben@chromium.org
Link to the patchset: https://codereview.chromium.org/2535983004/#ps40001
(title: "merge")

[commit-bot: I haz the power, 2016-11-30 18:53:01]

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-11-30 19:49:19]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1480531946355540,
"parent_rev": "097c6a8c900ce172ca18e75e1949b5cfaa5628e6", "commit_rev":
"dfdfadb8085c55c9435a2981311afe9dca583948"}

[commit-bot: I haz the power, 2016-11-30 19:49:45]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-11-30 19:54:00]

Description was changed from

==========
Makes sure bounds and visibilty are updated correctly for new displays

When WindowTreeClient is told about a new display it was not
fully configuring the WindowTreeHost. This patch makes the bounds and
visibility match.

BUG=664625
TEST=none
R=ben@chromium.org
==========

to

==========
Makes sure bounds and visibilty are updated correctly for new displays

When WindowTreeClient is told about a new display it was not
fully configuring the WindowTreeHost. This patch makes the bounds and
visibility match.

BUG=664625
TEST=none
R=ben@chromium.org

Committed: https://crrev.com/189712a4cd41d68fc08c4b054d3ed2d6a6b1e3a1
Cr-Commit-Position: refs/heads/master@{#435384}
==========

[commit-bot: I haz the power, 2016-11-30 19:54:01]

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