Makes DefaultContainerLayoutManager ignore windows with transient

Makes DefaultContainerLayoutManager ignore windows with transient
parents. This is needed otherwise the worskpace code tries to place
and position bubbles and other transient type windows.

BUG=none
TEST=none
R=ben@chromium.org,oshima@chromium.org

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=108006

[oshima, 2011-10-28 23:18:48]

Thanks. Can you add a test for this?

[sky, 2011-10-28 23:40:54]

Done.

  -Scott

On Fri, Oct 28, 2011 at 4:18 PM,  <oshima@chromium.org> wrote:
> Thanks. Can you add a test for this?
>
> http://codereview.chromium.org/8414035/
>

[oshima, 2011-10-28 23:55:00]

LGTM

[Ben Goodger (Google), 2011-10-31 18:24:27]

lgtm

http://codereview.chromium.org/8414035/diff/2001/ui/aura_shell/default_contai...
File ui/aura_shell/default_container_layout_manager_unittest.cc (right):

http://codereview.chromium.org/8414035/diff/2001/ui/aura_shell/default_contai...
ui/aura_shell/default_container_layout_manager_unittest.cc:147:
reinterpret_cast<void*>(Widget::InitParams::TYPE_WINDOW)));
It seems like we should have a utility function do this sort of thing more
easily.

[oshima, 2011-10-31 19:48:47]

http://codereview.chromium.org/8414035/diff/2001/ui/aura_shell/default_contai...
File ui/aura_shell/default_container_layout_manager_unittest.cc (right):

http://codereview.chromium.org/8414035/diff/2001/ui/aura_shell/default_contai...
ui/aura_shell/default_container_layout_manager_unittest.cc:147:
reinterpret_cast<void*>(Widget::InitParams::TYPE_WINDOW)));
On 2011/10/31 18:24:27, Ben Goodger (Google) wrote:
> It seems like we should have a utility function do this sort of thing more
> easily.

I'm adding Widow::SetIntProperty, which should be able to
simplify this.