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

Side by Side Diff: chrome/browser/views/bookmark_bar_view.h

Issue 1961001: Refactors animation to allow for cleaner subclassing. I'm doing this (Closed)
Patch Set: Incorporated review feedback Created 10 years, 7 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 (c) 2010 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2010 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_VIEWS_BOOKMARK_BAR_VIEW_H_ 5 #ifndef CHROME_BROWSER_VIEWS_BOOKMARK_BAR_VIEW_H_
6 #define CHROME_BROWSER_VIEWS_BOOKMARK_BAR_VIEW_H_ 6 #define CHROME_BROWSER_VIEWS_BOOKMARK_BAR_VIEW_H_
7 7
8 #include "app/slide_animation.h" 8 #include "app/slide_animation.h"
9 #include "chrome/browser/bookmarks/bookmark_drag_data.h" 9 #include "chrome/browser/bookmarks/bookmark_drag_data.h"
10 #include "chrome/browser/bookmarks/bookmark_model_observer.h" 10 #include "chrome/browser/bookmarks/bookmark_model_observer.h"
(...skipping 139 matching lines...) Expand 10 before | Expand all | Expand 10 after
150 bool IsAlwaysShown() const; 150 bool IsAlwaysShown() const;
151 151
152 // True if we're on a page where the bookmarks bar is always visible. 152 // True if we're on a page where the bookmarks bar is always visible.
153 bool OnNewTabPage() const; 153 bool OnNewTabPage() const;
154 154
155 // How much we want the bookmark bar to overlap the toolbar. If |return_max| 155 // How much we want the bookmark bar to overlap the toolbar. If |return_max|
156 // is true, we return the maximum overlap rather than the current overlap. 156 // is true, we return the maximum overlap rather than the current overlap.
157 int GetToolbarOverlap(bool return_max); 157 int GetToolbarOverlap(bool return_max);
158 158
159 // Whether or not we are animating. 159 // Whether or not we are animating.
160 bool IsAnimating() { return size_animation_->IsAnimating(); } 160 bool is_animating() { return size_animation_->is_animating(); }
161 161
162 // SlideAnimationDelegate implementation. 162 // SlideAnimationDelegate implementation.
163 void AnimationProgressed(const Animation* animation); 163 void AnimationProgressed(const Animation* animation);
164 void AnimationEnded(const Animation* animation); 164 void AnimationEnded(const Animation* animation);
165 165
166 // BookmarkMenuController::Observer 166 // BookmarkMenuController::Observer
167 virtual void BookmarkMenuDeleted(BookmarkMenuController* controller); 167 virtual void BookmarkMenuDeleted(BookmarkMenuController* controller);
168 168
169 // Returns the button at the specified index. 169 // Returns the button at the specified index.
170 views::TextButton* GetBookmarkButton(int index); 170 views::TextButton* GetBookmarkButton(int index);
(...skipping 312 matching lines...) Expand 10 before | Expand all | Expand 10 after
483 // overflow_button_ or a button on the bar. 483 // overflow_button_ or a button on the bar.
484 views::CustomButton* throbbing_view_; 484 views::CustomButton* throbbing_view_;
485 485
486 // Background for extension toolstrips. 486 // Background for extension toolstrips.
487 SkBitmap toolstrip_background_; 487 SkBitmap toolstrip_background_;
488 488
489 DISALLOW_COPY_AND_ASSIGN(BookmarkBarView); 489 DISALLOW_COPY_AND_ASSIGN(BookmarkBarView);
490 }; 490 };
491 491
492 #endif // CHROME_BROWSER_VIEWS_BOOKMARK_BAR_VIEW_H_ 492 #endif // CHROME_BROWSER_VIEWS_BOOKMARK_BAR_VIEW_H_
OLDNEW
« no previous file with comments | « chrome/browser/views/autocomplete/autocomplete_popup_contents_view.cc ('k') | chrome/browser/views/bookmark_bar_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698