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

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

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 24 matching lines @@
  * The code below is not a correct IDL but a mix of IDL and C++.
  *
  * The syntax for an instrumentation method is as follows:
  *
  *    returnValue methodName([paramAttr1] param1, [paramAttr2] param2, ...)
  *
  * Where:
  *   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.
  *
  *   paramList: C++ parameter list with optional names. Names will be deduced from types if omitted but you have to
  *        specify explicit names for multiple parameters of the same type.
  *
  * Parameters with type PassRefPtr<T> are converted to raw pointers,
  * so reference will not be passed or released until all agents are notified.
  */
@@ skipping 93 matching lines @@
   void didRunJavaScriptDialog(LocalFrame* frame, bool result);
   void documentWriteFetchScript([Keep] Document*);
   void didChangeViewport(LocalFrame* document);
   void breakableLocation(ExecutionContext* context, const char* name);
   RecalculateStyle(Document* document);
   UpdateLayout(Document*);
   ExecuteScript([Keep] ExecutionContext* context);
   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&);
+}