Commit 61fb3a24 authored by Jacobo Aragunde Pérez's avatar Jacobo Aragunde Pérez Committed by Commit Bot

Remove ax::mojom::Role::kMaxValue from switch statement.

Per Mojo docs (//mojo/public/cpp/bindings/README.md#Enums), kMaxValue
shares the value of the highest enumerator of the enum. In this case,
it shares the value of kWindow, that means kWindow is not being dealt
with in that switch statement.

Bug: None
Change-Id: I44a14426772b4b0e3e804e891cf2197a96650088
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2132159Reviewed-by: default avatarDominic Mazzoni <dmazzoni@chromium.org>
Commit-Queue: Jacobo Aragunde Pérez <jaragunde@igalia.com>
Cr-Commit-Position: refs/heads/master@{#756194}
parent 54171d06
......@@ -3727,6 +3727,7 @@ bool AXObject::NameFromContents(bool recursive) const {
case ax::mojom::Role::kVideo:
case ax::mojom::Role::kWebArea:
case ax::mojom::Role::kWebView:
case ax::mojom::Role::kWindow:
result = false;
break;
......@@ -3822,7 +3823,6 @@ bool AXObject::NameFromContents(bool recursive) const {
}
case ax::mojom::Role::kUnknown:
case ax::mojom::Role::kMaxValue:
LOG(ERROR) << "ax::mojom::Role::kUnknown for " << GetNode();
NOTREACHED();
break;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment