DevTools: Allow closing targets (workers and remote pages) even if the target is being inspected

BUG=375679

Review URL: https://codereview.chromium.org/298873007

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@272872 0039d316-1c4b-4281-b951-d872f2087c98

chrome/browser/devtools/device/devtools_android_bridge.cc

@@ -558,8 +558,6 @@ bool RemotePageTarget::Activate() const {
 }
 
 bool RemotePageTarget::Close() const {
-  if (IsAttached())
-    return false;
   std::string request = base::StringPrintf(kClosePageRequest,
                                            remote_id_.c_str());
   browser_->SendJsonRequest(request, base::Bind(&NoOp));

chrome/browser/resources/inspect/inspect.js

@@ -379,7 +379,7 @@ function populateRemoteTargets(devices) {
         }
         if (majorChromeVersion >= MIN_VERSION_TAB_CLOSE) {
           addActionLink(row, 'close',
-              sendTargetCommand.bind(null, 'close', page), page.attached);
+              sendTargetCommand.bind(null, 'close', page), false);
         }
       }
     }
@@ -419,7 +419,7 @@ function addToWorkersList(data) {
   var row =
       addTargetToList(data, $('workers-list'), ['name', 'description', 'url']);
   addActionLink(row, 'terminate',
-      sendTargetCommand.bind(null, 'close', data), data.attached);
+      sendTargetCommand.bind(null, 'close', data), false);
 }
 
 function addToOthersList(data) {