Revert of [instrumentation] Turn inspector override "probes" return values into output parameters.

third_party/WebKit/Source/core/probe/CoreProbes.pidl

Index: third_party/WebKit/Source/core/probe/CoreProbes.pidl
diff --git a/third_party/WebKit/Source/core/probe/CoreProbes.pidl b/third_party/WebKit/Source/core/probe/CoreProbes.pidl
index 0f91d11e2ee29793fa9ced64ae8271ab3f28b5d1..810ff7acc4ee01f3c44fe8c2d8eb8dfbec69f85e 100644
--- a/third_party/WebKit/Source/core/probe/CoreProbes.pidl
+++ b/third_party/WebKit/Source/core/probe/CoreProbes.pidl
@@ -42,7 +42,7 @@
  *   paramAttr - optional attribute controlling the parameters handling (one attribute per parameter max).
  *       Keep - pass first parameter (used to access the CoreProbeSink instance) to agents.
  *
- *   returnValue: "void" for instant probes.
+ *   returnValue: C++ return value. Only "void" and "bool" are supported.
  *                Omitted for the scoped probes.
  *
  *   methodName: C++ name for the public instrumentation method and agents methods.
@@ -156,8 +156,11 @@
   CallFunction([Keep] ExecutionContext* context, v8::Local<v8::Function> function, int depth = 0);
   UserCallback([Keep] ExecutionContext* context, const char* name, const AtomicString& atomicName, bool recurring, EventTarget* eventTarget = nullptr);
   ParseHTML(Document* document, HTMLDocumentParser* parser);
-  void forcePseudoState([Keep] Element* element, CSSSelector::PseudoType pseudoState, bool* result);
-  void shouldWaitForDebuggerOnWorkerStart(ExecutionContext* context, bool* result);
-  void shouldForceCORSPreflight(Document*, bool* result);
-  void shouldBlockRequest(LocalFrame*, const ResourceRequest&, bool* result);
 }
+
+interface InspectorOverrides {
+  bool forcePseudoState([Keep] Element* element, CSSSelector::PseudoType pseudoState);
+  bool shouldWaitForDebuggerOnWorkerStart(ExecutionContext* context);
+  bool shouldForceCORSPreflight(Document*);
+  bool shouldBlockRequest(LocalFrame*, const ResourceRequest&);
+}