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

Unified Diff: content/browser/host_zoom_map_impl.h

Issue 667943003: Standardize usage of virtual/override/final in content/browser/ (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: content/browser/host_zoom_map_impl.h
diff --git a/content/browser/host_zoom_map_impl.h b/content/browser/host_zoom_map_impl.h
index 464d424ff4382fa7999779ab53a6e183a8dc5f8a..d2a8a67c1247c7df9213bf1687b9856cb06b364d 100644
--- a/content/browser/host_zoom_map_impl.h
+++ b/content/browser/host_zoom_map_impl.h
@@ -28,35 +28,31 @@ class CONTENT_EXPORT HostZoomMapImpl : public NON_EXPORTED_BASE(HostZoomMap),
public base::SupportsUserData::Data {
public:
HostZoomMapImpl();
- virtual ~HostZoomMapImpl();
+ ~HostZoomMapImpl() override;
// HostZoomMap implementation:
- virtual void CopyFrom(HostZoomMap* copy) override;
- virtual double GetZoomLevelForHostAndScheme(
- const std::string& scheme,
- const std::string& host) const override;
+ void CopyFrom(HostZoomMap* copy) override;
+ double GetZoomLevelForHostAndScheme(const std::string& scheme,
+ const std::string& host) const override;
// TODO(wjmaclean) Should we use a GURL here? crbug.com/384486
- virtual bool HasZoomLevel(const std::string& scheme,
- const std::string& host) const override;
- virtual ZoomLevelVector GetAllZoomLevels() const override;
- virtual void SetZoomLevelForHost(
- const std::string& host,
- double level) override;
- virtual void SetZoomLevelForHostAndScheme(
- const std::string& scheme,
- const std::string& host,
- double level) override;
- virtual bool UsesTemporaryZoomLevel(int render_process_id,
- int render_view_id) const override;
- virtual void SetTemporaryZoomLevel(int render_process_id,
- int render_view_id,
- double level) override;
-
- virtual void ClearTemporaryZoomLevel(int render_process_id,
- int render_view_id) override;
- virtual double GetDefaultZoomLevel() const override;
- virtual void SetDefaultZoomLevel(double level) override;
- virtual scoped_ptr<Subscription> AddZoomLevelChangedCallback(
+ bool HasZoomLevel(const std::string& scheme,
+ const std::string& host) const override;
+ ZoomLevelVector GetAllZoomLevels() const override;
+ void SetZoomLevelForHost(const std::string& host, double level) override;
+ void SetZoomLevelForHostAndScheme(const std::string& scheme,
+ const std::string& host,
+ double level) override;
+ bool UsesTemporaryZoomLevel(int render_process_id,
+ int render_view_id) const override;
+ void SetTemporaryZoomLevel(int render_process_id,
+ int render_view_id,
+ double level) override;
+
+ void ClearTemporaryZoomLevel(int render_process_id,
+ int render_view_id) override;
+ double GetDefaultZoomLevel() const override;
+ void SetDefaultZoomLevel(double level) override;
+ scoped_ptr<Subscription> AddZoomLevelChangedCallback(
const ZoomLevelChangedCallback& callback) override;
// Returns the current zoom level for the specified WebContents. This may
@@ -86,9 +82,9 @@ class CONTENT_EXPORT HostZoomMapImpl : public NON_EXPORTED_BASE(HostZoomMap),
int render_view_id) const;
// NotificationObserver implementation.
- virtual void Observe(int type,
- const NotificationSource& source,
- const NotificationDetails& details) override;
+ void Observe(int type,
+ const NotificationSource& source,
+ const NotificationDetails& details) override;
private:
typedef std::map<std::string, double> HostZoomLevels;
« no previous file with comments | « content/browser/histogram_synchronizer.h ('k') | content/browser/indexed_db/indexed_db_active_blob_registry_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698