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

Unified Diff: chrome/browser/ui/views/tabs/tab_strip.cc

Issue 682763002: Standardize usage of virtual/override/final specifiers. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 6 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 | « chrome/browser/ui/views/tabs/tab_strip.h ('k') | chrome/browser/ui/views/tabs/tab_strip_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/views/tabs/tab_strip.cc
diff --git a/chrome/browser/ui/views/tabs/tab_strip.cc b/chrome/browser/ui/views/tabs/tab_strip.cc
index 0ee6c721f463a5da34f0482a75ce0003e279f50f..4a9423f670367fe76eb96891ebbfe833c73f7d15 100644
--- a/chrome/browser/ui/views/tabs/tab_strip.cc
+++ b/chrome/browser/ui/views/tabs/tab_strip.cc
@@ -134,9 +134,9 @@ base::string16 GetClipboardText() {
class TabAnimationDelegate : public gfx::AnimationDelegate {
public:
TabAnimationDelegate(TabStrip* tab_strip, Tab* tab);
- virtual ~TabAnimationDelegate();
+ ~TabAnimationDelegate() override;
- virtual void AnimationProgressed(const gfx::Animation* animation) override;
+ void AnimationProgressed(const gfx::Animation* animation) override;
protected:
TabStrip* tab_strip() { return tab_strip_; }
@@ -167,10 +167,10 @@ void TabAnimationDelegate::AnimationProgressed(
class ResetDraggingStateDelegate : public TabAnimationDelegate {
public:
ResetDraggingStateDelegate(TabStrip* tab_strip, Tab* tab);
- virtual ~ResetDraggingStateDelegate();
+ ~ResetDraggingStateDelegate() override;
- virtual void AnimationEnded(const gfx::Animation* animation) override;
- virtual void AnimationCanceled(const gfx::Animation* animation) override;
+ void AnimationEnded(const gfx::Animation* animation) override;
+ void AnimationCanceled(const gfx::Animation* animation) override;
private:
DISALLOW_COPY_AND_ASSIGN(ResetDraggingStateDelegate);
@@ -236,7 +236,7 @@ class NewTabButton : public views::ImageButton,
public views::MaskedTargeterDelegate {
public:
NewTabButton(TabStrip* tab_strip, views::ButtonListener* listener);
- virtual ~NewTabButton();
+ ~NewTabButton() override;
// Set the background offset used to match the background image to the frame
// image.
@@ -249,14 +249,14 @@ class NewTabButton : public views::ImageButton,
#if defined(OS_WIN)
virtual void OnMouseReleased(const ui::MouseEvent& event) override;
#endif
- virtual void OnPaint(gfx::Canvas* canvas) override;
+ void OnPaint(gfx::Canvas* canvas) override;
// ui::EventHandler:
- virtual void OnGestureEvent(ui::GestureEvent* event) override;
+ void OnGestureEvent(ui::GestureEvent* event) override;
private:
// views::MaskedTargeterDelegate:
- virtual bool GetHitTestMask(gfx::Path* mask) const override;
+ bool GetHitTestMask(gfx::Path* mask) const override;
bool ShouldWindowContentsBeTransparent() const;
gfx::ImageSkia GetBackgroundImage(views::CustomButton::ButtonState state,
@@ -476,8 +476,8 @@ class TabStrip::RemoveTabDelegate : public TabAnimationDelegate {
public:
RemoveTabDelegate(TabStrip* tab_strip, Tab* tab);
- virtual void AnimationEnded(const gfx::Animation* animation) override;
- virtual void AnimationCanceled(const gfx::Animation* animation) override;
+ void AnimationEnded(const gfx::Animation* animation) override;
+ void AnimationCanceled(const gfx::Animation* animation) override;
private:
DISALLOW_COPY_AND_ASSIGN(RemoveTabDelegate);
« no previous file with comments | « chrome/browser/ui/views/tabs/tab_strip.h ('k') | chrome/browser/ui/views/tabs/tab_strip_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698