Makes recreating a layer inherit target visibility and opacity

Makes recreating a layer inherit target visibility and opacity

This better matches expectations.

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

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

[sky, 2014-04-09 16:23:43]

I tried maximizing and closing a window and didn't see any weirdness. If you
think there are other cases I should verify let me know.

[oshima, 2014-04-09 17:28:13]

On 2014/04/09 16:23:43, sky wrote:
> I tried maximizing and closing a window and didn't see any weirdness. If you
> think there are other cases I should verify let me know.
lgtm

Can you also try switching between desktop with multi-profiles and switching
between maximized mode (and normal mode)?

stefan knows how to test it on desktop.

[sky, 2014-04-09 18:18:33]

It all seems to work for me.

  -Scott

On Wed, Apr 9, 2014 at 10:28 AM,  <oshima@chromium.org> wrote:
> On 2014/04/09 16:23:43, sky wrote:
>>
>> I tried maximizing and closing a window and didn't see any weirdness. If
>> you
>> think there are other cases I should verify let me know.
>
> lgtm
>
> Can you also try switching between desktop with multi-profiles and switching
> between maximized mode (and normal mode)?
>
> stefan knows how to test it on desktop.
>
> https://codereview.chromium.org/231193002/

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[oshima, 2014-04-09 18:27:51]

On 2014/04/09 18:18:33, sky wrote:
> It all seems to work for me.
> 
>   -Scott

Thank you for testing. slgtm.

> On Wed, Apr 9, 2014 at 10:28 AM,  <mailto:oshima@chromium.org> wrote:
> > On 2014/04/09 16:23:43, sky wrote:
> >>
> >> I tried maximizing and closing a window and didn't see any weirdness. If
> >> you
> >> think there are other cases I should verify let me know.
> >
> > lgtm
> >
> > Can you also try switching between desktop with multi-profiles and switching
> > between maximized mode (and normal mode)?
> >
> > stefan knows how to test it on desktop.
> >
> > https://codereview.chromium.org/231193002/
> 
> To unsubscribe from this group and stop receiving emails from it, send an
email
> to mailto:chromium-reviews+unsubscribe@chromium.org.

[oshima, 2014-04-09 18:27:51]

On 2014/04/09 18:18:33, sky wrote:
> It all seems to work for me.
> 
>   -Scott

Thank you for testing. slgtm.

> On Wed, Apr 9, 2014 at 10:28 AM,  <mailto:oshima@chromium.org> wrote:
> > On 2014/04/09 16:23:43, sky wrote:
> >>
> >> I tried maximizing and closing a window and didn't see any weirdness. If
> >> you
> >> think there are other cases I should verify let me know.
> >
> > lgtm
> >
> > Can you also try switching between desktop with multi-profiles and switching
> > between maximized mode (and normal mode)?
> >
> > stefan knows how to test it on desktop.
> >
> > https://codereview.chromium.org/231193002/
> 
> To unsubscribe from this group and stop receiving emails from it, send an
email
> to mailto:chromium-reviews+unsubscribe@chromium.org.

[sky, 2014-04-09 19:15:14]

The CQ bit was checked by sky@chromium.org

[commit-bot: I haz the power, 2014-04-09 19:16:34]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/sky@chromium.org/231193002/1

[commit-bot: I haz the power, 2014-04-09 19:48:49]

Change committed as 262778