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

Unified Diff: Source/modules/accessibility/AXMenuListPopup.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/AXMenuListOption.h ('k') | Source/modules/accessibility/AXMockObject.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/modules/accessibility/AXMenuListPopup.h
diff --git a/Source/modules/accessibility/AXMenuListPopup.h b/Source/modules/accessibility/AXMenuListPopup.h
index ff2a6b6aba799ddb841792e4955a9a04a25d6b8e..da11c7fb1497338f9c1395ed46fa42388b247789 100644
--- a/Source/modules/accessibility/AXMenuListPopup.h
+++ b/Source/modules/accessibility/AXMenuListPopup.h
@@ -38,8 +38,8 @@ class AXMenuListPopup final : public AXMockObject {
public:
static PassRefPtrWillBeRawPtr<AXMenuListPopup> create(AXObjectCacheImpl& axObjectCache) { return adoptRefWillBeNoop(new AXMenuListPopup(axObjectCache)); }
- virtual bool isEnabled() const override;
- virtual bool isOffScreen() const override;
+ bool isEnabled() const override;
+ bool isOffScreen() const override;
void didUpdateActiveOption(int optionIndex);
void didShow();
@@ -49,16 +49,16 @@ public:
private:
explicit AXMenuListPopup(AXObjectCacheImpl&);
- virtual bool isMenuListPopup() const override { return true; }
+ bool isMenuListPopup() const override { return true; }
- virtual LayoutRect elementRect() const override { return LayoutRect(); }
- virtual AccessibilityRole roleValue() const override { return MenuListPopupRole; }
+ LayoutRect elementRect() const override { return LayoutRect(); }
+ AccessibilityRole roleValue() const override { return MenuListPopupRole; }
- virtual bool isVisible() const override;
- virtual bool press() const override;
- virtual void addChildren() override;
- virtual void updateChildrenIfNecessary() override;
- virtual bool computeAccessibilityIsIgnored(IgnoredReasons* = nullptr) const override;
+ bool isVisible() const override;
+ bool press() const override;
+ void addChildren() override;
+ void updateChildrenIfNecessary() override;
+ bool computeAccessibilityIsIgnored(IgnoredReasons* = nullptr) const override;
AXMenuListOption* menuListOptionAXObject(HTMLElement*) const;
int getSelectedIndex() const;
« no previous file with comments | « Source/modules/accessibility/AXMenuListOption.h ('k') | Source/modules/accessibility/AXMockObject.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698