Add a treeChange type to Automation

chrome/renderer/extensions/automation_internal_custom_bindings.cc

 // Copyright 2014 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 "chrome/renderer/extensions/automation_internal_custom_bindings.h"
 
 #include <stddef.h>
 #include <stdint.h>
 
 #include "base/bind.h"
@@ skipping 971 matching lines @@
   v8::Local<v8::Object> event_params(v8::Object::New(GetIsolate()));
   event_params->Set(CreateV8String(isolate, "treeID"),
                     v8::Integer::New(GetIsolate(), params.tree_id));
   event_params->Set(CreateV8String(isolate, "targetID"),
                     v8::Integer::New(GetIsolate(), params.id));
   event_params->Set(CreateV8String(isolate, "eventType"),
                     CreateV8String(isolate, ToString(params.event_type)));
   args->Set(0U, event_params);
   context()->DispatchEvent("automationInternal.onAccessibilityEvent", args);
 }
+void AutomationInternalCustomBindings::OnNodeDataWillChange(
+    const ui::AXNodeData& old_node_data,
+    const ui::AXNodeData& new_node_data) {
+  if (old_node_data.GetStringAttribute(ui::AX_ATTR_NAME) !=
+      new_node_data.GetStringAttribute(ui::AX_ATTR_NAME))
+    text_changed_node_ids_.push_back(new_node_data.id);
+}
 
 void AutomationInternalCustomBindings::OnTreeDataChanged(ui::AXTree* tree) {}
 
 void AutomationInternalCustomBindings::OnNodeWillBeDeleted(ui::AXTree* tree,
                                                            ui::AXNode* node) {
   SendTreeChangeEvent(
       api::automation::TREE_CHANGE_TYPE_NODEREMOVED,
       tree, node);
   deleted_node_ids_.push_back(node->id());
 }
@@ skipping 41 matching lines @@
             api::automation::TREE_CHANGE_TYPE_SUBTREECREATED,
             tree, node);
         break;
       case NODE_CHANGED:
         SendTreeChangeEvent(
             api::automation::TREE_CHANGE_TYPE_NODECHANGED,
             tree, node);
         break;
     }
   }
+
+  for (size_t i = 0; i < text_changed_node_ids_.size(); ++i) {
+    SendTreeChangeEvent(api::automation::TREE_CHANGE_TYPE_TEXTCHANGED, tree,
+                        tree->GetFromId(text_changed_node_ids_[i]));
+    text_changed_node_ids_.clear();

[dmazzoni, 2016/02/19 23:20:23]

This clear should go after the for loop, not inside it.

+  }
 }
 
 void AutomationInternalCustomBindings::SendTreeChangeEvent(
     api::automation::TreeChangeType change_type,
     ui::AXTree* tree,
     ui::AXNode* node) {
   // Don't send tree change events when it's not the active profile.
   if (!is_active_profile_)
     return;
 
@@ skipping 83 matching lines @@
   v8::Local<v8::Array> args(v8::Array::New(GetIsolate(), 2U));
   args->Set(0U, v8::Integer::New(GetIsolate(), tree_id));
   v8::Local<v8::Array> nodes(v8::Array::New(GetIsolate(), ids.size()));
   args->Set(1U, nodes);
   for (size_t i = 0; i < ids.size(); ++i)
     nodes->Set(i, v8::Integer::New(GetIsolate(), ids[i]));
   context()->DispatchEvent("automationInternal.onNodesRemoved", args);
 }
 
 }  // namespace extensions