From 90e31b4f39d0d2baf08e9d8bee4e67b5f728e0fc Mon Sep 17 00:00:00 2001
From: Andreas Kling <kling@serenityos.org>
Date: Thu, 13 May 2021 23:26:23 +0200
Subject: [PATCH] Inspector: Don't check that target has pledged "accept"

This check only existed to prevent crashing the target process back
when programs were listening for incoming Inspector connections.

Now that we talk to InspectorServer instead, and it already has a
communication channel with the target, this is no longer an issue.
---
 Userland/DevTools/Inspector/main.cpp | 15 +--------------
 1 file changed, 1 insertion(+), 14 deletions(-)

diff --git a/Userland/DevTools/Inspector/main.cpp b/Userland/DevTools/Inspector/main.cpp
index c1274143d3..d0a0d643e9 100644
--- a/Userland/DevTools/Inspector/main.cpp
+++ b/Userland/DevTools/Inspector/main.cpp
@@ -9,7 +9,6 @@
 #include "RemoteObjectPropertyModel.h"
 #include "RemoteProcess.h"
 #include <AK/URL.h>
-#include <LibCore/ProcessStatisticsReader.h>
 #include <LibDesktop/Launcher.h>
 #include <LibGUI/Application.h>
 #include <LibGUI/BoxLayout.h>
@@ -23,6 +22,7 @@
 #include <LibGUI/TreeView.h>
 #include <LibGUI/Window.h>
 #include <stdio.h>
+#include <unistd.h>
 
 using namespace Inspector;
 
@@ -97,19 +97,6 @@ int main(int argc, char** argv)
         return 1;
     }
 
-    auto all_processes = Core::ProcessStatisticsReader::get_all();
-    for (auto& it : all_processes.value()) {
-        if (it.value.pid != pid)
-            continue;
-        if (it.value.pledge.is_empty())
-            break;
-        if (!it.value.pledge.contains("accept")) {
-            GUI::MessageBox::show(window, String::formatted("{} ({}) has not pledged accept!", it.value.name, pid), "Error", GUI::MessageBox::Type::Error);
-            return 1;
-        }
-        break;
-    }
-
     window->set_title("Inspector");
     window->resize(685, 500);
     window->set_icon(app_icon.bitmap_for_size(16));