Only call TextChanged on existing AXObjects

third_party/WebKit/Source/modules/accessibility/AXObjectCacheImpl.cpp

 /*
  * Copyright (C) 2014, Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
  * 2.  Redistributions in binary form must reproduce the above copyright
@@ skipping 556 matching lines @@
     Vector<AXID> child_axi_ds = aria_owner_to_children_mapping_.at(obj_id);
     for (size_t i = 0; i < child_axi_ds.size(); ++i)
       aria_owned_child_to_owner_mapping_.erase(child_axi_ds[i]);
     aria_owner_to_children_mapping_.erase(obj_id);
   }
   aria_owned_child_to_owner_mapping_.erase(obj_id);
   aria_owned_child_to_real_parent_mapping_.erase(obj_id);
   aria_owner_to_ids_mapping_.erase(obj_id);
 }
 
-void AXObjectCacheImpl::SelectionChanged(Node* node) {
+AXObjectImpl* AXObjectCacheImpl::NearestExistingAncestor(Node* node) {
   // Find the nearest ancestor that already has an accessibility object, since
   // we might be in the middle of a layout.
   while (node) {
-    if (AXObjectImpl* obj = Get(node)) {
-      obj->SelectionChanged();
-      return;
-    }
+    if (AXObjectImpl* obj = Get(node))
+      return obj;
     node = node->parentNode();
   }
+  return nullptr;
+}
+
+void AXObjectCacheImpl::SelectionChanged(Node* node) {
+  AXObjectImpl* nearestAncestor = NearestExistingAncestor(node);
+  if (nearestAncestor)
+    nearestAncestor->SelectionChanged();
 }
 
 void AXObjectCacheImpl::TextChanged(Node* node) {
-  TextChanged(GetOrCreate(node));
+  TextChanged(Get(node));
 }
 
 void AXObjectCacheImpl::TextChanged(LayoutObject* layout_object) {
-  TextChanged(GetOrCreate(layout_object));
+  TextChanged(Get(layout_object));
 }
 
 void AXObjectCacheImpl::TextChanged(AXObjectImpl* obj) {
   if (!obj)
     return;
 
   obj->TextChanged();
   PostNotification(obj, AXObjectCacheImpl::kAXTextChanged);
 }
 
@@ skipping 565 matching lines @@
     PostNotification(end_object, kAXChildrenChanged);
   }
 }
 
 void AXObjectCacheImpl::HandleValueChanged(Node* node) {
   PostNotification(node, AXObjectCache::kAXValueChanged);
 }
 
 void AXObjectCacheImpl::HandleUpdateActiveMenuOption(LayoutMenuList* menu_list,
                                                      int option_index) {
-  AXObjectImpl* obj = Get(menu_list);
+  AXObjectImpl* obj = GetOrCreate(menu_list);
   if (!obj || !obj->IsMenuList())
     return;
 
   ToAXMenuList(obj)->DidUpdateActiveOption(option_index);
 }
 
 void AXObjectCacheImpl::DidShowMenuListPopup(LayoutMenuList* menu_list) {
   AXObjectImpl* obj = Get(menu_list);
   if (!obj || !obj->IsMenuList())
     return;
@@ skipping 88 matching lines @@
   visitor->Trace(document_);
   visitor->Trace(node_object_mapping_);
 
   visitor->Trace(objects_);
   visitor->Trace(notifications_to_post_);
 
   AXObjectCache::Trace(visitor);
 }
 
 }  // namespace blink