From ca19a0eb2da0818bdb9b6c56df4f0675dd563b67 Mon Sep 17 00:00:00 2001 From: "avi@chromium.org" Date: Thu, 4 Apr 2013 22:54:29 +0000 Subject: Update comment on workaround with bug tracking info BUG=170189, 225659 TEST=change is to comment only; no behavior change Review URL: https://chromiumcodereview.appspot.com/13467018 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@192405 0039d316-1c4b-4281-b951-d872f2087c98 --- base/mac/mac_util.mm | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) (limited to 'base/mac') diff --git a/base/mac/mac_util.mm b/base/mac/mac_util.mm index 48b4e5a..a6501a41 100644 --- a/base/mac/mac_util.mm +++ b/base/mac/mac_util.mm @@ -70,10 +70,12 @@ void SetUIMode() { NSApplicationPresentationHideMenuBar; } - // Bug-fix: if the window is fullscreened (Lion-style) and + // Mac OS X bug: if the window is fullscreened (Lion-style) and // NSApplicationPresentationDefault is requested, the result is that the menu - // bar doesn't auto-hide. In that case, explicitly set the presentation - // options to the ones that are set by the system as it fullscreens a window. + // bar doesn't auto-hide. rdar://13576498 http://www.openradar.me/13576498 + // + // As a workaround, in that case, explicitly set the presentation options to + // the ones that are set by the system as it fullscreens a window. if (desired_options == NSApplicationPresentationDefault && current_options & NSApplicationPresentationFullScreen) { desired_options |= NSApplicationPresentationFullScreen | -- cgit v1.1