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

Unified Diff: chrome/browser/ui/views/frame/opaque_browser_frame_view_layout_unittest.cc

Issue 628773002: replace OVERRIDE and FINAL with override and final in chrome/browser/ui/[t-v]* (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: 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
Index: chrome/browser/ui/views/frame/opaque_browser_frame_view_layout_unittest.cc
diff --git a/chrome/browser/ui/views/frame/opaque_browser_frame_view_layout_unittest.cc b/chrome/browser/ui/views/frame/opaque_browser_frame_view_layout_unittest.cc
index b42f92d5f2676706351d872e6e02eeca04cc504e..238565d716816be5690c1728c38e95bb2cedd172 100644
--- a/chrome/browser/ui/views/frame/opaque_browser_frame_view_layout_unittest.cc
+++ b/chrome/browser/ui/views/frame/opaque_browser_frame_view_layout_unittest.cc
@@ -65,45 +65,45 @@ class TestLayoutDelegate : public OpaqueBrowserFrameViewLayoutDelegate {
// OpaqueBrowserFrameViewLayoutDelegate overrides:
- virtual bool ShouldShowWindowIcon() const OVERRIDE {
+ virtual bool ShouldShowWindowIcon() const override {
return !window_title_.empty();
}
- virtual bool ShouldShowWindowTitle() const OVERRIDE {
+ virtual bool ShouldShowWindowTitle() const override {
return !window_title_.empty();
}
- virtual base::string16 GetWindowTitle() const OVERRIDE {
+ virtual base::string16 GetWindowTitle() const override {
return window_title_;
}
- virtual int GetIconSize() const OVERRIDE {
+ virtual int GetIconSize() const override {
// The value on linux_aura and non-aura windows.
return 17;
}
- virtual bool ShouldLeaveOffsetNearTopBorder() const OVERRIDE {
+ virtual bool ShouldLeaveOffsetNearTopBorder() const override {
return !IsMaximized();
}
- virtual gfx::Size GetBrowserViewMinimumSize() const OVERRIDE {
+ virtual gfx::Size GetBrowserViewMinimumSize() const override {
// Taken from a calculation in BrowserViewLayout.
return gfx::Size(168, 64);
}
- virtual bool ShouldShowCaptionButtons() const OVERRIDE {
+ virtual bool ShouldShowCaptionButtons() const override {
return show_caption_buttons_;
}
- virtual bool ShouldShowAvatar() const OVERRIDE {
+ virtual bool ShouldShowAvatar() const override {
return show_avatar_;
}
- virtual bool IsRegularOrGuestSession() const OVERRIDE {
+ virtual bool IsRegularOrGuestSession() const override {
return true;
}
- virtual gfx::ImageSkia GetOTRAvatarIcon() const OVERRIDE {
+ virtual gfx::ImageSkia GetOTRAvatarIcon() const override {
// The calculations depend on the size of the OTR resource, and chromeos
// uses a different sized image, so hard code the size of the current
// windows/linux one.
@@ -112,27 +112,27 @@ class TestLayoutDelegate : public OpaqueBrowserFrameViewLayoutDelegate {
return image;
}
- virtual bool IsMaximized() const OVERRIDE {
+ virtual bool IsMaximized() const override {
return window_state_ == STATE_MAXIMIZED;
}
- virtual bool IsMinimized() const OVERRIDE {
+ virtual bool IsMinimized() const override {
return window_state_ == STATE_MINIMIZED;
}
- virtual bool IsFullscreen() const OVERRIDE {
+ virtual bool IsFullscreen() const override {
return window_state_ == STATE_FULLSCREEN;
}
- virtual bool IsTabStripVisible() const OVERRIDE {
+ virtual bool IsTabStripVisible() const override {
return window_title_.empty();
}
- virtual int GetTabStripHeight() const OVERRIDE {
+ virtual int GetTabStripHeight() const override {
return IsTabStripVisible() ? Tab::GetMinimumUnselectedSize().height() : 0;
}
- virtual gfx::Size GetTabstripPreferredSize() const OVERRIDE {
+ virtual gfx::Size GetTabstripPreferredSize() const override {
// Measured from Tabstrip::GetPreferredSize().
return IsTabStripVisible() ? gfx::Size(78, 29) : gfx::Size(0, 0);
}
@@ -153,7 +153,7 @@ class OpaqueBrowserFrameViewLayoutTest : public views::ViewsTestBase {
OpaqueBrowserFrameViewLayoutTest() {}
virtual ~OpaqueBrowserFrameViewLayoutTest() {}
- virtual void SetUp() OVERRIDE {
+ virtual void SetUp() override {
views::ViewsTestBase::SetUp();
delegate_.reset(new TestLayoutDelegate);
@@ -183,7 +183,7 @@ class OpaqueBrowserFrameViewLayoutTest : public views::ViewsTestBase {
VIEW_ID_CLOSE_BUTTON, gfx::Size(43, 18));
}
- virtual void TearDown() OVERRIDE {
+ virtual void TearDown() override {
widget_->CloseNow();
views::ViewsTestBase::TearDown();

Powered by Google App Engine
This is Rietveld 408576698