On Mac OS X, after entering fullscreen mode, the fullscreen button is still displayed on the toolbar. It should be hidden since the system provides such button for exiting on the top menu bar. This is a regression from bug 1162752.

Comment on attachment 8607845 [details] [diff] [review] patch >+ // For Lion fullscreen, all fullscreen controls are hidden, don't >+ // bother to touch them. If we don't stop here, the following code >+ // could cause the native fullscreen button be shown unexpectedly. >+ // See bug 1165570. >+ if (!aShow && this.useLionFullScreen) { >+ return; >+ } Why do you care about !aShow here?

I guess it doesn't really matter... When I was debugging bug 1105939, I suspected a bug was potentially related to this, which later was proven to be false. Hence it seems to me that either version is fine. If you think the old one is better, you could obsolete the new one and grant the old one :)

Comment on attachment 8606774 [details] [diff] [review] patch Approval Request Comment [Feature/regressing bug #]: bug 1162752 [User impact if declined]: will see fullscreen button on tabbar even in fullscreen mode [Describe test coverage new/current, TreeHerder]: n/a [Risks and why]: shouldn't be risky [String/UUID change made/needed]: n/a