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

Unified Diff: components/bookmarks/browser/bookmark_node.h

Issue 666133002: Standardize usage of virtual/override/final in components/ (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: components/bookmarks/browser/bookmark_node.h
diff --git a/components/bookmarks/browser/bookmark_node.h b/components/bookmarks/browser/bookmark_node.h
index a51c8ef18cc09d019333be0724cce2a3fcd3cbee..ef59ae64becfc2c917e4a9d4e8aa35888d698aa7 100644
--- a/components/bookmarks/browser/bookmark_node.h
+++ b/components/bookmarks/browser/bookmark_node.h
@@ -44,12 +44,12 @@ class BookmarkNode : public ui::TreeNode<BookmarkNode> {
// Creates a new node with |id| and |url|.
BookmarkNode(int64 id, const GURL& url);
- virtual ~BookmarkNode();
+ ~BookmarkNode() override;
// Set the node's internal title. Note that this neither invokes observers
// nor updates any bookmark model this node may be in. For that functionality,
// BookmarkModel::SetTitle(..) should be used instead.
- virtual void SetTitle(const base::string16& title) override;
+ void SetTitle(const base::string16& title) override;
// Returns an unique id for this node.
// For bookmark nodes that are managed by the bookmark model, the IDs are
@@ -195,13 +195,13 @@ class BookmarkNode : public ui::TreeNode<BookmarkNode> {
class BookmarkPermanentNode : public BookmarkNode {
public:
explicit BookmarkPermanentNode(int64 id);
- virtual ~BookmarkPermanentNode();
+ ~BookmarkPermanentNode() override;
// WARNING: this code is used for other projects. Contact noyau@ for details.
void set_visible(bool value) { visible_ = value; }
// BookmarkNode overrides:
- virtual bool IsVisible() const override;
+ bool IsVisible() const override;
private:
bool visible_;
« no previous file with comments | « components/bookmarks/browser/bookmark_model_unittest.cc ('k') | components/bookmarks/browser/bookmark_storage.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698