Commit 051aece1 authored by joone.hur's avatar joone.hur Committed by Commit bot

Support the full-screen state when showing a chrome window

The SHOW_STATE_FULLSCREEN state has not been supported in Windows
so this CL implements it and allows browser_tests to pass the
test for creating a full screen extension window without manually
calling setFullscreenMode().

BUG=473228

Review URL: https://codereview.chromium.org/1138383008

Cr-Commit-Position: refs/heads/master@{#330602}
parent fef3d83e
......@@ -669,8 +669,8 @@ bool WindowsCreateFunction::RunSync() {
if (create_params.initial_show_state == ui::SHOW_STATE_MINIMIZED)
new_window->window()->Minimize();
#endif
#if (defined(OS_LINUX) && !defined(OS_CHROMEOS)) || defined(OS_WIN)
// On Desktop Linux and Windows, managers don't handle fullscreen state to
#if (defined(OS_LINUX) && !defined(OS_CHROMEOS))
// On Desktop Linux managers don't handle fullscreen state to
// create window for now.
if (create_params.initial_show_state == ui::SHOW_STATE_FULLSCREEN)
controller->SetFullscreenMode(true, extension()->url());
......
......@@ -571,6 +571,10 @@ void HWNDMessageHandler::ShowWindowWithState(ui::WindowShowState show_state) {
case ui::SHOW_STATE_NORMAL:
native_show_state = SW_SHOWNORMAL;
break;
case ui::SHOW_STATE_FULLSCREEN:
native_show_state = SW_SHOWNORMAL;
SetFullscreen(true);
break;
default:
native_show_state = delegate_->GetInitialShowState();
break;
......
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