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

Unified Diff: chrome/browser/ui/views/tabs/tab.h

Issue 1393193002: Paint tab-loading throbbers into a ui::Layer. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: overrides -> private Created 5 years, 2 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 | chrome/browser/ui/views/tabs/tab.cc » ('j') | chrome/browser/ui/views/tabs/tab.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/views/tabs/tab.h
diff --git a/chrome/browser/ui/views/tabs/tab.h b/chrome/browser/ui/views/tabs/tab.h
index 45dcae6ad61de1c593402c65b8468375f1a584bd..179e457fe7fad5859dd9ca24314c8046b90dc82a 100644
--- a/chrome/browser/ui/views/tabs/tab.h
+++ b/chrome/browser/ui/views/tabs/tab.h
@@ -89,8 +89,7 @@ class Tab : public gfx::AnimationDelegate,
void SetData(const TabRendererData& data);
const TabRendererData& data() const { return data_; }
- // Sets the network state. If the network state changes NetworkStateChanged is
- // invoked.
+ // Sets the network state.
void UpdateLoadingAnimation(TabRendererData::NetworkState state);
// Starts/Stops a pulse animation.
@@ -163,7 +162,9 @@ class Tab : public gfx::AnimationDelegate,
// The animation object used to swap the favicon with the sad tab icon.
class FaviconCrashAnimation;
+
class TabCloseButton;
+ class ThrobberView;
// Contains a cached image and the values used to generate it.
struct ImageCacheEntry {
@@ -246,8 +247,7 @@ class Tab : public gfx::AnimationDelegate,
void PaintIcon(gfx::Canvas* canvas);
// Invoked if data_.network_state changes, or the network_state is not none.
- void AdvanceLoadingAnimation(TabRendererData::NetworkState old_state,
- TabRendererData::NetworkState state);
+ void AdvanceLoadingAnimation(TabRendererData::NetworkState state);
// Returns the number of favicon-size elements that can fit in the tab's
// current size.
@@ -341,15 +341,6 @@ class Tab : public gfx::AnimationDelegate,
// crashes.
int favicon_hiding_offset_;
- // The point in time when the tab icon was first painted in the waiting state.
- base::TimeTicks waiting_start_time_;
-
- // The point in time when the tab icon was first painted in the loading state.
- base::TimeTicks loading_start_time_;
-
- // Paint state for the throbber after the most recent waiting paint.
- gfx::ThrobberWaitingState waiting_state_;
-
// Step in the immersive loading progress indicator.
int immersive_loading_step_;
@@ -365,6 +356,7 @@ class Tab : public gfx::AnimationDelegate,
scoped_refptr<gfx::AnimationContainer> animation_container_;
+ ThrobberView* throbber_;
MediaIndicatorButton* media_indicator_button_;
views::ImageButton* close_button_;
views::Label* title_;
« no previous file with comments | « no previous file | chrome/browser/ui/views/tabs/tab.cc » ('j') | chrome/browser/ui/views/tabs/tab.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698