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

Unified Diff: chrome/browser/ui/cocoa/toolbar/app_toolbar_button.h

Issue 2859903003: [Mac] App Menu Animated Icon (Closed)
Patch Set: Clean up Created 3 years, 8 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: chrome/browser/ui/cocoa/toolbar/app_toolbar_button.h
diff --git a/chrome/browser/ui/cocoa/toolbar/app_toolbar_button.h b/chrome/browser/ui/cocoa/toolbar/app_toolbar_button.h
index bc4ef6894e4772e6c23def8777c6944be04aa079..3040d90d7efa88eb9a6930e45d7c0c5fc39f3d81 100644
--- a/chrome/browser/ui/cocoa/toolbar/app_toolbar_button.h
+++ b/chrome/browser/ui/cocoa/toolbar/app_toolbar_button.h
@@ -9,20 +9,29 @@
#include <memory>
+#import "base/mac/scoped_nsobject.h"
Robert Sesek 2017/05/03 22:44:29 Unused?
#import "chrome/browser/ui/cocoa/menu_button.h"
#include "chrome/browser/ui/toolbar/app_menu_icon_controller.h"
+class AppToolbarButtonIcon;
+
// Button for the app toolbar button.
@interface AppToolbarButton : MenuButton {
@private
AppMenuIconController::Severity severity_;
AppMenuIconController::IconType type_;
+
+ // Used for animating and drawing the app menu icon.
+ std::unique_ptr<AppToolbarButtonIcon> app_icon_;
Robert Sesek 2017/05/03 22:44:29 naming: appIcon
}
- (void)setSeverity:(AppMenuIconController::Severity)severity
iconType:(AppMenuIconController::IconType)iconType
shouldAnimate:(BOOL)shouldAnimate;
+// Called when a new tab had been inserted into the foreground.
+- (void)onTabInsertedInForeground;
+
@end
#endif // CHROME_BROWSER_UI_COCOA_TOOLBAR_APP_TOOLBAR_BUTTON_H_

Powered by Google App Engine
This is Rietveld 408576698