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

Unified Diff: chrome/browser/ui/cocoa/fullscreen/fullscreen_toolbar_controller.h

Issue 2516803002: [Mac] Tests for FullscreenToolbarController (Closed)
Patch Set: Fix for rsesek Created 4 years, 1 month 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: chrome/browser/ui/cocoa/fullscreen/fullscreen_toolbar_controller.h
diff --git a/chrome/browser/ui/cocoa/fullscreen/fullscreen_toolbar_controller.h b/chrome/browser/ui/cocoa/fullscreen/fullscreen_toolbar_controller.h
index 35b06126d62cb6d8b8594ea101c9db13964c28a8..19a8c712f58205670bc5fb810c86d45d0a6f019e 100644
--- a/chrome/browser/ui/cocoa/fullscreen/fullscreen_toolbar_controller.h
+++ b/chrome/browser/ui/cocoa/fullscreen/fullscreen_toolbar_controller.h
@@ -48,7 +48,6 @@ struct FullscreenToolbarLayout {
// tracking, and mouse tracking associated with the toolbar. It receives input
// from these objects to update and recompute the fullscreen toolbar laytout.
-// TODO (spqchan): Write tests for this class. See crbug.com/640064.
@interface FullscreenToolbarController : NSObject {
@private
// Our parent controller.
@@ -131,11 +130,24 @@ struct FullscreenToolbarLayout {
// Private methods exposed for testing.
@interface FullscreenToolbarController (ExposedForTesting)
-// Adjusts the AppKit Fullscreen options of the application.
-- (void)setSystemFullscreenModeTo:(base::mac::FullScreenMode)mode;
-// Callback for menu bar animations.
-- (void)setMenuBarRevealProgress:(CGFloat)progress;
+// Returns |animationController_|.
+- (FullscreenToolbarAnimationController*)animationController;
+
+// Allows tests to set a mock FullscreenMenubarTracker object.
+- (void)setMenubarTracker:(FullscreenMenubarTracker*)tracker;
+
+// Allows tests to set a mock FullscreenToolbarMouseTracker object.
+- (void)setMouseTracker:(FullscreenToolbarMouseTracker*)tracker;
+
+// Sets the value of |toolbarStyle_|.
+- (void)setToolbarStyle:(FullscreenToolbarStyle)style;
+
+// Sets the value of |inFullscreenMode_|.
+- (void)setTestFullscreenMode:(BOOL)isInFullscreen;
+
+// Returns |kToolbarVerticalOffset|.
+- (CGFloat)toolbarVerticalOffset;
@end

Powered by Google App Engine
This is Rietveld 408576698