[Mac] Fix for the fullscreen toolbar issue with Active Spaces

[Mac] Fix for the fullscreen toolbar issue with Active Spaces

Made sure that the menubar state is correct when the active space is
changed

BUG=647111
Committed: https://crrev.com/8a690e5e200f90623e69083917fea4677e11e660
Cr-Commit-Position: refs/heads/master@{#420645}

[spqchan, 2016-09-21 00:06:13]

Description was changed from

==========
[Mac] Fix for the fullscreen toolbar with active space

Made sure that the menubar state is correct when the active space is
changed

BUG=647111
==========

to

==========
[Mac] Fix for the fullscreen toolbar issue with Active Spaces

Made sure that the menubar state is correct when the active space is
changed

BUG=647111
==========

[spqchan, 2016-09-21 00:06:14]

spqchan@chromium.org changed reviewers:
+ rsesek@chromium.org

[spqchan, 2016-09-21 00:06:44]

PTAL

[Robert Sesek, 2016-09-21 20:32:13]

How does this work with multiple monitors and the user specifying their
preference for the menubar to be on both screens?

[spqchan, 2016-09-23 02:07:03]

On 2016/09/21 20:32:13, Robert Sesek wrote:
> How does this work with multiple monitors and the user specifying their
> preference for the menubar to be on both screens?

I've been playing around with it on multiple screens and as far as I can tell,
it works fine since even with that setting, menubar would be initially hidden
when you switch back.
Do you have any cases where it might break?

[Robert Sesek, 2016-09-23 15:35:54]

LGTM

On 2016/09/23 02:07:03, spqchan wrote:
> On 2016/09/21 20:32:13, Robert Sesek wrote:
> > How does this work with multiple monitors and the user specifying their
> > preference for the menubar to be on both screens?
> 
> I've been playing around with it on multiple screens and as far as I can tell,
> it works fine since even with that setting, menubar would be initially hidden
> when you switch back.
> Do you have any cases where it might break?

No, just wanted to make sure that case was tested.

[spqchan, 2016-09-23 16:43:34]

On 2016/09/23 15:35:54, Robert Sesek wrote:
> LGTM
> 
> On 2016/09/23 02:07:03, spqchan wrote:
> > On 2016/09/21 20:32:13, Robert Sesek wrote:
> > > How does this work with multiple monitors and the user specifying their
> > > preference for the menubar to be on both screens?
> > 
> > I've been playing around with it on multiple screens and as far as I can
tell,
> > it works fine since even with that setting, menubar would be initially
hidden
> > when you switch back.
> > Do you have any cases where it might break?
> 
> No, just wanted to make sure that case was tested.

Thanks!

[spqchan, 2016-09-23 16:43:38]

The CQ bit was checked by spqchan@chromium.org

[commit-bot: I haz the power, 2016-09-23 16:44:10]

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-09-23 17:25:49]

Description was changed from

==========
[Mac] Fix for the fullscreen toolbar issue with Active Spaces

Made sure that the menubar state is correct when the active space is
changed

BUG=647111
==========

to

==========
[Mac] Fix for the fullscreen toolbar issue with Active Spaces

Made sure that the menubar state is correct when the active space is
changed

BUG=647111
==========

[commit-bot: I haz the power, 2016-09-23 17:25:50]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-23 17:29:27]

Description was changed from

==========
[Mac] Fix for the fullscreen toolbar issue with Active Spaces

Made sure that the menubar state is correct when the active space is
changed

BUG=647111
==========

to

==========
[Mac] Fix for the fullscreen toolbar issue with Active Spaces

Made sure that the menubar state is correct when the active space is
changed

BUG=647111

Committed: https://crrev.com/8a690e5e200f90623e69083917fea4677e11e660
Cr-Commit-Position: refs/heads/master@{#420645}
==========

[commit-bot: I haz the power, 2016-09-23 17:29:28]

Patchset 1 (id:??) landed as
https://crrev.com/8a690e5e200f90623e69083917fea4677e11e660
Cr-Commit-Position: refs/heads/master@{#420645}