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

Unified Diff: chrome/renderer/resources/extensions/automation/automation_tree.js

Issue 203753002: Implement actions for Automation API. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Remove repricated API. Created 6 years, 9 months 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
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;
}

Powered by Google App Engine
This is Rietveld 408576698