New issue
Advanced search Search tips
Starred by 4 users

Issue metadata

Status: Available
Owner: ----
EstimatedDays: ----
NextAction: ----
OS: All , Mac
Pri: 3
Type: Bug

issue 671916

Participants' hotlists:

Sign in to add a comment

Make *::ShowWithWindowState(minimized, maximized, fullscreen) consistent across platforms

Project Member Reported by, Apr 2 2015

Issue description

Version: Version 41.0.2272.101 (64-bit)
OS: Ubuntu

This feature is not implemented yet, See

create_params.initial_show_state = ui::SHOW_STATE_MINIMIZED;
Browser* new_window = new Browser(create_params);

What is the expected output? What do you see instead?
Minimized window should be shown. but normal window is shown.


Comment 1 by, Apr 2 2015

@sky, Could you please triage this issue? whether it should be fixed or not?
My CL( is blocked by this issue. 
If this issue should be fixed, then I'm happy to try to fix.

Comment 2 by, Apr 17 2015

Labels: -OS-Linux OS-All
Status: Available
Summary: Make *::ShowWithWindowState(minimized, maximized, fullscreen) consistent across platforms (was: Implement create new window with ui::SHOW_STATE_MINIMIZED state.)
Thanks for investigating this! I think this is the summary of the status-quo (at r325564). It's not pretty.

ChromeOS (NativeWidgetAura): MINIMIZED missing only [1]
Windows (HWNDMessageHandler): FULLSCREEN missing only [2]
Desktop Linux (DesktopWindowTreeHostX11): MINIMIZED, FULLSCREEN missing [3]
MacViews (NativeWidgetMac): All NOTIMPLEMENTED() TODO(tapted) [4]
Mac Cocoa (BrowserWindowCocoa::Show()): no such thing as maximize on Mac (minimize, fullscreen implemented) [5]

I'll be poking the MacViews eventually. It's likely that will involve a cross-platform test that ensures things are consistent.


Comment 3 by, May 13 2015

Here is a patch for Linux:

I'll submit patches for other platforms soon.
Project Member

Comment 4 by, May 14 2015

The following revision refers to this bug:

commit 3a3f2f572a3a96957a6180056e7cbd30f551200d
Author: joone.hur <>
Date: Thu May 14 00:41:32 2015

Support MINIMIZED, MAXIMIZED, FULLSCREEN state when showing a chrome window.

Some of ui::WindowShowState were not supported in Linux so this CL implemented


Review URL:

Cr-Commit-Position: refs/heads/master@{#329764}


Project Member

Comment 5 by, May 19 2015

The following revision refers to this bug:

commit 051aece17c0e035938cdd99233967f1cba865b4a
Author: joone.hur <>
Date: Tue May 19 20:51:11 2015

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().


Review URL:

Cr-Commit-Position: refs/heads/master@{#330602}


Project Member

Comment 6 by, May 28 2015

The following revision refers to this bug:

commit 22231ecb82de3ad5ed6f80b81698170bc52f2991
Author: joone.hur <>
Date: Thu May 28 02:33:42 2015

Remove the exceptional cases to support the minimized/fullscreen state for Desktop Linux

ui::SHOW_STATE_MINIMIZED and ui::SHOW_STATE_FULLSCREEN started to be handled
in Desktop Linux when using API in r329764.

So, we can remove the exceptional cases to support the states for Desktop Linux.

BUG= 459841 ,473228

Review URL:

Cr-Commit-Position: refs/heads/master@{#331726}


Project Member

Comment 7 by, Nov 26 2015

The following revision refers to this bug:

commit d5d6762c5539bc7d6098588323a16a24826e1149
Author: joone.hur <>
Date: Thu Nov 26 16:06:04 2015

Make the minimize state work when creating a chrome window in CrOS

This CL makes SHOW_STATE_MINIMIZE state work when creating a
chrome window in CrOS, so we don't need to call Minimize()
manually in


Review URL:

Cr-Commit-Position: refs/heads/master@{#361898}


Project Member

Comment 8 by, Jul 14 2016

Labels: Hotlist-OpenBugWithCL
A change has landed for this issue, but it's been open for over 6 months. Please review and close it if applicable. If this issue should remain open, remove the "Hotlist-OpenBugWithCL" label. If no action is taken, it will be archived in 30 days.

For more details visit - Your friendly Sheriffbot

Comment 9 by, Jul 14 2016

Labels: -Pri-2 -Hotlist-OpenBugWithCL Proj-MacViews Pri-3
This still needs to be revisited, but it's not urgent
Status: Started (was: Available)
Labels: MacViews-Cleanup M-X OS-Mac
tapted, is this still in progress?
Blocking: 671916
Owner: ----
Status: Available (was: Started)
I think we still need NativeWidgetMac::ShowWithWindowState() to implement minimize and fullscreen. Maximize is less interesting, but I'm pretty sure the others are needed for session restore (and maybe extension APIs). Tentatively blocking Issue 671916 based on that. seems like it might have some coverage, but a lot of tests are disabled :/.

Other platforms seem to be feature-complete. Some of the tests in CLs above can probably be made cross-platform.

I'm moving this into Available since I'm swamped for at least the next few weeks for m67 stuff.

Sign in to add a comment