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

Unified Diff: chrome/browser/renderer_context_menu/spelling_menu_observer.h

Issue 1355063004: Template methods on Timer classes instead of the classes themselves. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: timer: fixcaller Created 5 years, 3 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/renderer_context_menu/spelling_menu_observer.h
diff --git a/chrome/browser/renderer_context_menu/spelling_menu_observer.h b/chrome/browser/renderer_context_menu/spelling_menu_observer.h
index a29965de2f727d95129101daa737c520e87fadaf..adfead7494ef92dbffb980b166a7759c4fb976f3 100644
--- a/chrome/browser/renderer_context_menu/spelling_menu_observer.h
+++ b/chrome/browser/renderer_context_menu/spelling_menu_observer.h
@@ -59,8 +59,8 @@ class SpellingMenuObserver : public RenderViewContextMenuObserver {
const std::vector<SpellCheckResult>& results);
private:
- // The callback function for base::RepeatingTimer<SpellingMenuClient>. This
- // function updates the "loading..." animation in the context-menu item.
+ // The callback function for base::RepeatingTimer. This function updates the
+ // "loading..." animation in the context-menu item.
void OnAnimationTimerExpired();
// The interface to add a context-menu item and update it. This class uses
@@ -104,7 +104,7 @@ class SpellingMenuObserver : public RenderViewContextMenuObserver {
scoped_ptr<SpellingServiceClient> client_;
// A timer used for loading animation.
- base::RepeatingTimer<SpellingMenuObserver> animation_timer_;
+ base::RepeatingTimer animation_timer_;
// Flag indicating whether online spelling correction service is enabled. When
// this variable is true and we right-click a misspelled word, we send a
« no previous file with comments | « chrome/browser/profiles/profile_impl.h ('k') | chrome/browser/renderer_host/safe_browsing_resource_throttle.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698