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

Unified Diff: chrome/browser/ui/cocoa/location_bar/zoom_decoration_unittest.mm

Issue 671653002: Standardize usage of virtual/override/final in chrome/browser/ui/ (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/cocoa/location_bar/zoom_decoration_unittest.mm
diff --git a/chrome/browser/ui/cocoa/location_bar/zoom_decoration_unittest.mm b/chrome/browser/ui/cocoa/location_bar/zoom_decoration_unittest.mm
index 9418bcc37f92eaa359b5e6f5bb7f8c2a9e25b9cd..ec3316c10b4987ebf94dc9ea1332220503bbf592 100644
--- a/chrome/browser/ui/cocoa/location_bar/zoom_decoration_unittest.mm
+++ b/chrome/browser/ui/cocoa/location_bar/zoom_decoration_unittest.mm
@@ -14,9 +14,9 @@ class MockZoomDecoration : public ZoomDecoration {
public:
explicit MockZoomDecoration(LocationBarViewMac* owner)
: ZoomDecoration(owner), update_ui_count_(0) {}
- virtual bool ShouldShowDecoration() const override { return true; }
- virtual void ShowAndUpdateUI(ZoomController* zoom_controller,
- NSString* tooltip_string) override {
+ bool ShouldShowDecoration() const override { return true; }
+ void ShowAndUpdateUI(ZoomController* zoom_controller,
+ NSString* tooltip_string) override {
++update_ui_count_;
ZoomDecoration::ShowAndUpdateUI(zoom_controller, tooltip_string);
}
@@ -31,7 +31,7 @@ class MockZoomController : public ZoomController {
public:
explicit MockZoomController(content::WebContents* web_contents)
: ZoomController(web_contents) {}
- virtual int GetZoomPercent() const override { return zoom_percent_; }
+ int GetZoomPercent() const override { return zoom_percent_; }
int zoom_percent_;

Powered by Google App Engine
This is Rietveld 408576698