Chromium Code Reviews| Index: chrome/renderer/resources/extensions/automation/automation_tree.js |
| diff --git a/chrome/renderer/resources/extensions/automation/automation_tree.js b/chrome/renderer/resources/extensions/automation/automation_tree.js |
| index d644ae580f7d4e103451b6ecb1175cbd89a5742d..10bec69681b20ca983587b69d7c54789ffde9cf0 100644 |
| --- a/chrome/renderer/resources/extensions/automation/automation_tree.js |
| +++ b/chrome/renderer/resources/extensions/automation/automation_tree.js |
| @@ -37,18 +37,12 @@ var AutomationAttributeTypes = [ |
| * AutomationNode object. |
| * Thus, tree traversals amount to a lookup in our hash. |
| * |
| + * The tree itself is identified by the process id and routing id of the |
| + * renderer/renderer host pair. |
|
dmazzoni
2014/04/01 17:05:25
What do you mean by renderer/renderer?
David Tseng
2014/04/01 21:27:15
revised to RWH.
|
| * @constructor |
| */ |
| -var AutomationTree = function(routingId) { |
| - privates(this).impl = new AutomationTreeImpl(routingId); |
| - |
| - /** |
| - * Event fired when a tree update occurs. |
| - * @deprecated TODO(aboxhall/dtseng): remove this event; it should not be |
| - * exposed in the public API. Replace with EventListener style API which |
| - * allows listening for events in the AXEvent enum. |
| - */ |
| - this.onUpdate = new Event(); |
| +var AutomationTree = function(processId, routingId) { |
| + privates(this).impl = new AutomationTreeImpl(processId, routingId); |
| }; |
| @@ -63,7 +57,8 @@ AutomationTree.prototype = { |
| }; |
| -var AutomationTreeImpl = function(routingId) { |
| +var AutomationTreeImpl = function(processId, routingId) { |
| + this.processId = processId; |
| this.routingId = routingId; |
| /** |
| @@ -179,6 +174,7 @@ AutomationTreeImpl.prototype = { |
| } |
| privates(node).impl.child_ids = new_child_ids; |
| this.axNodeDataCache_[node.id] = node; |
| + privates(node).impl.updateEventListeners(data.event_type); |
| } |
| return true; |
| } |