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

Unified Diff: ui/views/controls/throbber.h

Issue 1103903003: Clean up Throbber. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix spaces Created 5 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
« no previous file with comments | « no previous file | ui/views/controls/throbber.cc » ('j') | ui/views/controls/throbber.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/views/controls/throbber.h
diff --git a/ui/views/controls/throbber.h b/ui/views/controls/throbber.h
index 73eedb9f15e85e7c3392cd3ee3a1bceb19a49687..a136a7d02e16b1487bffe902fb02b55a4578482d 100644
--- a/ui/views/controls/throbber.h
+++ b/ui/views/controls/throbber.h
@@ -41,18 +41,14 @@ class VIEWS_EXPORT Throbber : public View {
void OnPaint(gfx::Canvas* canvas) override;
protected:
- bool running() const { return running_; }
- base::Time start_time() const { return start_time_; }
-
// Specifies whether the throbber is currently animating or not
+ bool IsRunning() const;
+ base::TimeTicks start_time() const { return start_time_; }
private:
- void Run();
-
- bool running_;
bool paint_while_stopped_;
int frame_count_; // How many frames we have.
- base::Time start_time_; // Time when Start was called.
+ base::TimeTicks start_time_; // Time when Start was called.
const gfx::ImageSkia* frames_; // Frames images.
base::TimeDelta frame_time_; // How long one frame is displayed.
base::RepeatingTimer<Throbber> timer_; // Used to schedule Run calls.
« no previous file with comments | « no previous file | ui/views/controls/throbber.cc » ('j') | ui/views/controls/throbber.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698