Fix for Fullscreen toolbar with multiple monitors on OSX

Fix for Fullscreen toolbar with multiple monitors on OSX

When checking for the active window during the menubar dropdown
progress, use the isMainWindow property instead of onActiveSpace.

If we have multiple monitors, onActiveSpace can return true, even if the
window isn't active.

BUG=606511
Committed: https://crrev.com/bde4e511c9b163ae0f2926a12834485cf9f6e996
Cr-Commit-Position: refs/heads/master@{#393642}

[spqchan, 2016-05-13 20:05:39]

Description was changed from

==========
Fix for Fullscreen Toolbar with multiple monitors on OSX

Use the isMainWindow property instead of onActiveSpace


BUG=606511
==========

to

==========
Fix for Fullscreen toolbar with multiple monitors on OSX

Use the isMainWindow property instead of onActiveSpace


BUG=606511
==========

[spqchan, 2016-05-13 20:07:22]

Description was changed from

==========
Fix for Fullscreen toolbar with multiple monitors on OSX

Use the isMainWindow property instead of onActiveSpace


BUG=606511
==========

to

==========
Fix for Fullscreen toolbar with multiple monitors on OSX

Use the isMainWindow property instead of onActiveSpace since
onActiveSpace would return true on multiple monitors.

BUG=606511
==========

[spqchan, 2016-05-13 20:09:10]

Description was changed from

==========
Fix for Fullscreen toolbar with multiple monitors on OSX

Use the isMainWindow property instead of onActiveSpace since
onActiveSpace would return true on multiple monitors.

BUG=606511
==========

to

==========
Fix for Fullscreen toolbar with multiple monitors on OSX

When checking for the active window during the menubar dropdown
progress, use the isMainWindow property instead of onActiveSpace.

If we have multiple monitors, onActiveSpace can return true, even if the
window isn't active.

BUG=606511
==========

[spqchan, 2016-05-13 20:09:11]

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

[spqchan, 2016-05-13 20:09:36]

PTAL

[Robert Sesek, 2016-05-13 20:22:26]

lgtm

[spqchan, 2016-05-13 20:23:36]

The CQ bit was checked by spqchan@chromium.org

[commit-bot: I haz the power, 2016-05-13 20:24:16]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1979663002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1979663002/1

[commit-bot: I haz the power, 2016-05-13 21:09:51]

Description was changed from

==========
Fix for Fullscreen toolbar with multiple monitors on OSX

When checking for the active window during the menubar dropdown
progress, use the isMainWindow property instead of onActiveSpace.

If we have multiple monitors, onActiveSpace can return true, even if the
window isn't active.

BUG=606511
==========

to

==========
Fix for Fullscreen toolbar with multiple monitors on OSX

When checking for the active window during the menubar dropdown
progress, use the isMainWindow property instead of onActiveSpace.

If we have multiple monitors, onActiveSpace can return true, even if the
window isn't active.

BUG=606511
==========

[commit-bot: I haz the power, 2016-05-13 21:09:53]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-05-13 21:11:15]

Description was changed from

==========
Fix for Fullscreen toolbar with multiple monitors on OSX

When checking for the active window during the menubar dropdown
progress, use the isMainWindow property instead of onActiveSpace.

If we have multiple monitors, onActiveSpace can return true, even if the
window isn't active.

BUG=606511
==========

to

==========
Fix for Fullscreen toolbar with multiple monitors on OSX

When checking for the active window during the menubar dropdown
progress, use the isMainWindow property instead of onActiveSpace.

If we have multiple monitors, onActiveSpace can return true, even if the
window isn't active.

BUG=606511

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

[commit-bot: I haz the power, 2016-05-13 21:11:16]

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