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

Unified Diff: ui/accessibility/ax_node_data.cc

Issue 1549313002: Add a missing switch-case break in Accessibility (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/accessibility/ax_node_data.cc
diff --git a/ui/accessibility/ax_node_data.cc b/ui/accessibility/ax_node_data.cc
index 3b263a9d9617c1af6f1bd11d1641d93f5738a1fc..fe1b3db56cb4507198d29babc33b33e545f45eae 100644
--- a/ui/accessibility/ax_node_data.cc
+++ b/ui/accessibility/ax_node_data.cc
@@ -430,22 +430,23 @@ std::string AXNodeData::ToString() const {
result += " text_direction=btt";
break;
}
- case AX_ATTR_TEXT_STYLE: {
- unsigned int text_style = int_attributes[i].second;
- if (text_style == AX_TEXT_STYLE_NONE)
- break;
- std::string text_style_value(" text_style=");
- if (text_style & AX_TEXT_STYLE_BOLD)
- text_style_value += "bold,";
- if (text_style & AX_TEXT_STYLE_ITALIC)
- text_style_value += "italic,";
- if (text_style & AX_TEXT_STYLE_UNDERLINE)
- text_style_value += "underline,";
- if (text_style & AX_TEXT_STYLE_LINE_THROUGH)
- text_style_value += "line-through,";
- result += text_style_value.substr(0, text_style_value.size() - 1);;
- }
break;
+ case AX_ATTR_TEXT_STYLE: {
+ unsigned int text_style = int_attributes[i].second;
+ if (text_style == AX_TEXT_STYLE_NONE)
+ break;
+ std::string text_style_value(" text_style=");
+ if (text_style & AX_TEXT_STYLE_BOLD)
+ text_style_value += "bold,";
+ if (text_style & AX_TEXT_STYLE_ITALIC)
+ text_style_value += "italic,";
+ if (text_style & AX_TEXT_STYLE_UNDERLINE)
+ text_style_value += "underline,";
+ if (text_style & AX_TEXT_STYLE_LINE_THROUGH)
+ text_style_value += "line-through,";
+ result += text_style_value.substr(0, text_style_value.size() - 1);
+ break;
+ }
case AX_ATTR_SET_SIZE:
result += " setsize=" + value;
break;
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698