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

Unified Diff: ash/wm/workspace/backdrop_controller.cc

Issue 2918403006: CrOS Tablet Window management - Split Screen part I (Closed)
Patch Set: Fix failed tests. Created 3 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/workspace/backdrop_controller.cc
diff --git a/ash/wm/workspace/backdrop_controller.cc b/ash/wm/workspace/backdrop_controller.cc
index b73113f0d316c163fa7d94d55429c69231c8c0ae..ae52971da16cd90f5ef9afaea8627ca46bf14f71 100644
--- a/ash/wm/workspace/backdrop_controller.cc
+++ b/ash/wm/workspace/backdrop_controller.cc
@@ -141,6 +141,19 @@ void BackdropController::OnOverviewModeStarting() {
}
void BackdropController::OnOverviewModeEnded() {
+ if (Shell::Get()->IsSplitViewModeActive())
+ return;
+
+ force_hidden_ = false;
+ UpdateBackdrop();
+}
+
+void BackdropController::OnSplitViewModeStarting() {
+ force_hidden_ = true;
+ Hide();
+}
+
+void BackdropController::OnSplitViewModeEnded() {
force_hidden_ = false;
oshima 2017/06/14 05:45:47 what happen if I close the snapped window while th
xdai1 2017/06/15 22:11:42 I didn't handle it in the previous patch. Handled
UpdateBackdrop();
}
« ash/wm/splitview/split_view_controller.cc ('K') | « ash/wm/workspace/backdrop_controller.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698