Skip to content
This repository has been archived by the owner on Apr 3, 2020. It is now read-only.

[Windows] Maximize the window over the taskbar in the fullscreen mode. #3501

Closed
wants to merge 1 commit into from

Conversation

joone
Copy link
Contributor

@joone joone commented Jan 22, 2016

It allows the users to close the fullscreen application.

BUG=XWALK-4861

@crosswalk-trybot
Copy link

Testing patch series with joone/crosswalk@b25a9b7 as its head.

Bot Status
Crosswalk Android-X86 [SUCCESS 💚](https://build.crosswalk-project.org/try/builders/Crosswalk Android-X86/builds/4125)
Crosswalk Android x86-64 [SUCCESS 💚](https://build.crosswalk-project.org/try/builders/Crosswalk Android x86-64/builds/1173)
Crosswalk Linux [SUCCESS 💚](https://build.crosswalk-project.org/try/builders/Crosswalk Linux/builds/4115)

It allows the users to close the fullscreen application.
This is a temporary solution. Metro driver will be used to make
Crosswalk applications work like Windows universal application,
which is in src/win8/metro_driver of Chromium source code.

BUG=XWALK-4861
@crosswalk-trybot
Copy link

Testing patch series with joone/crosswalk@56773e4 as its head.

Bot Status
Crosswalk Android-X86 [SUCCESS 💚](https://build.crosswalk-project.org/try/builders/Crosswalk Android-X86/builds/4131)
Crosswalk Android x86-64 [SUCCESS 💚](https://build.crosswalk-project.org/try/builders/Crosswalk Android x86-64/builds/1179)
Crosswalk Linux [SUCCESS 💚](https://build.crosswalk-project.org/try/builders/Crosswalk Linux/builds/4121)

// Set ui::SHOW_STATE_MAXIMIZED instead of ui::SHOW_STATE_FULLSCREEN
// because there is no way to exit from full screen mode.
// Refer to XWALK-4861 for more details.
#if defined(OS_WIN)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks like it breaks fullscreen on linux desktop

@rakuco
Copy link
Member

rakuco commented Apr 12, 2016

What's the status of this pull request?

@darktears
Copy link
Contributor

@joone ???

@joone
Copy link
Contributor Author

joone commented May 4, 2016

We don't need this commit.

@joone joone closed this May 4, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants