Changes OnWindowHierarchyChanged() to include transient parent

Changes OnWindowHierarchyChanged() to include transient parent

This is necessary for the case where a client creates a window, sets
the transient parent and then adds it. Without this change the wm
wouldn't know about the transient parent and would be in a weird
state.

BUG=663903
TEST=covered by tests
R=ben@chromium.org, tsepez@chromium.org

Committed: https://crrev.com/ae9171adecb864c873bbd1c475144a7d86385174
Cr-Commit-Position: refs/heads/master@{#437450}

[sky, 2016-12-08 19:26:48]

tsepez: mojom
ben: the rest

[sky, 2016-12-08 19:26:51]

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

[commit-bot: I haz the power, 2016-12-08 19:27:43]

Dry run: 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-12-08 19:31:35]

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

[commit-bot: I haz the power, 2016-12-08 19:31:35]

Dry run: Try jobs failed on following builders:
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Ben Goodger (Google), 2016-12-08 20:11:23]

lgtm

[Tom Sepez, 2016-12-08 20:15:54]

mojom LGTM

[sky, 2016-12-08 20:28:33]

The CQ bit was checked by sky@chromium.org

[sky, 2016-12-08 20:28:34]

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

[commit-bot: I haz the power, 2016-12-08 20:29:05]

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-12-08 21:14:44]

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

[commit-bot: I haz the power, 2016-12-08 21:14:45]

Try jobs failed on following builders:
  win_clang on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_clang/builds/...)

[sky, 2016-12-08 23:07:21]

The CQ bit was checked by sky@chromium.org

[commit-bot: I haz the power, 2016-12-08 23:07:44]

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-12-08 23:14:14]

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

[commit-bot: I haz the power, 2016-12-08 23:14:15]

Try jobs failed on following builders:
  win_clang on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_clang/builds/...)

[sky, 2016-12-08 23:32:48]

The CQ bit was checked by sky@chromium.org

[sky, 2016-12-08 23:32:49]

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

[commit-bot: I haz the power, 2016-12-08 23:33:12]

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-12-08 23:42:32]

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

[commit-bot: I haz the power, 2016-12-08 23:42:33]

Try jobs failed on following builders:
  cast_shell_android on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/cast_shell_a...)

[sky, 2016-12-09 00:22:53]

The CQ bit was checked by sky@chromium.org

[commit-bot: I haz the power, 2016-12-09 00:23:16]

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-12-09 01:19:25]

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

[commit-bot: I haz the power, 2016-12-09 01:19:26]

Try jobs failed on following builders:
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[sky, 2016-12-09 01:31:49]

The CQ bit was checked by sky@chromium.org

[sky, 2016-12-09 01:31:50]

The patchset sent to the CQ was uploaded after l-g-t-m from tsepez@chromium.org,
ben@chromium.org
Link to the patchset: https://codereview.chromium.org/2557353003/#ps80001
(title: "fix")

[commit-bot: I haz the power, 2016-12-09 01:33:33]

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-12-09 03:30:16]

CQ is committing da patch.

Bot data: {"patchset_id": 80001, "attempt_start_ts": 1481247109747130,
"parent_rev": "00660e01546384fdf2c1fb5271f70214f85a4bce", "commit_rev":
"54ebee9ceccbbacbc25ddf8c05020f2d1116dfa0"}

[commit-bot: I haz the power, 2016-12-09 03:30:50]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-12-09 03:35:02]

Description was changed from

==========
Changes OnWindowHierarchyChanged() to include transient parent

This is necessary for the case where a client creates a window, sets
the transient parent and then adds it. Without this change the wm
wouldn't know about the transient parent and would be in a weird
state.

BUG=663903
TEST=covered by tests
R=ben@chromium.org, tsepez@chromium.org
==========

to

==========
Changes OnWindowHierarchyChanged() to include transient parent

This is necessary for the case where a client creates a window, sets
the transient parent and then adds it. Without this change the wm
wouldn't know about the transient parent and would be in a weird
state.

BUG=663903
TEST=covered by tests
R=ben@chromium.org, tsepez@chromium.org

Committed: https://crrev.com/ae9171adecb864c873bbd1c475144a7d86385174
Cr-Commit-Position: refs/heads/master@{#437450}
==========

[commit-bot: I haz the power, 2016-12-09 03:35:03]

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