menuBar: failure to remove md-has-open-menu selector

[sivakrrish1994]

Bug:

When md-menu is used inside md-toolbar after clicking on some nested menu option, the md-toolbar still contains md-has-open-menu which causes z-index of toolbar to be increased all the time.

What is the expected behavior?

md-toolbar should remove md-has-open-menu after closing the md-menu.

What is the current behavior?

md-toolbar contains md-has-open-menu even after closing the md-menu.

CodePen and steps to reproduce the issue:

CodePen Demo which shows your issue: https://codepen.io/sivakrrish/pen/LdwzoV

Detailed Reproduction Steps:

1. Select on View menu.
2. Click on Mode --> Edit.
3. Now look at the md-toolbar, it will still contain .md-has-open-menu.

What is the use-case or motivation for changing an existing behavior?

The z-index caused because of md-has-open-menu breaks UI for some other elements(for ex: md-sidenav ui breaks, since its z-index is only 60 while that has 100)

Which versions of AngularJS, Material, OS, and browsers are affected?

The issue was there from v1.1.1 till now v1.1.8

Is there anything else we should know? Stack Traces, Screenshots, etc.