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

Unified Diff: Source/modules/accessibility/AXListBoxOption.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/AXListBox.h ('k') | Source/modules/accessibility/AXMediaControls.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/modules/accessibility/AXListBoxOption.h
diff --git a/Source/modules/accessibility/AXListBoxOption.h b/Source/modules/accessibility/AXListBoxOption.h
index 6ea79b0f61e2a7ef30bafa06a1c3d8e273008d7f..74db3c20c7710d40abbce61073091fa32d372537 100644
--- a/Source/modules/accessibility/AXListBoxOption.h
+++ b/Source/modules/accessibility/AXListBoxOption.h
@@ -45,21 +45,21 @@ private:
public:
static PassRefPtrWillBeRawPtr<AXListBoxOption> create(LayoutObject*, AXObjectCacheImpl&);
- virtual ~AXListBoxOption();
+ ~AXListBoxOption() override;
- virtual bool isAXListBoxOption() const override { return true; }
- virtual AccessibilityRole determineAccessibilityRole() override final;
- virtual bool isSelected() const override;
- virtual bool isEnabled() const override;
- virtual bool isSelectedOptionActive() const override;
- virtual void setSelected(bool) override;
- virtual bool canSetSelectedAttribute() const override;
- virtual String stringValue() const override;
- virtual String deprecatedTitle(TextUnderElementMode) const override { return String(); }
+ bool isAXListBoxOption() const override { return true; }
+ AccessibilityRole determineAccessibilityRole() final;
+ bool isSelected() const override;
+ bool isEnabled() const override;
+ bool isSelectedOptionActive() const override;
+ void setSelected(bool) override;
+ bool canSetSelectedAttribute() const override;
+ String stringValue() const override;
+ String deprecatedTitle(TextUnderElementMode) const override { return String(); }
private:
- virtual bool canHaveChildren() const override { return false; }
- virtual bool computeAccessibilityIsIgnored(IgnoredReasons* = nullptr) const override;
+ bool canHaveChildren() const override { return false; }
+ bool computeAccessibilityIsIgnored(IgnoredReasons* = nullptr) const override;
HTMLSelectElement* listBoxOptionParentNode() const;
int listBoxOptionIndex() const;
« no previous file with comments | « Source/modules/accessibility/AXListBox.h ('k') | Source/modules/accessibility/AXMediaControls.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698