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

Side by Side Diff: chrome/browser/ui/views/frame/immersive_mode_controller_ash.h

Issue 2251213004: Moves ImmersiveFullscreenController::SetupForTest into test class (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: feedback Created 4 years, 4 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 unified diff | Download patch
OLDNEW
1 // Copyright 2012 The Chromium Authors. All rights reserved. 1 // Copyright 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_BROWSER_UI_VIEWS_FRAME_IMMERSIVE_MODE_CONTROLLER_ASH_H_ 5 #ifndef CHROME_BROWSER_UI_VIEWS_FRAME_IMMERSIVE_MODE_CONTROLLER_ASH_H_
6 #define CHROME_BROWSER_UI_VIEWS_FRAME_IMMERSIVE_MODE_CONTROLLER_ASH_H_ 6 #define CHROME_BROWSER_UI_VIEWS_FRAME_IMMERSIVE_MODE_CONTROLLER_ASH_H_
7 7
8 #include <memory> 8 #include <memory>
9 9
10 #include "ash/common/wm/immersive/wm_immersive_fullscreen_controller_delegate.h" 10 #include "ash/common/wm/immersive/wm_immersive_fullscreen_controller_delegate.h"
(...skipping 11 matching lines...) Expand all
22 22
23 class ImmersiveModeControllerAsh 23 class ImmersiveModeControllerAsh
24 : public ImmersiveModeController, 24 : public ImmersiveModeController,
25 public ash::WmImmersiveFullscreenControllerDelegate, 25 public ash::WmImmersiveFullscreenControllerDelegate,
26 public ash::wm::WindowStateObserver, 26 public ash::wm::WindowStateObserver,
27 public content::NotificationObserver { 27 public content::NotificationObserver {
28 public: 28 public:
29 ImmersiveModeControllerAsh(); 29 ImmersiveModeControllerAsh();
30 ~ImmersiveModeControllerAsh() override; 30 ~ImmersiveModeControllerAsh() override;
31 31
32 ash::ImmersiveFullscreenController* controller() { return controller_.get(); }
33
32 // ImmersiveModeController overrides: 34 // ImmersiveModeController overrides:
33 void Init(BrowserView* browser_view) override; 35 void Init(BrowserView* browser_view) override;
34 void SetEnabled(bool enabled) override; 36 void SetEnabled(bool enabled) override;
35 bool IsEnabled() const override; 37 bool IsEnabled() const override;
36 bool ShouldHideTabIndicators() const override; 38 bool ShouldHideTabIndicators() const override;
37 bool ShouldHideTopViews() const override; 39 bool ShouldHideTopViews() const override;
38 bool IsRevealed() const override; 40 bool IsRevealed() const override;
39 int GetTopContainerVerticalOffset( 41 int GetTopContainerVerticalOffset(
40 const gfx::Size& top_container_size) const override; 42 const gfx::Size& top_container_size) const override;
41 ImmersiveRevealedLock* GetRevealedLock(AnimateReveal animate_reveal) override 43 ImmersiveRevealedLock* GetRevealedLock(AnimateReveal animate_reveal) override
42 WARN_UNUSED_RESULT; 44 WARN_UNUSED_RESULT;
43 void OnFindBarVisibleBoundsChanged( 45 void OnFindBarVisibleBoundsChanged(
44 const gfx::Rect& new_visible_bounds_in_screen) override; 46 const gfx::Rect& new_visible_bounds_in_screen) override;
45 void SetupForTest() override;
46 47
47 private: 48 private:
48 // Enables or disables observers for window restore and entering / exiting 49 // Enables or disables observers for window restore and entering / exiting
49 // tab fullscreen. 50 // tab fullscreen.
50 void EnableWindowObservers(bool enable); 51 void EnableWindowObservers(bool enable);
51 52
52 // Updates the browser root view's layout including window caption controls. 53 // Updates the browser root view's layout including window caption controls.
53 void LayoutBrowserRootView(); 54 void LayoutBrowserRootView();
54 55
55 // Updates whether the tab strip is painted in a short "light bar" style. 56 // Updates whether the tab strip is painted in a short "light bar" style.
(...skipping 39 matching lines...) Expand 10 before | Expand all | Expand 10 after
95 // the top-of-window views are not revealed regardless of 96 // the top-of-window views are not revealed regardless of
96 // |use_tab_indicators_|. 97 // |use_tab_indicators_|.
97 double visible_fraction_; 98 double visible_fraction_;
98 99
99 content::NotificationRegistrar registrar_; 100 content::NotificationRegistrar registrar_;
100 101
101 DISALLOW_COPY_AND_ASSIGN(ImmersiveModeControllerAsh); 102 DISALLOW_COPY_AND_ASSIGN(ImmersiveModeControllerAsh);
102 }; 103 };
103 104
104 #endif // CHROME_BROWSER_UI_VIEWS_FRAME_IMMERSIVE_MODE_CONTROLLER_ASH_H_ 105 #endif // CHROME_BROWSER_UI_VIEWS_FRAME_IMMERSIVE_MODE_CONTROLLER_ASH_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698