Handling of different types of empty alt

content/browser/accessibility/accessibility_tree_formatter_auralinux.cc

 // Copyright (c) 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/accessibility/accessibility_tree_formatter.h"
 
 #include <atk/atk.h>
 
 #include "base/logging.h"
 #include "base/strings/string_number_conversions.h"
@@ skipping 33 matching lines @@
 void AccessibilityTreeFormatterAuraLinux::AddProperties(
     const BrowserAccessibility& node,
     base::DictionaryValue* dict) {
   dict->SetInteger("id", node.GetId());
   BrowserAccessibilityAuraLinux* acc_obj =
       ToBrowserAccessibilityAuraLinux(const_cast<BrowserAccessibility*>(&node));
 
   AtkObject* atk_object = acc_obj->GetAtkObject();
   AtkRole role = acc_obj->atk_role();
   if (role != ATK_ROLE_UNKNOWN)
-    dict->SetString("role", atk_role_get_name(role));
-  dict->SetString("name", atk_object_get_name(atk_object));
-  dict->SetString("description", atk_object_get_description(atk_object));
+    dict->SetString("role", std::string(atk_role_get_name(role)));
+  const gchar* name = atk_object_get_name(atk_object);
+  if (name)
+    dict->SetString("name", std::string(name));
+  const gchar* description = atk_object_get_description(atk_object);
+  if (description)
+    dict->SetString("description", std::string(description));
+
   AtkStateSet* state_set = atk_object_ref_state_set(atk_object);
   base::ListValue* states = new base::ListValue;
   for (int i = ATK_STATE_INVALID; i < ATK_STATE_LAST_DEFINED; i++) {
     AtkStateType state_type = static_cast<AtkStateType>(i);
     if (atk_state_set_contains_state(state_set, state_type))
       states->AppendString(atk_state_type_get_name(state_type));
   }
   dict->Set("states", states);
 }
 
 base::string16 AccessibilityTreeFormatterAuraLinux::ToString(
     const base::DictionaryValue& node) {
   base::string16 line;
   std::string role_value;
   node.GetString("role", &role_value);
   if (!role_value.empty()) {
     WriteAttribute(true, base::StringPrintf("[%s]", role_value.c_str()), &line);
   }
 
   std::string name_value;
-  node.GetString("name", &name_value);
-  WriteAttribute(true, base::StringPrintf("name='%s'", name_value.c_str()),
-                 &line);
+  if (node.GetString("name", &name_value))
+    WriteAttribute(true, base::StringPrintf("name='%s'", name_value.c_str()),
+                   &line);
 
   std::string description_value;
   node.GetString("description", &description_value);
   WriteAttribute(
       false, base::StringPrintf("description='%s'", description_value.c_str()),
       &line);
 
   const base::ListValue* states_value;
   node.GetList("states", &states_value);
   for (base::ListValue::const_iterator it = states_value->begin();
@@ skipping 21 matching lines @@
 
 const std::string AccessibilityTreeFormatterAuraLinux::GetAllowString() {
   return "@AURALINUX-ALLOW:";
 }
 
 const std::string AccessibilityTreeFormatterAuraLinux::GetDenyString() {
   return "@AURALINUX-DENY:";
 }
 
 }