Commit 3c8c7950 authored by scheib@chromium.org's avatar scheib@chromium.org

Fix failing test on Mac 10.6: TestInstallThemeInFullScreen

The initial change in r179554 [1] presumed that IDC_FULLSCREEN would only be issued on Mac 10.7 and later. TestInstallThemeInFullScreen calls fullscreen without a platform & os version check.

This solution is to tolerate IDC_FULLSCREEN and convert it to a IDC_PRESENTATION_MODE on Mac 10.6 and earlier. Both are conceptually the same "fullscreen with no browser chrome".

[1] http://src.chromium.org/viewvc/chrome?view=rev&revision=179554

BUG=169138
TBR=sky@chromium.org


Review URL: https://chromiumcodereview.appspot.com/12096063

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@179747 0039d316-1c4b-4281-b951-d872f2087c98
parent e435376c
......@@ -40,6 +40,7 @@
#include "ui/base/keycodes/keyboard_codes.h"
#if defined(OS_MACOSX)
#include "base/mac/mac_util.h"
#include "chrome/browser/ui/browser_commands_mac.h"
#endif
......@@ -426,7 +427,10 @@ void BrowserCommandController::ExecuteCommandWithDisposition(
break;
case IDC_FULLSCREEN:
#if defined(OS_MACOSX)
chrome::ToggleFullscreenWithChrome(browser_);
if (base::mac::IsOSLionOrLater())
chrome::ToggleFullscreenWithChrome(browser_);
else
chrome::ToggleFullscreenMode(browser_);
#else
chrome::ToggleFullscreenMode(browser_);
#endif
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment