Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(323)

Issue 407893002: Prevent a window from being put into fullscreen upon startup on desktop Linux (Closed)

Created:
6 years, 5 months ago by pkotwicz
Modified:
6 years, 4 months ago
Reviewers:
Elliot Glaysher
CC:
chromium-reviews, tfarina, ben+views_chromium.org, tdanderson+views_chromium.org
Project:
chromium
Visibility:
Public.

Description

Prevent a window from being put into fullscreen upon startup on desktop Linux The GTK port had logic which prevented windows from being the same size as the monitor because some window managers would put those windows into fullscreen. (gtk_window_util::SetWindowSize()). This CL implements this logic for the Linux aura port BUG=382126 TEST=Manual, see instructions below Testing steps: 1) Open a browser window in the Gnome Classic window manager (this is very important) 2) Maximize the browser window, fullscreen with <F11>, then unfullscreen it. 3) Close Chrome. 4) Reopen Chrome. The browser window should be maximized. 5) Unmaximize the browser window and ensure that it is possible to resize the browser window. Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=285906

Patch Set 1 : #

Total comments: 2

Patch Set 2 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+44 lines, -5 lines) Patch
M ui/views/widget/desktop_aura/desktop_window_tree_host_x11.h View 3 chunks +7 lines, -2 lines 0 comments Download
M ui/views/widget/desktop_aura/desktop_window_tree_host_x11.cc View 1 6 chunks +37 lines, -3 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
pkotwicz
Elliot, PTAL This should hopefully fix the remaining bugs wrt to putting Chrome inadvertently into ...
6 years, 5 months ago (2014-07-25 19:31:52 UTC) #1
Elliot Glaysher
lgtm
6 years, 5 months ago (2014-07-25 19:44:17 UTC) #2
pkotwicz
The CQ bit was checked by pkotwicz@chromium.org
6 years, 4 months ago (2014-07-28 13:41:18 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/pkotwicz@chromium.org/407893002/40001
6 years, 4 months ago (2014-07-28 13:42:44 UTC) #4
commit-bot: I haz the power
6 years, 4 months ago (2014-07-28 14:52:53 UTC) #5
Message was sent while issue was closed.
Change committed as 285906

Powered by Google App Engine
This is Rietveld 408576698