Remove logging that exposes url in error output

Bug: 639322
Change-Id: I9443dab4aeaeef75e722bba8d3835f00406a3c65
Reviewed-on: https://chromium-review.googlesource.com/c/1481570
Commit-Queue: David Tseng <dtseng@chromium.org>
Reviewed-by: Devlin <rdevlin.cronin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#636253}

chrome/browser/extensions/api/automation_internal/automation_internal_api.cc

@@ -58,8 +58,7 @@ namespace extensions {
 namespace {
 
 const char kCannotRequestAutomationOnPage[] =
-    "Cannot request automation tree on url \"*\". "
-    "Extension manifest must request permission to access this host.";
+    "Failed request of automation on a page";
 const char kRendererDestroyed[] = "The tab was closed.";
 const char kNoDocument[] = "No document.";
 const char kNodeDestroyed[] =
@@ -283,8 +282,7 @@ AutomationInternalEnableTabFunction::Run() {
     return RespondNow(Error("Could not enable accessibility for active tab"));
 
   if (!CanRequestAutomation(extension(), automation_info, contents)) {
-    return RespondNow(
-        Error(kCannotRequestAutomationOnPage, contents->GetURL().spec()));
+    return RespondNow(Error(kCannotRequestAutomationOnPage));
   }
 
   AutomationWebContentsObserver::CreateForWebContents(contents);
@@ -508,8 +506,7 @@ AutomationInternalPerformActionFunction::Run() {
       content::WebContents* contents =
           content::WebContents::FromRenderFrameHost(rfh);
       if (!CanRequestAutomation(extension(), automation_info, contents)) {
-        return RespondNow(
-            Error(kCannotRequestAutomationOnPage, contents->GetURL().spec()));
+        return RespondNow(Error(kCannotRequestAutomationOnPage));
       }
 
       // Handle internal actions.

chrome/test/data/extensions/api_test/active_tab/background.js

@@ -100,8 +100,7 @@ chrome.webNavigation.onCompleted.addListener(function(details) {
           'respective host.'));
 
   chrome.automation.getTree(callbackFail(
-      'Cannot request automation tree on url "' + details.url +
-      '". Extension manifest must request permission to access this host.'));
+      'Failed request of automation on a page'));
 
   assertFalse(canXhr(details.url));
 });

chrome/test/data/extensions/api_test/automation/tests/tabs_automation_boolean/permissions.js

@@ -21,8 +21,8 @@ var allTestsADomain = [
 
 var allTestsBDomain = [
   function testError() {
-    var expectedError = 'Cannot request automation tree on url "' + urlB +
-        '". Extension manifest must request permission to access this host.';
+    var expectedError = 'Failed request of automation on a page';
+
     chrome.automation.getTree(callbackFail(expectedError, function(tree) {
       assertEq(undefined, tree);
       chrome.test.succeed();
@@ -46,4 +46,3 @@ chrome.test.getConfig(function(config) {
     });
   });
 });
-

chrome/test/data/extensions/api_test/automation/tests/tabs_automation_hosts/permissions.js

@@ -21,8 +21,7 @@ var allTestsADomain = [
 
 var allTestsBDomain = [
   function testError() {
-    var expectedError = 'Cannot request automation tree on url "' + urlB +
-        '". Extension manifest must request permission to access this host.';
+    var expectedError = 'Failed request of automation on a page';
     chrome.automation.getTree(callbackFail(expectedError, function(tree) {
       assertEq(undefined, tree);
       chrome.test.succeed();
@@ -46,4 +45,3 @@ chrome.test.getConfig(function(config) {
     });
   });
 });
-