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

Unified Diff: chrome/renderer/resources/extensions/automation_custom_bindings.js

Issue 561923002: Revert of Add logspam to help diagnose flaky tests (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 3 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_custom_bindings.js
diff --git a/chrome/renderer/resources/extensions/automation_custom_bindings.js b/chrome/renderer/resources/extensions/automation_custom_bindings.js
index aaa75a09e8cfa2e6545c60a643d8810ba0a5ea60..596032d6bbcfbd5549b4ce3e5bf46a79471dea1b 100644
--- a/chrome/renderer/resources/extensions/automation_custom_bindings.js
+++ b/chrome/renderer/resources/extensions/automation_custom_bindings.js
@@ -13,7 +13,6 @@
var forEach = require('utils').forEach;
var lastError = require('lastError');
var schema = requireNative('automationInternal').GetSchemaAdditions();
-var logging = requireNative('logging');
// TODO(aboxhall): Look into using WeakMap
var idToAutomationRootNode = {};
@@ -34,6 +33,7 @@
automation.registerCustomHook(function(bindingsAPI) {
var apiFunctions = bindingsAPI.apiFunctions;
+ // TODO(aboxhall, dtseng): Make this return the speced AutomationRootNode obj.
apiFunctions.setHandleRequest('getTree', function getTree(tabId, callback) {
// enableTab() ensures the renderer for the active or specified tab has
// accessibility enabled, and fetches its process and routing ids to use as
@@ -92,8 +92,6 @@
automationInternal.onAccessibilityEvent.addListener(function(data) {
var pid = data.processID;
var rid = data.routingID;
- logging.LOG('onAccessibilityEvent { processID: ' + pid + ', routingID: ' +
- rid + ', eventType: ' + data.eventType + ' }');
var id = createAutomationRootNodeID(pid, rid);
var targetTree = idToAutomationRootNode[id];
if (!targetTree) {
@@ -103,7 +101,6 @@
targetTree = new AutomationRootNode(pid, rid);
idToAutomationRootNode[id] = targetTree;
}
-
if (!privates(targetTree).impl.onAccessibilityEvent(data))
return;
@@ -124,7 +121,7 @@
// have been cached in idToCallback, so call and delete it now that we
// have the complete tree.
for (var i = 0; i < idToCallback[id].length; i++) {
- logging.LOG('calling getTree() callback');
+ console.log('calling getTree() callback');
var callback = idToCallback[id][i];
callback(targetTree);
}

Powered by Google App Engine
This is Rietveld 408576698