Explicit container priority

Explicit container priority

* This will allow a client to create & destroy a container with specific z order, without relying on the order it is created. (will be used for login/lock screen)

* Changed to single click to open home card, instead of double click because it's easier and consistent with ash. (ash uses single click to open/activate things)

* I tried to add check but I couldn't come  up with good solution using window oberver as the child is always added to the top first.

BUG=394121
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=285185

[sadrul, 2014-07-23 17:00:13]

Do you have the right CL up?

[oshima, 2014-07-23 17:17:10]

On 2014/07/23 17:00:13, sadrul wrote:
> Do you have the right CL up?

oops, sorry. I uploaded different patch set to this somehow. It's removed. PTAL.

[sadrul, 2014-07-23 18:37:24]

LGTM

https://codereview.chromium.org/411543006/diff/90001/athena/common/container_...
File athena/common/container_priorities.h (right):

https://codereview.chromium.org/411543006/diff/90001/athena/common/container_...
athena/common/container_priorities.h:15: CP_DEBUG = 4,
Do you want to number each explicitly? (in case we need to add a new one in
between, it would be necessary to update all the rest)

https://codereview.chromium.org/411543006/diff/90001/athena/screen/screen_man...
File athena/screen/screen_manager_impl.cc (right):

https://codereview.chromium.org/411543006/diff/90001/athena/screen/screen_man...
athena/screen/screen_manager_impl.cc:206: << params.z_order_priority << "
already exists.";
nice

[oshima, 2014-07-23 20:51:39]

https://codereview.chromium.org/411543006/diff/90001/athena/common/container_...
File athena/common/container_priorities.h (right):

https://codereview.chromium.org/411543006/diff/90001/athena/common/container_...
athena/common/container_priorities.h:15: CP_DEBUG = 4,
On 2014/07/23 18:37:24, sadrul wrote:
> Do you want to number each explicitly? (in case we need to add a new one in
> between, it would be necessary to update all the rest)

yeah, that's unnecessary. removed

[oshima, 2014-07-23 20:52:08]

The CQ bit was checked by oshima@chromium.org

[commit-bot: I haz the power, 2014-07-23 20:54:25]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/oshima@chromium.org/411543006/130001

[oshima, 2014-07-23 22:40:44]

The CQ bit was checked by oshima@chromium.org

[commit-bot: I haz the power, 2014-07-23 22:43:24]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/oshima@chromium.org/411543006/170001

[oshima, 2014-07-24 00:19:12]

The CQ bit was checked by oshima@chromium.org

[commit-bot: I haz the power, 2014-07-24 00:21:45]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/oshima@chromium.org/411543006/190001

[oshima, 2014-07-24 04:47:18]

The CQ bit was unchecked by oshima@chromium.org

[oshima, 2014-07-24 04:47:25]

The CQ bit was checked by oshima@chromium.org

[commit-bot: I haz the power, 2014-07-24 04:49:16]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/oshima@chromium.org/411543006/190001

[oshima, 2014-07-24 06:01:11]

The CQ bit was checked by oshima@chromium.org

[commit-bot: I haz the power, 2014-07-24 06:01:46]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/oshima@chromium.org/411543006/210001

[commit-bot: I haz the power, 2014-07-24 07:58:53]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  mac_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/mac_chromium_rel/buil...)

[commit-bot: I haz the power, 2014-07-24 11:09:17]

Change committed as 285185