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

Unified Diff: Source/modules/accessibility/AXMenuList.h

Issue 1227783004: Fix virtual/override/final usage in Source/modules/. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: rebase Created 5 years, 5 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 | « Source/modules/accessibility/AXMediaControls.h ('k') | Source/modules/accessibility/AXMenuListOption.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/modules/accessibility/AXMenuList.h
diff --git a/Source/modules/accessibility/AXMenuList.h b/Source/modules/accessibility/AXMenuList.h
index e3696fd007217d9f39d5da908a09b36a3d53118a..9a9a3b647612229befe6df18523d2664999de57f 100644
--- a/Source/modules/accessibility/AXMenuList.h
+++ b/Source/modules/accessibility/AXMenuList.h
@@ -37,10 +37,10 @@ class AXMenuList final : public AXLayoutObject {
public:
static PassRefPtrWillBeRawPtr<AXMenuList> create(LayoutMenuList* layoutObject, AXObjectCacheImpl&);
- virtual bool isCollapsed() const override;
- virtual AccessibilityExpanded isExpanded() const override final;
- virtual bool press() const override;
- virtual void clearChildren() override;
+ bool isCollapsed() const override;
+ AccessibilityExpanded isExpanded() const final;
+ bool press() const override;
+ void clearChildren() override;
void didUpdateActiveOption(int optionIndex);
void didShowPopup();
@@ -49,11 +49,11 @@ public:
private:
AXMenuList(LayoutMenuList*, AXObjectCacheImpl&);
- virtual bool isMenuList() const override { return true; }
- virtual AccessibilityRole determineAccessibilityRole() override final;
- virtual bool canSetFocusAttribute() const override;
+ bool isMenuList() const override { return true; }
+ AccessibilityRole determineAccessibilityRole() final;
+ bool canSetFocusAttribute() const override;
- virtual void addChildren() override;
+ void addChildren() override;
};
DEFINE_AX_OBJECT_TYPE_CASTS(AXMenuList, isMenuList());
« no previous file with comments | « Source/modules/accessibility/AXMediaControls.h ('k') | Source/modules/accessibility/AXMenuListOption.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698