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

Unified Diff: ash/wm/default_state.cc

Issue 1157843009: Added wm::IsWindowUserPositionable(...) method in window_util.h. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Added WindowState::IsUserPositionable() method. Created 5 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ash/wm/default_state.cc
diff --git a/ash/wm/default_state.cc b/ash/wm/default_state.cc
index 3364d298dde9131b18c49ea3738b699e4dd7ee83..321f535bbbe6931f481cd383e6d893e4e7b620e9 100644
--- a/ash/wm/default_state.cc
+++ b/ash/wm/default_state.cc
@@ -422,10 +422,8 @@ bool DefaultState::ProcessWorkspaceEvents(WindowState* window_state,
// adjusted to have minimum visibility, because they are positioned by the
// user and user should always be able to interact with them. Other
// windows are positioned programmatically.
- if (window_state->window()->type() != ui::wm::WINDOW_TYPE_NORMAL &&
- window_state->window()->type() != ui::wm::WINDOW_TYPE_PANEL) {
+ if (!wm::IsWindowUserPositionable(window_state->window()))
oshima 2015/06/04 19:49:12 ditto
bruthig 2015/06/04 19:56:23 Done.
return true;
- }
// Use entire display instead of workarea because the workarea can
// be further shrunk by the docked area. The logic ensures 30%

Powered by Google App Engine
This is Rietveld 408576698