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

Unified Diff: components/exo/shell_surface.cc

Issue 2687683004: exo: Do not call UpdateShelfStateForFullscreenChange() in ShellSurface (Closed)
Patch Set: Created 3 years, 10 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/exo/shell_surface.cc
diff --git a/components/exo/shell_surface.cc b/components/exo/shell_surface.cc
index 82f2f5db68fdedaf96c12cd14dc1f7d468b5c088..715e92ebe97116aad451dbc5fc23c852c3beafb7 100644
--- a/components/exo/shell_surface.cc
+++ b/components/exo/shell_surface.cc
@@ -11,7 +11,6 @@
#include "ash/common/wm/window_resizer.h"
#include "ash/common/wm/window_state.h"
#include "ash/common/wm/window_state_delegate.h"
-#include "ash/common/wm_shell.h"
#include "ash/common/wm_window.h"
#include "ash/public/cpp/shell_window_ids.h"
#include "ash/wm/window_state_aura.h"
@@ -57,11 +56,6 @@ const struct Accelerator {
{ui::VKEY_W, ui::EF_SHIFT_DOWN | ui::EF_CONTROL_DOWN},
{ui::VKEY_F4, ui::EF_ALT_DOWN}};
-void UpdateShelfStateForFullscreenChange(views::Widget* widget) {
- for (ash::WmWindow* root_window : ash::WmShell::Get()->GetAllRootWindows())
- ash::WmShelf::ForWindow(root_window)->UpdateVisibilityState();
-}
-
class CustomFrameView : public views::NonClientFrameView {
public:
explicit CustomFrameView(views::Widget* widget) : widget_(widget) {}
@@ -171,7 +165,6 @@ class CustomWindowStateDelegate : public ash::wm::WindowStateDelegate,
ash::wm::WindowState* window_state =
ash::wm::GetWindowState(widget_->GetNativeWindow());
window_state->set_in_immersive_fullscreen(enter_fullscreen);
- UpdateShelfStateForFullscreenChange(widget_);
}
return true;
}
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698