<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN"
"http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head><meta http-equiv="content-type" content="text/html; charset=utf-8" />
<title>[269524] trunk</title>
</head>
<body>

<style type="text/css"><!--
#msg dl.meta { border: 1px #006 solid; background: #369; padding: 6px; color: #fff; }
#msg dl.meta dt { float: left; width: 6em; font-weight: bold; }
#msg dt:after { content:':';}
#msg dl, #msg dt, #msg ul, #msg li, #header, #footer, #logmsg { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt;  }
#msg dl a { font-weight: bold}
#msg dl a:link    { color:#fc3; }
#msg dl a:active  { color:#ff0; }
#msg dl a:visited { color:#cc6; }
h3 { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; font-weight: bold; }
#msg pre { overflow: auto; background: #ffc; border: 1px #fa0 solid; padding: 6px; }
#logmsg { background: #ffc; border: 1px #fa0 solid; padding: 1em 1em 0 1em; }
#logmsg p, #logmsg pre, #logmsg blockquote { margin: 0 0 1em 0; }
#logmsg p, #logmsg li, #logmsg dt, #logmsg dd { line-height: 14pt; }
#logmsg h1, #logmsg h2, #logmsg h3, #logmsg h4, #logmsg h5, #logmsg h6 { margin: .5em 0; }
#logmsg h1:first-child, #logmsg h2:first-child, #logmsg h3:first-child, #logmsg h4:first-child, #logmsg h5:first-child, #logmsg h6:first-child { margin-top: 0; }
#logmsg ul, #logmsg ol { padding: 0; list-style-position: inside; margin: 0 0 0 1em; }
#logmsg ul { text-indent: -1em; padding-left: 1em; }#logmsg ol { text-indent: -1.5em; padding-left: 1.5em; }
#logmsg > ul, #logmsg > ol { margin: 0 0 1em 0; }
#logmsg pre { background: #eee; padding: 1em; }
#logmsg blockquote { border: 1px solid #fa0; border-left-width: 10px; padding: 1em 1em 0 1em; background: white;}
#logmsg dl { margin: 0; }
#logmsg dt { font-weight: bold; }
#logmsg dd { margin: 0; padding: 0 0 0.5em 0; }
#logmsg dd:before { content:'\00bb';}
#logmsg table { border-spacing: 0px; border-collapse: collapse; border-top: 4px solid #fa0; border-bottom: 1px solid #fa0; background: #fff; }
#logmsg table th { text-align: left; font-weight: normal; padding: 0.2em 0.5em; border-top: 1px dotted #fa0; }
#logmsg table td { text-align: right; border-top: 1px dotted #fa0; padding: 0.2em 0.5em; }
#logmsg table thead th { text-align: center; border-bottom: 1px solid #fa0; }
#logmsg table th.Corner { text-align: left; }
#logmsg hr { border: none 0; border-top: 2px dashed #fa0; height: 1px; }
#header, #footer { color: #fff; background: #636; border: 1px #300 solid; padding: 6px; }
#patch { width: 100%; }
#patch h4 {font-family: verdana,arial,helvetica,sans-serif;font-size:10pt;padding:8px;background:#369;color:#fff;margin:0;}
#patch .propset h4, #patch .binary h4 {margin:0;}
#patch pre {padding:0;line-height:1.2em;margin:0;}
#patch .diff {width:100%;background:#eee;padding: 0 0 10px 0;overflow:auto;}
#patch .propset .diff, #patch .binary .diff  {padding:10px 0;}
#patch span {display:block;padding:0 10px;}
#patch .modfile, #patch .addfile, #patch .delfile, #patch .propset, #patch .binary, #patch .copfile {border:1px solid #ccc;margin:10px 0;}
#patch ins {background:#dfd;text-decoration:none;display:block;padding:0 10px;}
#patch del {background:#fdd;text-decoration:none;display:block;padding:0 10px;}
#patch .lines, .info {color:#888;background:#fff;}
--></style>
<div id="msg">
<dl class="meta">
<dt>Revision</dt> <dd><a href="http://trac.webkit.org/projects/webkit/changeset/269524">269524</a></dd>
<dt>Author</dt> <dd>tsavell@apple.com</dd>
<dt>Date</dt> <dd>2020-11-06 10:44:09 -0800 (Fri, 06 Nov 2020)</dd>
</dl>

<h3>Log Message</h3>
<pre>Unreviewed, reverting <a href="http://trac.webkit.org/projects/webkit/changeset/269486">r269486</a>.

Caused 50+ timeouts on Mac Debug WK2

Reverted changeset:

"[Cocoa] Inspector Extensions: Add _WKInspectorExtension and
related plumbing"
https://bugs.webkit.org/show_bug.cgi?id=217783
https://trac.webkit.org/changeset/269486</pre>

<h3>Modified Paths</h3>
<ul>
<li><a href="#trunkSourceWebCoreChangeLog">trunk/Source/WebCore/ChangeLog</a></li>
<li><a href="#trunkSourceWebCoreinspectorInspectorFrontendClienth">trunk/Source/WebCore/inspector/InspectorFrontendClient.h</a></li>
<li><a href="#trunkSourceWebCoreinspectorInspectorFrontendClientLocalh">trunk/Source/WebCore/inspector/InspectorFrontendClientLocal.h</a></li>
<li><a href="#trunkSourceWebInspectorUIChangeLog">trunk/Source/WebInspectorUI/ChangeLog</a></li>
<li><a href="#trunkSourceWebInspectorUIUserInterfaceControllersAppControllerBasejs">trunk/Source/WebInspectorUI/UserInterface/Controllers/AppControllerBase.js</a></li>
<li><a href="#trunkSourceWebInspectorUIUserInterfaceMainhtml">trunk/Source/WebInspectorUI/UserInterface/Main.html</a></li>
<li><a href="#trunkSourceWebInspectorUIUserInterfaceProtocolInspectorFrontendAPIjs">trunk/Source/WebInspectorUI/UserInterface/Protocol/InspectorFrontendAPI.js</a></li>
<li><a href="#trunkSourceWebInspectorUIUserInterfaceTesthtml">trunk/Source/WebInspectorUI/UserInterface/Test.html</a></li>
<li><a href="#trunkSourceWebKitChangeLog">trunk/Source/WebKit/ChangeLog</a></li>
<li><a href="#trunkSourceWebKitDerivedSourcesinputxcfilelist">trunk/Source/WebKit/DerivedSources-input.xcfilelist</a></li>
<li><a href="#trunkSourceWebKitDerivedSourcesoutputxcfilelist">trunk/Source/WebKit/DerivedSources-output.xcfilelist</a></li>
<li><a href="#trunkSourceWebKitDerivedSourcesmake">trunk/Source/WebKit/DerivedSources.make</a></li>
<li><a href="#trunkSourceWebKitScriptswebkitmessagespy">trunk/Source/WebKit/Scripts/webkit/messages.py</a></li>
<li><a href="#trunkSourceWebKitSharedAPIAPIObjecth">trunk/Source/WebKit/Shared/API/APIObject.h</a></li>
<li><a href="#trunkSourceWebKitSharedCocoaAPIObjectmm">trunk/Source/WebKit/Shared/Cocoa/APIObject.mm</a></li>
<li><a href="#trunkSourceWebKitSourcestxt">trunk/Source/WebKit/Sources.txt</a></li>
<li><a href="#trunkSourceWebKitSourcesCocoatxt">trunk/Source/WebKit/SourcesCocoa.txt</a></li>
<li><a href="#trunkSourceWebKitUIProcessAPICocoa_WKInspectorh">trunk/Source/WebKit/UIProcess/API/Cocoa/_WKInspector.h</a></li>
<li><a href="#trunkSourceWebKitUIProcessAPICocoa_WKInspectormm">trunk/Source/WebKit/UIProcess/API/Cocoa/_WKInspector.mm</a></li>
<li><a href="#trunkSourceWebKitUIProcessAPICocoa_WKRemoteWebInspectorViewControllerh">trunk/Source/WebKit/UIProcess/API/Cocoa/_WKRemoteWebInspectorViewController.h</a></li>
<li><a href="#trunkSourceWebKitUIProcessAPICocoa_WKRemoteWebInspectorViewControllermm">trunk/Source/WebKit/UIProcess/API/Cocoa/_WKRemoteWebInspectorViewController.mm</a></li>
<li><a href="#trunkSourceWebKitUIProcessInspectorRemoteWebInspectorProxycpp">trunk/Source/WebKit/UIProcess/Inspector/RemoteWebInspectorProxy.cpp</a></li>
<li><a href="#trunkSourceWebKitUIProcessInspectorRemoteWebInspectorProxyh">trunk/Source/WebKit/UIProcess/Inspector/RemoteWebInspectorProxy.h</a></li>
<li><a href="#trunkSourceWebKitUIProcessInspectorRemoteWebInspectorProxymessagesin">trunk/Source/WebKit/UIProcess/Inspector/RemoteWebInspectorProxy.messages.in</a></li>
<li><a href="#trunkSourceWebKitUIProcessInspectorWebInspectorProxycpp">trunk/Source/WebKit/UIProcess/Inspector/WebInspectorProxy.cpp</a></li>
<li><a href="#trunkSourceWebKitUIProcessInspectorWebInspectorProxyh">trunk/Source/WebKit/UIProcess/Inspector/WebInspectorProxy.h</a></li>
<li><a href="#trunkSourceWebKitUIProcessWebAuthenticationCocoaLocalAuthenticatormm">trunk/Source/WebKit/UIProcess/WebAuthentication/Cocoa/LocalAuthenticator.mm</a></li>
<li><a href="#trunkSourceWebKitWebKitxcodeprojprojectpbxproj">trunk/Source/WebKit/WebKit.xcodeproj/project.pbxproj</a></li>
<li><a href="#trunkSourceWebKitWebProcessInspectorRemoteWebInspectorUIcpp">trunk/Source/WebKit/WebProcess/Inspector/RemoteWebInspectorUI.cpp</a></li>
<li><a href="#trunkSourceWebKitWebProcessInspectorRemoteWebInspectorUIh">trunk/Source/WebKit/WebProcess/Inspector/RemoteWebInspectorUI.h</a></li>
<li><a href="#trunkSourceWebKitWebProcessInspectorWebInspectorUIcpp">trunk/Source/WebKit/WebProcess/Inspector/WebInspectorUI.cpp</a></li>
<li><a href="#trunkSourceWebKitWebProcessInspectorWebInspectorUIh">trunk/Source/WebKit/WebProcess/Inspector/WebInspectorUI.h</a></li>
<li><a href="#trunkToolsChangeLog">trunk/Tools/ChangeLog</a></li>
<li><a href="#trunkToolsTestWebKitAPITestWebKitAPIxcodeprojprojectpbxproj">trunk/Tools/TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj</a></li>
</ul>

<h3>Removed Paths</h3>
<ul>
<li><a href="#trunkSourceWebInspectorUIUserInterfaceControllersWebInspectorExtensionControllerjs">trunk/Source/WebInspectorUI/UserInterface/Controllers/WebInspectorExtensionController.js</a></li>
<li><a href="#trunkSourceWebInspectorUIUserInterfaceModelsWebInspectorExtensionjs">trunk/Source/WebInspectorUI/UserInterface/Models/WebInspectorExtension.js</a></li>
<li><a href="#trunkSourceWebKitSharedInspectorExtensionTypescpp">trunk/Source/WebKit/Shared/InspectorExtensionTypes.cpp</a></li>
<li><a href="#trunkSourceWebKitSharedInspectorExtensionTypesh">trunk/Source/WebKit/Shared/InspectorExtensionTypes.h</a></li>
<li><a href="#trunkSourceWebKitUIProcessAPIAPIInspectorExtensionh">trunk/Source/WebKit/UIProcess/API/APIInspectorExtension.h</a></li>
<li><a href="#trunkSourceWebKitUIProcessAPICocoa_WKInspectorExtensionh">trunk/Source/WebKit/UIProcess/API/Cocoa/_WKInspectorExtension.h</a></li>
<li><a href="#trunkSourceWebKitUIProcessAPICocoa_WKInspectorExtensionmm">trunk/Source/WebKit/UIProcess/API/Cocoa/_WKInspectorExtension.mm</a></li>
<li><a href="#trunkSourceWebKitUIProcessAPICocoa_WKInspectorExtensionHosth">trunk/Source/WebKit/UIProcess/API/Cocoa/_WKInspectorExtensionHost.h</a></li>
<li><a href="#trunkSourceWebKitUIProcessAPICocoa_WKInspectorExtensionInternalh">trunk/Source/WebKit/UIProcess/API/Cocoa/_WKInspectorExtensionInternal.h</a></li>
<li><a href="#trunkSourceWebKitUIProcessInspectorWebInspectorUIExtensionControllerProxycpp">trunk/Source/WebKit/UIProcess/Inspector/WebInspectorUIExtensionControllerProxy.cpp</a></li>
<li><a href="#trunkSourceWebKitUIProcessInspectorWebInspectorUIExtensionControllerProxyh">trunk/Source/WebKit/UIProcess/Inspector/WebInspectorUIExtensionControllerProxy.h</a></li>
<li><a href="#trunkSourceWebKitWebProcessInspectorWebInspectorUIExtensionControllercpp">trunk/Source/WebKit/WebProcess/Inspector/WebInspectorUIExtensionController.cpp</a></li>
<li><a href="#trunkSourceWebKitWebProcessInspectorWebInspectorUIExtensionControllerh">trunk/Source/WebKit/WebProcess/Inspector/WebInspectorUIExtensionController.h</a></li>
<li><a href="#trunkSourceWebKitWebProcessInspectorWebInspectorUIExtensionControllermessagesin">trunk/Source/WebKit/WebProcess/Inspector/WebInspectorUIExtensionController.messages.in</a></li>
<li><a href="#trunkToolsTestWebKitAPITestsWebKitCocoaWKInspectorExtensionHostmm">trunk/Tools/TestWebKitAPI/Tests/WebKitCocoa/WKInspectorExtensionHost.mm</a></li>
</ul>

</div>
<div id="patch">
<h3>Diff</h3>
<a id="trunkSourceWebCoreChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/ChangeLog (269523 => 269524)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/ChangeLog   2020-11-06 18:42:14 UTC (rev 269523)
+++ trunk/Source/WebCore/ChangeLog      2020-11-06 18:44:09 UTC (rev 269524)
</span><span class="lines">@@ -1,3 +1,16 @@
</span><ins>+2020-11-06  Truitt Savell  <tsavell@apple.com>
+
+        Unreviewed, reverting r269486.
+
+        Caused 50+ timeouts on Mac Debug WK2
+
+        Reverted changeset:
+
+        "[Cocoa] Inspector Extensions: Add _WKInspectorExtension and
+        related plumbing"
+        https://bugs.webkit.org/show_bug.cgi?id=217783
+        https://trac.webkit.org/changeset/269486
+
</ins><span class="cx"> 2020-11-06  Jer Noble  <jer.noble@apple.com>
</span><span class="cx"> 
</span><span class="cx">         Do not allow pages to enter fullscreen while an exit fullscreen operation is ongoing.
</span></span></pre></div>
<a id="trunkSourceWebCoreinspectorInspectorFrontendClienth"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/inspector/InspectorFrontendClient.h (269523 => 269524)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/inspector/InspectorFrontendClient.h 2020-11-06 18:42:14 UTC (rev 269523)
+++ trunk/Source/WebCore/inspector/InspectorFrontendClient.h    2020-11-06 18:44:09 UTC (rev 269524)
</span><span class="lines">@@ -36,7 +36,6 @@
</span><span class="cx"> #include "InspectorDebuggableType.h"
</span><span class="cx"> #include "UserInterfaceLayoutDirection.h"
</span><span class="cx"> #include <wtf/Forward.h>
</span><del>-#include <wtf/WeakPtr.h>
</del><span class="cx"> #include <wtf/text/WTFString.h>
</span><span class="cx"> 
</span><span class="cx"> namespace WebCore {
</span><span class="lines">@@ -43,9 +42,8 @@
</span><span class="cx"> 
</span><span class="cx"> class FloatRect;
</span><span class="cx"> class InspectorFrontendAPIDispatcher;
</span><del>-class Page;
</del><span class="cx"> 
</span><del>-class InspectorFrontendClient : public CanMakeWeakPtr<InspectorFrontendClient> {
</del><ins>+class InspectorFrontendClient {
</ins><span class="cx"> public:
</span><span class="cx">     enum class DockSide {
</span><span class="cx">         Undocked = 0,
</span><span class="lines">@@ -113,7 +111,6 @@
</span><span class="cx"> 
</span><span class="cx">     WEBCORE_EXPORT virtual void sendMessageToBackend(const String&) = 0;
</span><span class="cx">     WEBCORE_EXPORT virtual InspectorFrontendAPIDispatcher& frontendAPIDispatcher() = 0;
</span><del>-    WEBCORE_EXPORT virtual Page* frontendPage() = 0;
</del><span class="cx"> 
</span><span class="cx">     WEBCORE_EXPORT virtual bool isUnderTest() = 0;
</span><span class="cx"> };
</span></span></pre></div>
<a id="trunkSourceWebCoreinspectorInspectorFrontendClientLocalh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/inspector/InspectorFrontendClientLocal.h (269523 => 269524)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/inspector/InspectorFrontendClientLocal.h    2020-11-06 18:42:14 UTC (rev 269523)
+++ trunk/Source/WebCore/inspector/InspectorFrontendClientLocal.h       2020-11-06 18:44:09 UTC (rev 269524)
</span><span class="lines">@@ -95,8 +95,7 @@
</span><span class="cx">     String backendCommandsURL() const final { return String(); };
</span><span class="cx"> 
</span><span class="cx">     InspectorFrontendAPIDispatcher& frontendAPIDispatcher() final { return m_frontendAPIDispatcher; }
</span><del>-    Page* frontendPage() final { return m_frontendPage; }
-    
</del><ins>+
</ins><span class="cx">     WEBCORE_EXPORT bool canAttachWindow();
</span><span class="cx">     WEBCORE_EXPORT void setDockingUnavailable(bool);
</span><span class="cx"> 
</span></span></pre></div>
<a id="trunkSourceWebInspectorUIChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebInspectorUI/ChangeLog (269523 => 269524)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebInspectorUI/ChangeLog    2020-11-06 18:42:14 UTC (rev 269523)
+++ trunk/Source/WebInspectorUI/ChangeLog       2020-11-06 18:44:09 UTC (rev 269524)
</span><span class="lines">@@ -1,3 +1,16 @@
</span><ins>+2020-11-06  Truitt Savell  <tsavell@apple.com>
+
+        Unreviewed, reverting r269486.
+
+        Caused 50+ timeouts on Mac Debug WK2
+
+        Reverted changeset:
+
+        "[Cocoa] Inspector Extensions: Add _WKInspectorExtension and
+        related plumbing"
+        https://bugs.webkit.org/show_bug.cgi?id=217783
+        https://trac.webkit.org/changeset/269486
+
</ins><span class="cx"> 2020-11-05  Brian Burg  <bburg@apple.com>
</span><span class="cx"> 
</span><span class="cx">         [Cocoa] Inspector Extensions: Add _WKInspectorExtension and related plumbing
</span></span></pre></div>
<a id="trunkSourceWebInspectorUIUserInterfaceControllersAppControllerBasejs"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebInspectorUI/UserInterface/Controllers/AppControllerBase.js (269523 => 269524)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebInspectorUI/UserInterface/Controllers/AppControllerBase.js       2020-11-06 18:42:14 UTC (rev 269523)
+++ trunk/Source/WebInspectorUI/UserInterface/Controllers/AppControllerBase.js  2020-11-06 18:44:09 UTC (rev 269524)
</span><span class="lines">@@ -41,14 +41,11 @@
</span><span class="cx">     constructor()
</span><span class="cx">     {
</span><span class="cx">         this._initialized = false;
</span><del>-
-        this._extensionController = new WI.WebInspectorExtensionController;
</del><span class="cx">     }
</span><span class="cx"> 
</span><span class="cx">     // Public
</span><span class="cx"> 
</span><span class="cx">     get debuggableType() { throw WI.NotImplementedError.subclassMustOverride(); }
</span><del>-    get extensionController() { return this._extensionController; }
</del><span class="cx"> 
</span><span class="cx">     // Since various members of the app controller depend on the global singleton to exist,
</span><span class="cx">     // some initialization needs to happen after the app controller has been constructed.
</span></span></pre></div>
<a id="trunkSourceWebInspectorUIUserInterfaceControllersWebInspectorExtensionControllerjs"></a>
<div class="delfile"><h4>Deleted: trunk/Source/WebInspectorUI/UserInterface/Controllers/WebInspectorExtensionController.js (269523 => 269524)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebInspectorUI/UserInterface/Controllers/WebInspectorExtensionController.js 2020-11-06 18:42:14 UTC (rev 269523)
+++ trunk/Source/WebInspectorUI/UserInterface/Controllers/WebInspectorExtensionController.js    2020-11-06 18:44:09 UTC (rev 269524)
</span><span class="lines">@@ -1,65 +0,0 @@
</span><del>-/*
- * Copyright (C) 2020 Apple 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:
- * 1. Redistributions of source code must retain the above copyright
- *    notice, this list of conditions and the following disclaimer.
- * 2. Redistributions in binary form must reproduce the above copyright
- *    notice, this list of conditions and the following disclaimer in the
- *    documentation and/or other materials provided with the distribution.
- *
- * THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS''
- * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO,
- * THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
- * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR ITS CONTRIBUTORS
- * BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
- * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
- * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
- * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
- * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
- * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF
- * THE POSSIBILITY OF SUCH DAMAGE.
- */
-
-WI.WebInspectorExtensionController = class WebInspectorExtensionController extends WI.Object
-{
-    constructor()
-    {
-        super();
-
-        this._extensionIDMap = new Map;
-    }
-
-    // Public
-
-    registerExtension(extensionID, displayName)
-    {
-        if (this._extensionIDMap.has(extensionID)) {
-            WI.reportInternalError("Unable to register extension, it's already registered: ", extensionID, displayName);
-            return WI.WebInspectorExtension.ErrorCode.RegistrationFailed;
-        }
-
-        let extension = new WI.WebInspectorExtension(extensionID, displayName);
-        this._extensionIDMap.set(extensionID, extension);
-
-        this.dispatchEventToListeners(WI.WebInspectorExtensionController.Event.ExtensionAdded, {extension});
-    }
-
-    unregisterExtension(extensionID)
-    {
-        let extension = this._extensionIDMap.take(extensionID);
-        if (!extension) {
-            WI.reportInternalError("Unable to unregister extension with unknown ID: ", extensionID);
-            return WI.WebInspectorExtension.ErrorCode.InvalidRequest;
-        }
-
-        this.dispatchEventToListeners(WI.WebInspectorExtensionController.Event.ExtensionRemoved, {extension});
-    }
-};
-
-WI.WebInspectorExtensionController.Event = {
-    ExtensionAdded: "extension-added",
-    ExtensionRemoved: "extension-removed",
-};
</del></span></pre></div>
<a id="trunkSourceWebInspectorUIUserInterfaceMainhtml"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebInspectorUI/UserInterface/Main.html (269523 => 269524)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebInspectorUI/UserInterface/Main.html      2020-11-06 18:42:14 UTC (rev 269523)
+++ trunk/Source/WebInspectorUI/UserInterface/Main.html 2020-11-06 18:44:09 UTC (rev 269524)
</span><span class="lines">@@ -484,7 +484,6 @@
</span><span class="cx">     <script src="Models/TypeDescription.js"></script>
</span><span class="cx">     <script src="Models/TypeSet.js"></script>
</span><span class="cx">     <script src="Models/URLBreakpoint.js"></script>
</span><del>-    <script src="Models/WebInspectorExtension.js"></script>
</del><span class="cx">     <script src="Models/WebSocketResource.js"></script>
</span><span class="cx">     <script src="Models/WrappedPromise.js"></script>
</span><span class="cx"> 
</span><span class="lines">@@ -899,7 +898,6 @@
</span><span class="cx">     <script src="Controllers/TargetManager.js"></script>
</span><span class="cx">     <script src="Controllers/TimelineManager.js"></script>
</span><span class="cx">     <script src="Controllers/TypeTokenAnnotator.js"></script>
</span><del>-    <script src="Controllers/WebInspectorExtensionController.js"></script>
</del><span class="cx">     <script src="Controllers/WorkerManager.js"></script>
</span><span class="cx"> 
</span><span class="cx">     <script src="Controllers/DiagnosticController.js"></script>
</span></span></pre></div>
<a id="trunkSourceWebInspectorUIUserInterfaceModelsWebInspectorExtensionjs"></a>
<div class="delfile"><h4>Deleted: trunk/Source/WebInspectorUI/UserInterface/Models/WebInspectorExtension.js (269523 => 269524)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebInspectorUI/UserInterface/Models/WebInspectorExtension.js        2020-11-06 18:42:14 UTC (rev 269523)
+++ trunk/Source/WebInspectorUI/UserInterface/Models/WebInspectorExtension.js   2020-11-06 18:44:09 UTC (rev 269524)
</span><span class="lines">@@ -1,49 +0,0 @@
</span><del>-/*
- * Copyright (C) 2020 Apple 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:
- * 1. Redistributions of source code must retain the above copyright
- *    notice, this list of conditions and the following disclaimer.
- * 2. Redistributions in binary form must reproduce the above copyright
- *    notice, this list of conditions and the following disclaimer in the
- *    documentation and/or other materials provided with the distribution.
- *
- * THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS''
- * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO,
- * THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
- * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR ITS CONTRIBUTORS
- * BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
- * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
- * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
- * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
- * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
- * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF
- * THE POSSIBILITY OF SUCH DAMAGE.
- */
-
-WI.WebInspectorExtension = class WebInspectorExtension
-{
-    constructor(extensionID, displayName)
-    {
-        console.assert(typeof extensionID === "string", extensionID);
-        console.assert(typeof displayName === "string", displayName);
-
-        this._extensionID = extensionID;
-        this._displayName = displayName;
-    }
-
-    // Public
-
-    get extensionID() { return this._extensionID; }
-    get displayName() { return this._displayName; }
-};
-
-// Note: these values are synonymous with the values of enum class WebKit::InspectorExtensionError.
-WI.WebInspectorExtension.ErrorCode = {
-    ContextDestroyed: "ContextDestroyed",
-    InternalError: "InternalError",
-    InvalidRequest: "InvalidRequest",
-    RegistrationFailed: "RegistrationFailed",
-};
</del></span></pre></div>
<a id="trunkSourceWebInspectorUIUserInterfaceProtocolInspectorFrontendAPIjs"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebInspectorUI/UserInterface/Protocol/InspectorFrontendAPI.js (269523 => 269524)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebInspectorUI/UserInterface/Protocol/InspectorFrontendAPI.js       2020-11-06 18:42:14 UTC (rev 269523)
+++ trunk/Source/WebInspectorUI/UserInterface/Protocol/InspectorFrontendAPI.js  2020-11-06 18:44:09 UTC (rev 269524)
</span><span class="lines">@@ -196,17 +196,5 @@
</span><span class="cx">             InspectorFrontendAPI.dispatch(InspectorFrontendAPI._pendingCommands[i]);
</span><span class="cx"> 
</span><span class="cx">         delete InspectorFrontendAPI._pendingCommands;
</span><del>-    },
-
-    // Returns a WI.WebInspectorExtension.ErrorCode if an error occurred, otherwise nothing.
-    registerExtension(extensionID, displayName)
-    {
-        return WI.sharedApp.extensionController.registerExtension(extensionID, displayName);
-    },
-
-    // Returns a WI.WebInspectorExtension.ErrorCode if an error occurred, otherwise nothing.
-    unregisterExtension(extensionID)
-    {
-        return WI.sharedApp.extensionController.unregisterExtension(extensionID);
-    },
</del><ins>+    }
</ins><span class="cx"> };
</span></span></pre></div>
<a id="trunkSourceWebInspectorUIUserInterfaceTesthtml"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebInspectorUI/UserInterface/Test.html (269523 => 269524)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebInspectorUI/UserInterface/Test.html      2020-11-06 18:42:14 UTC (rev 269523)
+++ trunk/Source/WebInspectorUI/UserInterface/Test.html 2020-11-06 18:44:09 UTC (rev 269524)
</span><span class="lines">@@ -220,7 +220,6 @@
</span><span class="cx">     <script src="Models/TimelineMarker.js"></script>
</span><span class="cx">     <script src="Models/TimelineRecording.js"></script>
</span><span class="cx">     <script src="Models/URLBreakpoint.js"></script>
</span><del>-    <script src="Models/WebInspectorExtension.js"></script>
</del><span class="cx">     <script src="Models/WebSocketResource.js"></script>
</span><span class="cx">     <script src="Models/WrappedPromise.js"></script>
</span><span class="cx"> 
</span><span class="lines">@@ -264,7 +263,6 @@
</span><span class="cx">     <script src="Controllers/RuntimeManager.js"></script>
</span><span class="cx">     <script src="Controllers/TargetManager.js"></script>
</span><span class="cx">     <script src="Controllers/TimelineManager.js"></script>
</span><del>-    <script src="Controllers/WebInspectorExtensionController.js"></script>
</del><span class="cx">     <script src="Controllers/WorkerManager.js"></script>
</span><span class="cx"> 
</span><span class="cx">     <script src="Controllers/ResourceQueryController.js"></script>
</span></span></pre></div>
<a id="trunkSourceWebKitChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/ChangeLog (269523 => 269524)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/ChangeLog    2020-11-06 18:42:14 UTC (rev 269523)
+++ trunk/Source/WebKit/ChangeLog       2020-11-06 18:44:09 UTC (rev 269524)
</span><span class="lines">@@ -1,3 +1,16 @@
</span><ins>+2020-11-06  Truitt Savell  <tsavell@apple.com>
+
+        Unreviewed, reverting r269486.
+
+        Caused 50+ timeouts on Mac Debug WK2
+
+        Reverted changeset:
+
+        "[Cocoa] Inspector Extensions: Add _WKInspectorExtension and
+        related plumbing"
+        https://bugs.webkit.org/show_bug.cgi?id=217783
+        https://trac.webkit.org/changeset/269486
+
</ins><span class="cx"> 2020-11-06  Wenson Hsieh  <wenson_hsieh@apple.com>
</span><span class="cx"> 
</span><span class="cx">         Add some missing header includes in WebAuthenticatorCoordinator.cpp
</span></span></pre></div>
<a id="trunkSourceWebKitDerivedSourcesinputxcfilelist"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/DerivedSources-input.xcfilelist (269523 => 269524)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/DerivedSources-input.xcfilelist      2020-11-06 18:42:14 UTC (rev 269523)
+++ trunk/Source/WebKit/DerivedSources-input.xcfilelist 2020-11-06 18:44:09 UTC (rev 269524)
</span><span class="lines">@@ -95,7 +95,6 @@
</span><span class="cx"> $(PROJECT_DIR)/UIProcess/DrawingAreaProxy.messages.in
</span><span class="cx"> $(PROJECT_DIR)/UIProcess/GPU/GPUProcessProxy.messages.in
</span><span class="cx"> $(PROJECT_DIR)/UIProcess/Inspector/RemoteWebInspectorProxy.messages.in
</span><del>-$(PROJECT_DIR)/UIProcess/Inspector/WebInspectorUIExtensionControllerProxy.messages.in
</del><span class="cx"> $(PROJECT_DIR)/UIProcess/Inspector/WebInspectorProxy.messages.in
</span><span class="cx"> $(PROJECT_DIR)/UIProcess/Media/AudioSessionRoutingArbitratorProxy.messages.in
</span><span class="cx"> $(PROJECT_DIR)/UIProcess/Network/CustomProtocols/LegacyCustomProtocolManagerProxy.messages.in
</span><span class="lines">@@ -143,7 +142,6 @@
</span><span class="cx"> $(PROJECT_DIR)/WebProcess/Geolocation/WebGeolocationManager.messages.in
</span><span class="cx"> $(PROJECT_DIR)/WebProcess/Inspector/RemoteWebInspectorUI.messages.in
</span><span class="cx"> $(PROJECT_DIR)/WebProcess/Inspector/WebInspector.messages.in
</span><del>-$(PROJECT_DIR)/WebProcess/Inspector/WebInspectorUIExtensionController.messages.in
</del><span class="cx"> $(PROJECT_DIR)/WebProcess/Inspector/WebInspectorInterruptDispatcher.messages.in
</span><span class="cx"> $(PROJECT_DIR)/WebProcess/Inspector/WebInspectorUI.messages.in
</span><span class="cx"> $(PROJECT_DIR)/WebProcess/Network/NetworkProcessConnection.messages.in
</span></span></pre></div>
<a id="trunkSourceWebKitDerivedSourcesoutputxcfilelist"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/DerivedSources-output.xcfilelist (269523 => 269524)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/DerivedSources-output.xcfilelist     2020-11-06 18:42:14 UTC (rev 269523)
+++ trunk/Source/WebKit/DerivedSources-output.xcfilelist        2020-11-06 18:44:09 UTC (rev 269524)
</span><span class="lines">@@ -348,9 +348,6 @@
</span><span class="cx"> $(BUILT_PRODUCTS_DIR)/DerivedSources/WebKit2/WebIDBServerMessageReceiver.cpp
</span><span class="cx"> $(BUILT_PRODUCTS_DIR)/DerivedSources/WebKit2/WebIDBServerMessages.h
</span><span class="cx"> $(BUILT_PRODUCTS_DIR)/DerivedSources/WebKit2/WebIDBServerMessagesReplies.h
</span><del>-$(BUILT_PRODUCTS_DIR)/DerivedSources/WebKit2/WebInspectorUIExtensionControllerMessageReceiver.cpp
-$(BUILT_PRODUCTS_DIR)/DerivedSources/WebKit2/WebInspectorUIExtensionControllerMessages.h
-$(BUILT_PRODUCTS_DIR)/DerivedSources/WebKit2/WebInspectorUIExtensionControllerMessagesReplies.h
</del><span class="cx"> $(BUILT_PRODUCTS_DIR)/DerivedSources/WebKit2/WebInspectorInterruptDispatcherMessageReceiver.cpp
</span><span class="cx"> $(BUILT_PRODUCTS_DIR)/DerivedSources/WebKit2/WebInspectorInterruptDispatcherMessages.h
</span><span class="cx"> $(BUILT_PRODUCTS_DIR)/DerivedSources/WebKit2/WebInspectorInterruptDispatcherMessagesReplies.h
</span></span></pre></div>
<a id="trunkSourceWebKitDerivedSourcesmake"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/DerivedSources.make (269523 => 269524)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/DerivedSources.make  2020-11-06 18:42:14 UTC (rev 269523)
+++ trunk/Source/WebKit/DerivedSources.make     2020-11-06 18:44:09 UTC (rev 269524)
</span><span class="lines">@@ -184,7 +184,6 @@
</span><span class="cx">  WebProcess/UserContent/WebUserContentController \
</span><span class="cx">  WebProcess/Inspector/WebInspectorInterruptDispatcher \
</span><span class="cx">  WebProcess/Inspector/WebInspectorUI \
</span><del>-       WebProcess/Inspector/WebInspectorUIExtensionController \
</del><span class="cx">   WebProcess/Inspector/WebInspector \
</span><span class="cx">  WebProcess/Inspector/RemoteWebInspectorUI \
</span><span class="cx">  WebProcess/Plugins/PluginProcessConnectionManager \
</span></span></pre></div>
<a id="trunkSourceWebKitScriptswebkitmessagespy"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/Scripts/webkit/messages.py (269523 => 269524)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/Scripts/webkit/messages.py   2020-11-06 18:42:14 UTC (rev 269523)
+++ trunk/Source/WebKit/Scripts/webkit/messages.py      2020-11-06 18:44:09 UTC (rev 269524)
</span><span class="lines">@@ -673,7 +673,6 @@
</span><span class="cx">         'WebKit::FindOptions': ['"WebFindOptions.h"'],
</span><span class="cx">         'WebKit::GestureRecognizerState': ['"GestureTypes.h"'],
</span><span class="cx">         'WebKit::GestureType': ['"GestureTypes.h"'],
</span><del>-        'WebKit::InspectorExtensionError': ['"InspectorExtensionTypes.h"'],
</del><span class="cx">         'WebKit::LayerHostingContextID': ['"LayerHostingContext.h"'],
</span><span class="cx">         'WebKit::LayerHostingMode': ['"LayerTreeContext.h"'],
</span><span class="cx">         'WebKit::PageState': ['"SessionState.h"'],
</span></span></pre></div>
<a id="trunkSourceWebKitSharedAPIAPIObjecth"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/Shared/API/APIObject.h (269523 => 269524)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/Shared/API/APIObject.h       2020-11-06 18:42:14 UTC (rev 269523)
+++ trunk/Source/WebKit/Shared/API/APIObject.h  2020-11-06 18:44:09 UTC (rev 269524)
</span><span class="lines">@@ -137,9 +137,6 @@
</span><span class="cx">         IconDatabase,
</span><span class="cx">         Inspector,
</span><span class="cx">         InspectorConfiguration,
</span><del>-#if ENABLE(INSPECTOR_EXTENSIONS)
-        InspectorExtension,
-#endif
</del><span class="cx">         KeyValueStorageManager,
</span><span class="cx">         MediaCacheManager,
</span><span class="cx">         MessageListener,
</span><span class="lines">@@ -385,9 +382,6 @@
</span><span class="cx">         API::Object::Type::IconDatabase,
</span><span class="cx">         API::Object::Type::Inspector,
</span><span class="cx">         API::Object::Type::InspectorConfiguration,
</span><del>-#if ENABLE(INSPECTOR_EXTENSIONS)
-        API::Object::Type::InspectorExtension,
-#endif
</del><span class="cx">         API::Object::Type::KeyValueStorageManager,
</span><span class="cx">         API::Object::Type::MediaCacheManager,
</span><span class="cx">         API::Object::Type::MessageListener,
</span></span></pre></div>
<a id="trunkSourceWebKitSharedCocoaAPIObjectmm"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/Shared/Cocoa/APIObject.mm (269523 => 269524)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/Shared/Cocoa/APIObject.mm    2020-11-06 18:42:14 UTC (rev 269523)
+++ trunk/Source/WebKit/Shared/Cocoa/APIObject.mm       2020-11-06 18:44:09 UTC (rev 269524)
</span><span class="lines">@@ -99,10 +99,6 @@
</span><span class="cx"> #import "_WKApplicationManifestInternal.h"
</span><span class="cx"> #endif
</span><span class="cx"> 
</span><del>-#if ENABLE(INSPECTOR_EXTENSIONS)
-#import "_WKInspectorExtensionInternal.h"
-#endif
-
</del><span class="cx"> static const size_t minimumObjectAlignment = alignof(std::aligned_storage<std::numeric_limits<size_t>::max()>::type);
</span><span class="cx"> static_assert(minimumObjectAlignment >= alignof(void*), "Objects should always be at least pointer-aligned.");
</span><span class="cx"> static const size_t maximumExtraSpaceForAlignment = minimumObjectAlignment - alignof(void*);
</span><span class="lines">@@ -274,12 +270,6 @@
</span><span class="cx">         wrapper = [_WKInspectorConfiguration alloc];
</span><span class="cx">         break;
</span><span class="cx"> 
</span><del>-#if ENABLE(INSPECTOR_EXTENSIONS)
-    case Type::InspectorExtension:
-        wrapper = [_WKInspectorExtension alloc];
-        break;
-#endif
-
</del><span class="cx">     case Type::Navigation:
</span><span class="cx">         wrapper = [WKNavigation alloc];
</span><span class="cx">         break;
</span></span></pre></div>
<a id="trunkSourceWebKitSharedInspectorExtensionTypescpp"></a>
<div class="delfile"><h4>Deleted: trunk/Source/WebKit/Shared/InspectorExtensionTypes.cpp (269523 => 269524)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/Shared/InspectorExtensionTypes.cpp   2020-11-06 18:42:14 UTC (rev 269523)
+++ trunk/Source/WebKit/Shared/InspectorExtensionTypes.cpp      2020-11-06 18:44:09 UTC (rev 269524)
</span><span class="lines">@@ -1,54 +0,0 @@
</span><del>-/*
- * Copyright (C) 2020 Apple 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:
- * 1. Redistributions of source code must retain the above copyright
- *    notice, this list of conditions and the following disclaimer.
- * 2. Redistributions in binary form must reproduce the above copyright
- *    notice, this list of conditions and the following disclaimer in the
- *    documentation and/or other materials provided with the distribution.
- *
- * THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS''
- * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO,
- * THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
- * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR ITS CONTRIBUTORS
- * BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
- * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
- * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
- * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
- * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
- * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF
- * THE POSSIBILITY OF SUCH DAMAGE.
- */
-
-#include "config.h"
-#include "InspectorExtensionTypes.h"
-
-#if ENABLE(INSPECTOR_EXTENSIONS)
-
-#include <wtf/text/WTFString.h>
-
-namespace WebKit {
-
-WTF::String inspectorExtensionErrorToString(InspectorExtensionError error)
-{
-    switch (error) {
-    case InspectorExtensionError::InternalError:
-        return "InternalError"_s;
-    case InspectorExtensionError::InvalidRequest:
-        return "InvalidRequest"_s;
-    case InspectorExtensionError::ContextDestroyed:
-        return "ContextDestroyed"_s;
-    case InspectorExtensionError::RegistrationFailed:
-        return "RegistrationFailed"_s;
-    }
-
-    ASSERT_NOT_REACHED();
-    return "InternalError";
-}
-
-} // namespace WebKit
-
-#endif // ENABLE(INSPECTOR_EXTENSIONS)
</del></span></pre></div>
<a id="trunkSourceWebKitSharedInspectorExtensionTypesh"></a>
<div class="delfile"><h4>Deleted: trunk/Source/WebKit/Shared/InspectorExtensionTypes.h (269523 => 269524)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/Shared/InspectorExtensionTypes.h     2020-11-06 18:42:14 UTC (rev 269523)
+++ trunk/Source/WebKit/Shared/InspectorExtensionTypes.h        2020-11-06 18:44:09 UTC (rev 269524)
</span><span class="lines">@@ -1,61 +0,0 @@
</span><del>-/*
- * Copyright (C) 2020 Apple 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:
- * 1. Redistributions of source code must retain the above copyright
- *    notice, this list of conditions and the following disclaimer.
- * 2. Redistributions in binary form must reproduce the above copyright
- *    notice, this list of conditions and the following disclaimer in the
- *    documentation and/or other materials provided with the distribution.
- *
- * THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS''
- * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO,
- * THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
- * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR ITS CONTRIBUTORS
- * BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
- * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
- * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
- * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
- * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
- * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF
- * THE POSSIBILITY OF SUCH DAMAGE.
- */
-
-#pragma once
-
-#include <wtf/Forward.h>
-
-#if ENABLE(INSPECTOR_EXTENSIONS)
-
-namespace WebKit {
-
-using InspectorExtensionTabID = WTF::String;
-using InspectorExtensionID = WTF::String;
-
-enum class InspectorExtensionError : uint8_t {
-    ContextDestroyed,
-    InternalError,
-    InvalidRequest,
-    RegistrationFailed,
-};
-
-WTF::String inspectorExtensionErrorToString(InspectorExtensionError);
-
-} // namespace WebKit
-
-namespace WTF {
-
-template<> struct EnumTraits<WebKit::InspectorExtensionError> {
-    using values = EnumValues<
-        WebKit::InspectorExtensionError,
-        WebKit::InspectorExtensionError::InvalidRequest,
-        WebKit::InspectorExtensionError::ContextDestroyed,
-        WebKit::InspectorExtensionError::RegistrationFailed
-    >;
-};
-
-}
-
-#endif // ENABLE(INSPECTOR_EXTENSIONS)
</del></span></pre></div>
<a id="trunkSourceWebKitSourcestxt"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/Sources.txt (269523 => 269524)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/Sources.txt  2020-11-06 18:42:14 UTC (rev 269523)
+++ trunk/Source/WebKit/Sources.txt     2020-11-06 18:44:09 UTC (rev 269524)
</span><span class="lines">@@ -161,7 +161,6 @@
</span><span class="cx"> Shared/EditorState.cpp
</span><span class="cx"> Shared/FontInfo.cpp
</span><span class="cx"> Shared/FrameInfoData.cpp
</span><del>-Shared/InspectorExtensionTypes.cpp
</del><span class="cx"> Shared/LayerTreeContext.cpp
</span><span class="cx"> Shared/LoadParameters.cpp
</span><span class="cx"> Shared/NavigationActionData.cpp
</span><span class="lines">@@ -438,7 +437,6 @@
</span><span class="cx"> UIProcess/Inspector/InspectorTargetProxy.cpp
</span><span class="cx"> UIProcess/Inspector/RemoteWebInspectorProxy.cpp
</span><span class="cx"> UIProcess/Inspector/WebInspectorProxy.cpp
</span><del>-UIProcess/Inspector/WebInspectorUIExtensionControllerProxy.cpp
</del><span class="cx"> UIProcess/Inspector/WebInspectorUtilities.cpp
</span><span class="cx"> UIProcess/Inspector/WebPageDebuggable.cpp
</span><span class="cx"> UIProcess/Inspector/WebPageInspectorController.cpp
</span><span class="lines">@@ -552,7 +550,6 @@
</span><span class="cx"> WebProcess/Inspector/WebInspectorClient.cpp
</span><span class="cx"> WebProcess/Inspector/WebInspectorInterruptDispatcher.cpp
</span><span class="cx"> WebProcess/Inspector/WebInspectorUI.cpp
</span><del>-WebProcess/Inspector/WebInspectorUIExtensionController.cpp
</del><span class="cx"> WebProcess/Inspector/WebPageInspectorTarget.cpp
</span><span class="cx"> WebProcess/Inspector/WebPageInspectorTargetController.cpp
</span><span class="cx"> WebProcess/Inspector/WebPageInspectorTargetFrontendChannel.cpp
</span></span></pre></div>
<a id="trunkSourceWebKitSourcesCocoatxt"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/SourcesCocoa.txt (269523 => 269524)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/SourcesCocoa.txt     2020-11-06 18:42:14 UTC (rev 269523)
+++ trunk/Source/WebKit/SourcesCocoa.txt        2020-11-06 18:44:09 UTC (rev 269524)
</span><span class="lines">@@ -259,7 +259,6 @@
</span><span class="cx"> UIProcess/API/Cocoa/_WKInspectorConfiguration.mm
</span><span class="cx"> UIProcess/API/Cocoa/_WKInspectorTesting.mm
</span><span class="cx"> UIProcess/API/Cocoa/_WKInspectorDebuggableInfo.mm
</span><del>-UIProcess/API/Cocoa/_WKInspectorExtension.mm
</del><span class="cx"> UIProcess/API/Cocoa/_WKInspectorWindow.mm
</span><span class="cx"> UIProcess/API/Cocoa/_WKInternalDebugFeature.mm
</span><span class="cx"> UIProcess/API/Cocoa/_WKLinkIconParameters.mm
</span></span></pre></div>
<a id="trunkSourceWebKitUIProcessAPIAPIInspectorExtensionh"></a>
<div class="delfile"><h4>Deleted: trunk/Source/WebKit/UIProcess/API/APIInspectorExtension.h (269523 => 269524)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/UIProcess/API/APIInspectorExtension.h        2020-11-06 18:42:14 UTC (rev 269523)
+++ trunk/Source/WebKit/UIProcess/API/APIInspectorExtension.h   2020-11-06 18:44:09 UTC (rev 269524)
</span><span class="lines">@@ -1,55 +0,0 @@
</span><del>-/*
- * Copyright (C) 2020 Apple 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:
- * 1. Redistributions of source code must retain the above copyright
- *    notice, this list of conditions and the following disclaimer.
- * 2. Redistributions in binary form must reproduce the above copyright
- *    notice, this list of conditions and the following disclaimer in the
- *    documentation and/or other materials provided with the distribution.
- *
- * THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS''
- * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO,
- * THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
- * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR ITS CONTRIBUTORS
- * BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
- * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
- * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
- * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
- * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
- * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF
- * THE POSSIBILITY OF SUCH DAMAGE.
- */
-
-#pragma once
-
-#if ENABLE(INSPECTOR_EXTENSIONS)
-
-#include "APIObject.h"
-#include <wtf/Forward.h>
-
-namespace API {
-
-class InspectorExtension final : public API::ObjectImpl<Object::Type::InspectorExtension> {
-public:
-    static Ref<InspectorExtension> create(const WTF::String& identifier)
-    {
-        return adoptRef(*new InspectorExtension(identifier));
-    }
-
-    explicit InspectorExtension(const WTF::String& identifier)
-        : m_identifier(identifier)
-    {
-    }
-
-    const WTF::String& identifier() const { return m_identifier; }
-
-private:
-    WTF::String m_identifier;
-};
-
-} // namespace API
-
-#endif // ENABLE(INSPECTOR_EXTENSIONS)
</del></span></pre></div>
<a id="trunkSourceWebKitUIProcessAPICocoa_WKInspectorh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/UIProcess/API/Cocoa/_WKInspector.h (269523 => 269524)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/UIProcess/API/Cocoa/_WKInspector.h   2020-11-06 18:42:14 UTC (rev 269523)
+++ trunk/Source/WebKit/UIProcess/API/Cocoa/_WKInspector.h      2020-11-06 18:44:09 UTC (rev 269524)
</span><span class="lines">@@ -25,15 +25,17 @@
</span><span class="cx"> 
</span><span class="cx"> #import <Foundation/Foundation.h>
</span><span class="cx"> #import <WebKit/WKFoundation.h>
</span><del>-#import <WebKit/_WKInspectorExtensionHost.h>
</del><span class="cx"> 
</span><del>-NS_ASSUME_NONNULL_BEGIN
-
</del><span class="cx"> @class WKWebView;
</span><span class="cx"> @class _WKFrameHandle;
</span><del>-@class _WKInspectorExtension;
</del><span class="cx"> @protocol _WKInspectorDelegate;
</span><span class="cx"> 
</span><ins>+NS_ASSUME_NONNULL_BEGIN
+
+@protocol _WKInspectorExtensionHost
+- (void)close;
+@end
+
</ins><span class="cx"> WK_CLASS_AVAILABLE(macos(10.14.4), ios(12.2))
</span><span class="cx"> @interface _WKInspector : NSObject <_WKInspectorExtensionHost>
</span><span class="cx"> 
</span><span class="lines">@@ -50,7 +52,6 @@
</span><span class="cx"> 
</span><span class="cx"> - (void)connect;
</span><span class="cx"> - (void)show;
</span><del>-- (void)close;
</del><span class="cx"> - (void)hide;
</span><span class="cx"> - (void)showConsole;
</span><span class="cx"> - (void)showResources;
</span></span></pre></div>
<a id="trunkSourceWebKitUIProcessAPICocoa_WKInspectormm"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/UIProcess/API/Cocoa/_WKInspector.mm (269523 => 269524)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/UIProcess/API/Cocoa/_WKInspector.mm  2020-11-06 18:42:14 UTC (rev 269523)
+++ trunk/Source/WebKit/UIProcess/API/Cocoa/_WKInspector.mm     2020-11-06 18:44:09 UTC (rev 269524)
</span><span class="lines">@@ -27,7 +27,6 @@
</span><span class="cx"> #import "_WKInspectorInternal.h"
</span><span class="cx"> 
</span><span class="cx"> #import "InspectorDelegate.h"
</span><del>-#import "WKError.h"
</del><span class="cx"> #import "WKWebViewInternal.h"
</span><span class="cx"> #import "WebPageProxy.h"
</span><span class="cx"> #import "WebProcessProxy.h"
</span><span class="lines">@@ -38,13 +37,6 @@
</span><span class="cx"> #import <wtf/RetainPtr.h>
</span><span class="cx"> #import <wtf/text/WTFString.h>
</span><span class="cx"> 
</span><del>-#if ENABLE(INSPECTOR_EXTENSIONS)
-#import "APIInspectorExtension.h"
-#import "WebInspectorUIExtensionControllerProxy.h"
-#import "_WKInspectorExtensionInternal.h"
-#import <wtf/BlockPtr.h>
-#endif
-
</del><span class="cx"> @implementation _WKInspector
</span><span class="cx"> 
</span><span class="cx"> // MARK: _WKInspector methods
</span><span class="lines">@@ -202,38 +194,4 @@
</span><span class="cx">     return *_inspector;
</span><span class="cx"> }
</span><span class="cx"> 
</span><del>-// MARK: _WKInspectorExtensionHost methods
-
-- (void)registerExtensionWithID:(NSString *)extensionID displayName:(NSString *)displayName completionHandler:(void(^)(NSError *, _WKInspectorExtension *))completionHandler
-{
-#if ENABLE(INSPECTOR_EXTENSIONS)
-    _inspector->extensionController().registerExtension(extensionID, displayName, [protectedExtensionID = retainPtr(extensionID), protectedSelf = retainPtr(self), capturedBlock = makeBlockPtr(completionHandler)] (Expected<bool, WebKit::InspectorExtensionError> result) mutable {
-        if (!result) {
-            capturedBlock([NSError errorWithDomain:WKErrorDomain code:WKErrorUnknown userInfo:@{ NSLocalizedFailureReasonErrorKey: inspectorExtensionErrorToString(result.error())}], nil);
-            return;
-        }
-
-        capturedBlock(nil, [[wrapper(API::InspectorExtension::create(protectedExtensionID.get())) retain] autorelease]);
-    });
-#else
-    completionHandler([NSError errorWithDomain:WKErrorDomain code:WKErrorUnknown userInfo:nil], nil);
-#endif
-}
-
-- (void)unregisterExtension:(_WKInspectorExtension *)extension completionHandler:(void(^)(NSError *))completionHandler
-{
-#if ENABLE(INSPECTOR_EXTENSIONS)
-    _inspector->extensionController().unregisterExtension(extension.extensionID, [protectedSelf = retainPtr(self), capturedBlock = makeBlockPtr(completionHandler)] (Expected<bool, WebKit::InspectorExtensionError> result) mutable {
-        if (!result) {
-            capturedBlock([NSError errorWithDomain:WKErrorDomain code:WKErrorUnknown userInfo:@{ NSLocalizedFailureReasonErrorKey: inspectorExtensionErrorToString(result.error())}]);
-            return;
-        }
-
-        capturedBlock(nil);
-    });
-#else
-    completionHandler([NSError errorWithDomain:WKErrorDomain code:WKErrorUnknown userInfo:nil]);
-#endif
-}
-
</del><span class="cx"> @end
</span></span></pre></div>
<a id="trunkSourceWebKitUIProcessAPICocoa_WKInspectorExtensionh"></a>
<div class="delfile"><h4>Deleted: trunk/Source/WebKit/UIProcess/API/Cocoa/_WKInspectorExtension.h (269523 => 269524)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/UIProcess/API/Cocoa/_WKInspectorExtension.h  2020-11-06 18:42:14 UTC (rev 269523)
+++ trunk/Source/WebKit/UIProcess/API/Cocoa/_WKInspectorExtension.h     2020-11-06 18:44:09 UTC (rev 269524)
</span><span class="lines">@@ -1,40 +0,0 @@
</span><del>-/*
- * Copyright (C) 2020 Apple 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:
- * 1. Redistributions of source code must retain the above copyright
- *    notice, this list of conditions and the following disclaimer.
- * 2. Redistributions in binary form must reproduce the above copyright
- *    notice, this list of conditions and the following disclaimer in the
- *    documentation and/or other materials provided with the distribution.
- *
- * THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS''
- * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO,
- * THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
- * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR ITS CONTRIBUTORS
- * BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
- * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
- * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
- * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
- * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
- * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF
- * THE POSSIBILITY OF SUCH DAMAGE.
- */
-
-#import <Foundation/Foundation.h>
-#import <WebKit/WKFoundation.h>
-
-NS_ASSUME_NONNULL_BEGIN
-
-WK_CLASS_AVAILABLE(macos(WK_MAC_TBA))
-@interface _WKInspectorExtension : NSObject
-
-- (instancetype)init NS_UNAVAILABLE;
-
-@property (readonly, nonatomic) NSString *extensionID;
-
-@end
-
-NS_ASSUME_NONNULL_END
</del></span></pre></div>
<a id="trunkSourceWebKitUIProcessAPICocoa_WKInspectorExtensionmm"></a>
<div class="delfile"><h4>Deleted: trunk/Source/WebKit/UIProcess/API/Cocoa/_WKInspectorExtension.mm (269523 => 269524)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/UIProcess/API/Cocoa/_WKInspectorExtension.mm 2020-11-06 18:42:14 UTC (rev 269523)
+++ trunk/Source/WebKit/UIProcess/API/Cocoa/_WKInspectorExtension.mm    2020-11-06 18:44:09 UTC (rev 269524)
</span><span class="lines">@@ -1,69 +0,0 @@
</span><del>-/*
- * Copyright (C) 2020 Apple 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:
- * 1. Redistributions of source code must retain the above copyright
- *    notice, this list of conditions and the following disclaimer.
- * 2. Redistributions in binary form must reproduce the above copyright
- *    notice, this list of conditions and the following disclaimer in the
- *    documentation and/or other materials provided with the distribution.
- *
- * THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS''
- * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO,
- * THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
- * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR ITS CONTRIBUTORS
- * BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
- * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
- * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
- * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
- * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
- * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF
- * THE POSSIBILITY OF SUCH DAMAGE.
- */
-
-#import "config.h"
-#import "_WKInspectorExtensionInternal.h"
-
-NS_ASSUME_NONNULL_BEGIN
-
-@implementation _WKInspectorExtension
-
-#if ENABLE(INSPECTOR_EXTENSIONS)
-
-- (instancetype)initWithIdentifier:(NSString *)extensionID
-{
-    if (!(self = [super init]))
-        return nil;
-
-    API::Object::constructInWrapper<API::InspectorExtension>(self, extensionID);
-    
-    return self;
-}
-
-- (void)dealloc
-{
-    _extension->API::InspectorExtension::~InspectorExtension();
-
-    [super dealloc];
-}
-
-
-- (API::Object&)_apiObject
-{
-    return *_extension;
-}
-
-// MARK: Properties.
-
-- (NSString *)extensionID
-{
-    return _extension->identifier();
-}
-
-#endif // ENABLE(INSPECTOR_EXTENSIONS)
-
-@end
-
-NS_ASSUME_NONNULL_END
</del></span></pre></div>
<a id="trunkSourceWebKitUIProcessAPICocoa_WKInspectorExtensionHosth"></a>
<div class="delfile"><h4>Deleted: trunk/Source/WebKit/UIProcess/API/Cocoa/_WKInspectorExtensionHost.h (269523 => 269524)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/UIProcess/API/Cocoa/_WKInspectorExtensionHost.h      2020-11-06 18:42:14 UTC (rev 269523)
+++ trunk/Source/WebKit/UIProcess/API/Cocoa/_WKInspectorExtensionHost.h 2020-11-06 18:44:09 UTC (rev 269524)
</span><span class="lines">@@ -1,62 +0,0 @@
</span><del>-/*
- * Copyright (C) 2020 Apple 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:
- * 1. Redistributions of source code must retain the above copyright
- *    notice, this list of conditions and the following disclaimer.
- * 2. Redistributions in binary form must reproduce the above copyright
- *    notice, this list of conditions and the following disclaimer in the
- *    documentation and/or other materials provided with the distribution.
- *
- * THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS''
- * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO,
- * THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
- * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR ITS CONTRIBUTORS
- * BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
- * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
- * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
- * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
- * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
- * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF
- * THE POSSIBILITY OF SUCH DAMAGE.
- */
-
-#import <Foundation/Foundation.h>
-#import <WebKit/WKFoundation.h>
-
-NS_ASSUME_NONNULL_BEGIN
-
-@class _WKInspectorExtension;
-
-@protocol _WKInspectorExtensionHost
-@optional
-
-/**
- * @abstract Registers a Web Extension with the associated Web Inspector.
- * @param extensionID A unique identifier for the extension.
- * @param displayName A localized display name for the extension.
- * @param completionHandler The completion handler to be called when registration succeeds or fails.
- *
- * Web Extensions in Web Inspector are active as soon as they are registered.
- */
-- (void)registerExtensionWithID:(NSString *)extensionID displayName:(NSString *)displayName completionHandler:(void(^)(NSError * _Nullable, _WKInspectorExtension * _Nullable))completionHandler;
-
-/**
- * @abstract Unregisters a Web Extension with the associated Web Inspector.
- * @param extensionID A unique identifier for the extension.
- * @param completionHandler The completion handler to be called when unregistering succeeds or fails.
- *
- * Unregistering an extension will automatically close any associated sidebars/tabs.
- */
-- (void)unregisterExtension:(_WKInspectorExtension *)extension completionHandler:(void(^)(NSError * _Nullable))completionHandler;
-
-/**
- * @abstract Closes the associated Web Inspector instance. This will cause all
- * registered _WKInspectorExtensions to be unregistered and invalidated.
- */
-- (void)close;
-@end
-
-NS_ASSUME_NONNULL_END
</del></span></pre></div>
<a id="trunkSourceWebKitUIProcessAPICocoa_WKInspectorExtensionInternalh"></a>
<div class="delfile"><h4>Deleted: trunk/Source/WebKit/UIProcess/API/Cocoa/_WKInspectorExtensionInternal.h (269523 => 269524)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/UIProcess/API/Cocoa/_WKInspectorExtensionInternal.h  2020-11-06 18:42:14 UTC (rev 269523)
+++ trunk/Source/WebKit/UIProcess/API/Cocoa/_WKInspectorExtensionInternal.h     2020-11-06 18:44:09 UTC (rev 269524)
</span><span class="lines">@@ -1,54 +0,0 @@
</span><del>-/*
- * Copyright (C) 2020 Apple 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:
- * 1. Redistributions of source code must retain the above copyright
- *    notice, this list of conditions and the following disclaimer.
- * 2. Redistributions in binary form must reproduce the above copyright
- *    notice, this list of conditions and the following disclaimer in the
- *    documentation and/or other materials provided with the distribution.
- *
- * THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS''
- * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO,
- * THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
- * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR ITS CONTRIBUTORS
- * BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
- * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
- * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
- * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
- * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
- * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF
- * THE POSSIBILITY OF SUCH DAMAGE.
- */
-
-#import "_WKInspectorExtension.h"
-
-#if ENABLE(INSPECTOR_EXTENSIONS)
-
-#import "APIInspectorExtension.h"
-#import "WKObject.h"
-
-namespace WebKit {
-
-template<> struct WrapperTraits<API::InspectorExtension> {
-    using WrapperClass = _WKInspectorExtension;
-};
-
-} // namespace WebKit
-
-NS_ASSUME_NONNULL_BEGIN
-
-@interface _WKInspectorExtension () <WKObject> {
-@package
-    API::ObjectStorage<API::InspectorExtension> _extension;
-}
-
-- (instancetype)initWithIdentifier:(NSString *)extensionIdentifier;
-
-@end
-
-NS_ASSUME_NONNULL_END
-
-#endif // ENABLE(INSPECTOR_EXTENSIONS)
</del></span></pre></div>
<a id="trunkSourceWebKitUIProcessAPICocoa_WKRemoteWebInspectorViewControllerh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/UIProcess/API/Cocoa/_WKRemoteWebInspectorViewController.h (269523 => 269524)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/UIProcess/API/Cocoa/_WKRemoteWebInspectorViewController.h    2020-11-06 18:42:14 UTC (rev 269523)
+++ trunk/Source/WebKit/UIProcess/API/Cocoa/_WKRemoteWebInspectorViewController.h       2020-11-06 18:44:09 UTC (rev 269524)
</span><span class="lines">@@ -24,7 +24,6 @@
</span><span class="cx">  */
</span><span class="cx"> 
</span><span class="cx"> #import <WebKit/WKFoundation.h>
</span><del>-#import <WebKit/_WKInspectorExtensionHost.h>
</del><span class="cx"> 
</span><span class="cx"> #if !TARGET_OS_IPHONE
</span><span class="cx"> 
</span><span class="lines">@@ -45,7 +44,7 @@
</span><span class="cx"> } WK_API_AVAILABLE(macos(10.12.3), ios(10.3));
</span><span class="cx"> 
</span><span class="cx"> WK_CLASS_AVAILABLE(macos(10.12.3), ios(10.3))
</span><del>-@interface _WKRemoteWebInspectorViewController : NSObject <_WKInspectorExtensionHost>
</del><ins>+@interface _WKRemoteWebInspectorViewController : NSObject
</ins><span class="cx"> 
</span><span class="cx"> @property (nonatomic, assign) id <_WKRemoteWebInspectorViewControllerDelegate> delegate;
</span><span class="cx"> 
</span></span></pre></div>
<a id="trunkSourceWebKitUIProcessAPICocoa_WKRemoteWebInspectorViewControllermm"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/UIProcess/API/Cocoa/_WKRemoteWebInspectorViewController.mm (269523 => 269524)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/UIProcess/API/Cocoa/_WKRemoteWebInspectorViewController.mm   2020-11-06 18:42:14 UTC (rev 269523)
+++ trunk/Source/WebKit/UIProcess/API/Cocoa/_WKRemoteWebInspectorViewController.mm      2020-11-06 18:44:09 UTC (rev 269524)
</span><span class="lines">@@ -36,14 +36,6 @@
</span><span class="cx"> #import "_WKInspectorConfigurationInternal.h"
</span><span class="cx"> #import "_WKInspectorDebuggableInfoInternal.h"
</span><span class="cx"> 
</span><del>-#if ENABLE(INSPECTOR_EXTENSIONS)
-#import "APIInspectorExtension.h"
-#import "WKError.h"
-#import "WebInspectorUIExtensionControllerProxy.h"
-#import "_WKInspectorExtensionInternal.h"
-#import <wtf/BlockPtr.h>
-#endif
-
</del><span class="cx"> NS_ASSUME_NONNULL_BEGIN
</span><span class="cx"> 
</span><span class="cx"> @interface _WKRemoteWebInspectorViewController ()
</span><span class="lines">@@ -190,40 +182,6 @@
</span><span class="cx">     m_remoteInspectorProxy->setDiagnosticLoggingAvailable(!!delegate);
</span><span class="cx"> }
</span><span class="cx"> 
</span><del>-// MARK: _WKInspectorExtensionHost methods
-
-- (void)registerExtensionWithID:(NSString *)extensionID displayName:(NSString *)displayName completionHandler:(void(^)(NSError *, _WKInspectorExtension *))completionHandler
-{
-#if ENABLE(INSPECTOR_EXTENSIONS)
-    m_remoteInspectorProxy->extensionController().registerExtension(extensionID, displayName, [protectedExtensionID = retainPtr(extensionID), protectedSelf = retainPtr(self), capturedBlock = makeBlockPtr(completionHandler)] (Expected<bool, WebKit::InspectorExtensionError> result) mutable {
-        if (!result) {
-            capturedBlock([NSError errorWithDomain:WKErrorDomain code:WKErrorUnknown userInfo:@{ NSLocalizedFailureReasonErrorKey: inspectorExtensionErrorToString(result.error())}], nil);
-            return;
-        }
-
-        capturedBlock(nil, [[wrapper(API::InspectorExtension::create(protectedExtensionID.get())) retain] autorelease]);
-    });
-#else
-    completionHandler([NSError errorWithDomain:WKErrorDomain code:WKErrorUnknown userInfo:nil], nil);
-#endif
-}
-
-- (void)unregisterExtension:(_WKInspectorExtension *)extension completionHandler:(void(^)(NSError *))completionHandler
-{
-#if ENABLE(INSPECTOR_EXTENSIONS)
-    m_remoteInspectorProxy->extensionController().unregisterExtension(extension.extensionID, [protectedSelf = retainPtr(self), capturedBlock = makeBlockPtr(completionHandler)] (Expected<bool, WebKit::InspectorExtensionError> result) mutable {
-        if (!result) {
-            capturedBlock([NSError errorWithDomain:WKErrorDomain code:WKErrorUnknown userInfo:@{ NSLocalizedFailureReasonErrorKey: inspectorExtensionErrorToString(result.error())}]);
-            return;
-        }
-
-        capturedBlock(nil);
-    });
-#else
-    completionHandler([NSError errorWithDomain:WKErrorDomain code:WKErrorUnknown userInfo:nil]);
-#endif
-}
-
</del><span class="cx"> @end
</span><span class="cx"> 
</span><span class="cx"> NS_ASSUME_NONNULL_END
</span></span></pre></div>
<a id="trunkSourceWebKitUIProcessInspectorRemoteWebInspectorProxycpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/UIProcess/Inspector/RemoteWebInspectorProxy.cpp (269523 => 269524)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/UIProcess/Inspector/RemoteWebInspectorProxy.cpp      2020-11-06 18:42:14 UTC (rev 269523)
+++ trunk/Source/WebKit/UIProcess/Inspector/RemoteWebInspectorProxy.cpp 2020-11-06 18:44:09 UTC (rev 269524)
</span><span class="lines">@@ -36,10 +36,6 @@
</span><span class="cx"> #include <WebCore/CertificateInfo.h>
</span><span class="cx"> #include <WebCore/NotImplemented.h>
</span><span class="cx"> 
</span><del>-#if ENABLE(INSPECTOR_EXTENSIONS)
-#include "WebInspectorUIExtensionControllerProxy.h"
-#endif
-
</del><span class="cx"> namespace WebKit {
</span><span class="cx"> using namespace WebCore;
</span><span class="cx"> 
</span><span class="lines">@@ -99,13 +95,6 @@
</span><span class="cx">     m_inspectorPage->send(Messages::RemoteWebInspectorUI::SendMessageToFrontend(message));
</span><span class="cx"> }
</span><span class="cx"> 
</span><del>-void RemoteWebInspectorProxy::frontendLoaded()
-{
-#if ENABLE(INSPECTOR_EXTENSIONS)
-    m_extensionController->inspectorFrontendLoaded();
-#endif
-}
-
</del><span class="cx"> void RemoteWebInspectorProxy::frontendDidClose()
</span><span class="cx"> {
</span><span class="cx">     Ref<RemoteWebInspectorProxy> protect(*this);
</span><span class="lines">@@ -186,10 +175,6 @@
</span><span class="cx"> 
</span><span class="cx">     m_inspectorPage->process().addMessageReceiver(Messages::RemoteWebInspectorProxy::messageReceiverName(), m_inspectorPage->webPageID(), *this);
</span><span class="cx">     m_inspectorPage->process().assumeReadAccessToBaseURL(*m_inspectorPage, WebInspectorProxy::inspectorBaseURL());
</span><del>-
-#if ENABLE(INSPECTOR_EXTENSIONS)
-    m_extensionController = makeUnique<WebInspectorUIExtensionControllerProxy>(*m_inspectorPage);
-#endif
</del><span class="cx"> }
</span><span class="cx"> 
</span><span class="cx"> void RemoteWebInspectorProxy::closeFrontendPageAndWindow()
</span></span></pre></div>
<a id="trunkSourceWebKitUIProcessInspectorRemoteWebInspectorProxyh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/UIProcess/Inspector/RemoteWebInspectorProxy.h (269523 => 269524)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/UIProcess/Inspector/RemoteWebInspectorProxy.h        2020-11-06 18:42:14 UTC (rev 269523)
+++ trunk/Source/WebKit/UIProcess/Inspector/RemoteWebInspectorProxy.h   2020-11-06 18:44:09 UTC (rev 269524)
</span><span class="lines">@@ -56,9 +56,6 @@
</span><span class="cx"> class RemoteWebInspectorProxy;
</span><span class="cx"> class WebPageProxy;
</span><span class="cx"> class WebView;
</span><del>-#if ENABLE(INSPECTOR_EXTENSIONS)
-class WebInspectorUIExtensionControllerProxy;
-#endif
</del><span class="cx"> 
</span><span class="cx"> class RemoteWebInspectorProxyClient {
</span><span class="cx"> public:
</span><span class="lines">@@ -91,10 +88,6 @@
</span><span class="cx"> 
</span><span class="cx">     void sendMessageToFrontend(const String& message);
</span><span class="cx"> 
</span><del>-#if ENABLE(INSPECTOR_EXTENSIONS)
-    WebInspectorUIExtensionControllerProxy& extensionController() const { return *m_extensionController; }
-#endif
-    
</del><span class="cx"> #if PLATFORM(MAC)
</span><span class="cx">     NSWindow *window() const { return m_window.get(); }
</span><span class="cx">     WKWebView *webView() const;
</span><span class="lines">@@ -124,7 +117,6 @@
</span><span class="cx">     void didReceiveMessage(IPC::Connection&, IPC::Decoder&) override;
</span><span class="cx"> 
</span><span class="cx">     // RemoteWebInspectorProxy messages.
</span><del>-    void frontendLoaded();
</del><span class="cx">     void frontendDidClose();
</span><span class="cx">     void reopen();
</span><span class="cx">     void resetState();
</span><span class="lines">@@ -157,10 +149,6 @@
</span><span class="cx">     RemoteWebInspectorProxyClient* m_client { nullptr };
</span><span class="cx">     WebPageProxy* m_inspectorPage { nullptr };
</span><span class="cx"> 
</span><del>-#if ENABLE(INSPECTOR_EXTENSIONS)
-    std::unique_ptr<WebInspectorUIExtensionControllerProxy> m_extensionController;
-#endif
-    
</del><span class="cx">     Ref<API::DebuggableInfo> m_debuggableInfo;
</span><span class="cx">     String m_backendCommandsURL;
</span><span class="cx"> 
</span></span></pre></div>
<a id="trunkSourceWebKitUIProcessInspectorRemoteWebInspectorProxymessagesin"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/UIProcess/Inspector/RemoteWebInspectorProxy.messages.in (269523 => 269524)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/UIProcess/Inspector/RemoteWebInspectorProxy.messages.in      2020-11-06 18:42:14 UTC (rev 269523)
+++ trunk/Source/WebKit/UIProcess/Inspector/RemoteWebInspectorProxy.messages.in 2020-11-06 18:44:09 UTC (rev 269524)
</span><span class="lines">@@ -21,7 +21,6 @@
</span><span class="cx"> # OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
</span><span class="cx"> 
</span><span class="cx"> messages -> RemoteWebInspectorProxy {
</span><del>-    FrontendLoaded()
</del><span class="cx">     FrontendDidClose()
</span><span class="cx">     Reopen()
</span><span class="cx">     ResetState()
</span></span></pre></div>
<a id="trunkSourceWebKitUIProcessInspectorWebInspectorProxycpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/UIProcess/Inspector/WebInspectorProxy.cpp (269523 => 269524)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/UIProcess/Inspector/WebInspectorProxy.cpp    2020-11-06 18:42:14 UTC (rev 269523)
+++ trunk/Source/WebKit/UIProcess/Inspector/WebInspectorProxy.cpp       2020-11-06 18:44:09 UTC (rev 269524)
</span><span class="lines">@@ -37,7 +37,6 @@
</span><span class="cx"> #include "WebInspectorInterruptDispatcherMessages.h"
</span><span class="cx"> #include "WebInspectorMessages.h"
</span><span class="cx"> #include "WebInspectorProxyMessages.h"
</span><del>-#include "WebInspectorUIExtensionControllerProxy.h"
</del><span class="cx"> #include "WebInspectorUIMessages.h"
</span><span class="cx"> #include "WebPageGroup.h"
</span><span class="cx"> #include "WebPageInspectorController.h"
</span><span class="lines">@@ -422,10 +421,6 @@
</span><span class="cx"> 
</span><span class="cx">     m_inspectorPage->process().addMessageReceiver(Messages::WebInspectorProxy::messageReceiverName(), m_inspectedPage->identifier(), *this);
</span><span class="cx">     m_inspectorPage->process().assumeReadAccessToBaseURL(*m_inspectorPage, WebInspectorProxy::inspectorBaseURL());
</span><del>-
-#if ENABLE(INSPECTOR_EXTENSIONS)
-    m_extensionController = makeUnique<WebInspectorUIExtensionControllerProxy>(*m_inspectorPage);
-#endif
</del><span class="cx"> }
</span><span class="cx"> 
</span><span class="cx"> void WebInspectorProxy::openLocalInspectorFrontend(bool canAttach, bool underTest)
</span><span class="lines">@@ -551,10 +546,6 @@
</span><span class="cx">     if (m_isAttached)
</span><span class="cx">         platformDetach();
</span><span class="cx"> 
</span><del>-#if ENABLE(INSPECTOR_EXTENSIONS)
-    m_extensionController = nullptr;
-#endif
-    
</del><span class="cx">     // Null out m_inspectorPage after platformDetach(), so the views can be cleaned up correctly.
</span><span class="cx">     m_inspectorPage = nullptr;
</span><span class="cx"> 
</span><span class="lines">@@ -580,10 +571,6 @@
</span><span class="cx"> 
</span><span class="cx">     if (auto* automationSession = m_inspectedPage->process().processPool().automationSession())
</span><span class="cx">         automationSession->inspectorFrontendLoaded(*m_inspectedPage);
</span><del>-    
-#if ENABLE(INSPECTOR_EXTENSIONS)
-    m_extensionController->inspectorFrontendLoaded();
-#endif
</del><span class="cx"> }
</span><span class="cx"> 
</span><span class="cx"> void WebInspectorProxy::bringToFront()
</span></span></pre></div>
<a id="trunkSourceWebKitUIProcessInspectorWebInspectorProxyh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/UIProcess/Inspector/WebInspectorProxy.h (269523 => 269524)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/UIProcess/Inspector/WebInspectorProxy.h      2020-11-06 18:42:14 UTC (rev 269523)
+++ trunk/Source/WebKit/UIProcess/Inspector/WebInspectorProxy.h 2020-11-06 18:44:09 UTC (rev 269524)
</span><span class="lines">@@ -37,7 +37,6 @@
</span><span class="cx"> #include <WebCore/InspectorFrontendClient.h>
</span><span class="cx"> #include <wtf/Forward.h>
</span><span class="cx"> #include <wtf/RefPtr.h>
</span><del>-#include <wtf/WeakPtr.h>
</del><span class="cx"> #include <wtf/text/WTFString.h>
</span><span class="cx"> 
</span><span class="cx"> #if PLATFORM(MAC)
</span><span class="lines">@@ -70,9 +69,6 @@
</span><span class="cx"> class WebInspectorProxyClient;
</span><span class="cx"> class WebPageProxy;
</span><span class="cx"> class WebPreferences;
</span><del>-#if ENABLE(INSPECTOR_EXTENSIONS)
-class WebInspectorUIExtensionControllerProxy;
-#endif
</del><span class="cx"> 
</span><span class="cx"> enum class AttachmentSide {
</span><span class="cx">     Bottom,
</span><span class="lines">@@ -84,7 +80,6 @@
</span><span class="cx">     : public API::ObjectImpl<API::Object::Type::Inspector>
</span><span class="cx">     , public IPC::MessageReceiver
</span><span class="cx">     , public Inspector::FrontendChannel
</span><del>-    , public CanMakeWeakPtr<WebInspectorProxy>
</del><span class="cx"> #if PLATFORM(WIN)
</span><span class="cx">     , public WebCore::WindowMessageListener
</span><span class="cx"> #endif
</span><span class="lines">@@ -107,10 +102,6 @@
</span><span class="cx">     WebPageProxy* inspectedPage() const { return m_inspectedPage; }
</span><span class="cx">     WebPageProxy* inspectorPage() const { return m_inspectorPage; }
</span><span class="cx"> 
</span><del>-#if ENABLE(INSPECTOR_EXTENSIONS)
-    WebInspectorUIExtensionControllerProxy& extensionController() const { return *m_extensionController; }
-#endif
-
</del><span class="cx">     bool isConnected() const { return !!m_inspectorPage; }
</span><span class="cx">     bool isVisible() const { return m_isVisible; }
</span><span class="cx">     bool isFront();
</span><span class="lines">@@ -295,10 +286,6 @@
</span><span class="cx">     WebPageProxy* m_inspectorPage { nullptr };
</span><span class="cx">     std::unique_ptr<API::InspectorClient> m_inspectorClient;
</span><span class="cx"> 
</span><del>-#if ENABLE(INSPECTOR_EXTENSIONS)
-    std::unique_ptr<WebInspectorUIExtensionControllerProxy> m_extensionController;
-#endif
-    
</del><span class="cx">     bool m_underTest { false };
</span><span class="cx">     bool m_isVisible { false };
</span><span class="cx">     bool m_isAttached { false };
</span></span></pre></div>
<a id="trunkSourceWebKitUIProcessInspectorWebInspectorUIExtensionControllerProxycpp"></a>
<div class="delfile"><h4>Deleted: trunk/Source/WebKit/UIProcess/Inspector/WebInspectorUIExtensionControllerProxy.cpp (269523 => 269524)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/UIProcess/Inspector/WebInspectorUIExtensionControllerProxy.cpp       2020-11-06 18:42:14 UTC (rev 269523)
+++ trunk/Source/WebKit/UIProcess/Inspector/WebInspectorUIExtensionControllerProxy.cpp  2020-11-06 18:44:09 UTC (rev 269524)
</span><span class="lines">@@ -1,103 +0,0 @@
</span><del>-/*
- * Copyright (C) 2020 Apple 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:
- * 1. Redistributions of source code must retain the above copyright
- *    notice, this list of conditions and the following disclaimer.
- * 2. Redistributions in binary form must reproduce the above copyright
- *    notice, this list of conditions and the following disclaimer in the
- *    documentation and/or other materials provided with the distribution.
- *
- * THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS''
- * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO,
- * THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
- * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR ITS CONTRIBUTORS
- * BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
- * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
- * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
- * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
- * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
- * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF
- * THE POSSIBILITY OF SUCH DAMAGE.
- */
-
-#include "config.h"
-#include "WebInspectorUIExtensionControllerProxy.h"
-
-#if ENABLE(INSPECTOR_EXTENSIONS)
-
-#include "WebInspectorUIExtensionControllerMessages.h"
-#include "WebPageProxy.h"
-#include "WebProcessProxy.h"
-
-namespace WebKit {
-
-WebInspectorUIExtensionControllerProxy::WebInspectorUIExtensionControllerProxy(WebPageProxy& inspectorPage)
-    : m_inspectorPage(makeWeakPtr(inspectorPage))
-{
-}
-
-WebInspectorUIExtensionControllerProxy::~WebInspectorUIExtensionControllerProxy()
-{
-    if (!m_inspectorPage)
-        return;
-
-    m_inspectorPage = nullptr;
-
-    auto callbacks = std::exchange(m_frontendLoadedCallbackQueue, { });
-    for (auto& callback : callbacks)
-        callback();
-}
-
-void WebInspectorUIExtensionControllerProxy::whenFrontendHasLoaded(Function<void()>&& callback)
-{
-    if (m_frontendLoaded && m_inspectorPage) {
-        callback();
-        return;
-    }
-
-    m_frontendLoadedCallbackQueue.append(WTFMove(callback));
-}
-
-void WebInspectorUIExtensionControllerProxy::inspectorFrontendLoaded()
-{
-    ASSERT(m_inspectorPage);
-
-    m_frontendLoaded = true;
-
-    auto callbacks = std::exchange(m_frontendLoadedCallbackQueue, { });
-    for (auto& callback : callbacks)
-        callback();
-}
-
-// API
-
-void WebInspectorUIExtensionControllerProxy::registerExtension(const InspectorExtensionID& extensionID, const String& displayName, WTF::CompletionHandler<void(Expected<bool, InspectorExtensionError>)>&& completionHandler)
-{
-    whenFrontendHasLoaded([weakThis = makeWeakPtr(this), extensionID, displayName, completionHandler = WTFMove(completionHandler)] () mutable {
-        if (!weakThis || !weakThis->m_inspectorPage) {
-            completionHandler(makeUnexpected(InspectorExtensionError::InvalidRequest));
-            return;
-        }
-
-        weakThis->m_inspectorPage->sendWithAsyncReply(Messages::WebInspectorUIExtensionController::RegisterExtension { extensionID, displayName }, WTFMove(completionHandler));
-    });
-}
-
-void WebInspectorUIExtensionControllerProxy::unregisterExtension(const InspectorExtensionID& extensionID, WTF::CompletionHandler<void(Expected<bool, InspectorExtensionError>)>&& completionHandler)
-{
-    whenFrontendHasLoaded([weakThis = makeWeakPtr(this), extensionID, completionHandler = WTFMove(completionHandler)] () mutable {
-        if (!weakThis || !weakThis->m_inspectorPage) {
-            completionHandler(makeUnexpected(InspectorExtensionError::InvalidRequest));
-            return;
-        }
-
-        weakThis->m_inspectorPage->sendWithAsyncReply(Messages::WebInspectorUIExtensionController::UnregisterExtension { extensionID }, WTFMove(completionHandler));
-    });
-}
-
-} // namespace WebKit
-
-#endif // ENABLE(INSPECTOR_EXTENSIONS)
</del></span></pre></div>
<a id="trunkSourceWebKitUIProcessInspectorWebInspectorUIExtensionControllerProxyh"></a>
<div class="delfile"><h4>Deleted: trunk/Source/WebKit/UIProcess/Inspector/WebInspectorUIExtensionControllerProxy.h (269523 => 269524)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/UIProcess/Inspector/WebInspectorUIExtensionControllerProxy.h 2020-11-06 18:42:14 UTC (rev 269523)
+++ trunk/Source/WebKit/UIProcess/Inspector/WebInspectorUIExtensionControllerProxy.h    2020-11-06 18:44:09 UTC (rev 269524)
</span><span class="lines">@@ -1,70 +0,0 @@
</span><del>-/*
- * Copyright (C) 2020 Apple 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:
- * 1. Redistributions of source code must retain the above copyright
- *    notice, this list of conditions and the following disclaimer.
- * 2. Redistributions in binary form must reproduce the above copyright
- *    notice, this list of conditions and the following disclaimer in the
- *    documentation and/or other materials provided with the distribution.
- *
- * THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS''
- * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO,
- * THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
- * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR ITS CONTRIBUTORS
- * BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
- * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
- * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
- * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
- * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
- * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF
- * THE POSSIBILITY OF SUCH DAMAGE.
- */
-
-#pragma once
-
-#if ENABLE(INSPECTOR_EXTENSIONS)
-
-#include "InspectorExtensionTypes.h"
-#include "MessageReceiver.h"
-#include <wtf/Expected.h>
-#include <wtf/Forward.h>
-#include <wtf/WeakPtr.h>
-
-namespace WebKit {
-
-class WebPageProxy;
-
-class WebInspectorUIExtensionControllerProxy final
-    : public IPC::MessageReceiver
-    , public CanMakeWeakPtr<WebInspectorUIExtensionControllerProxy> {
-    WTF_MAKE_FAST_ALLOCATED;
-    WTF_MAKE_NONCOPYABLE(WebInspectorUIExtensionControllerProxy);
-public:
-    explicit WebInspectorUIExtensionControllerProxy(WebPageProxy& inspectorPage);
-    virtual ~WebInspectorUIExtensionControllerProxy();
-
-    // API.
-    void registerExtension(const InspectorExtensionID&, const String& displayName, WTF::CompletionHandler<void(Expected<bool, InspectorExtensionError>)>&&);
-    void unregisterExtension(const InspectorExtensionID&, WTF::CompletionHandler<void(Expected<bool, InspectorExtensionError>)>&&);
-
-    // Notifications.
-    void inspectorFrontendLoaded();
-
-private:
-    void whenFrontendHasLoaded(Function<void()>&&);
-
-    WeakPtr<WebPageProxy> m_inspectorPage;
-
-    // Used to queue actions such as registering extensions that happen early on.
-    // There's no point sending these before the frontend is fully loaded.
-    Vector<Function<void()>> m_frontendLoadedCallbackQueue;
-
-    bool m_frontendLoaded { false };
-};
-
-} // namespace WebKit
-
-#endif // ENABLE(INSPECTOR_EXTENSIONS)
</del></span></pre></div>
<a id="trunkSourceWebKitUIProcessWebAuthenticationCocoaLocalAuthenticatormm"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/UIProcess/WebAuthentication/Cocoa/LocalAuthenticator.mm (269523 => 269524)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/UIProcess/WebAuthentication/Cocoa/LocalAuthenticator.mm      2020-11-06 18:42:14 UTC (rev 269523)
+++ trunk/Source/WebKit/UIProcess/WebAuthentication/Cocoa/LocalAuthenticator.mm 2020-11-06 18:44:09 UTC (rev 269524)
</span><span class="lines">@@ -148,7 +148,7 @@
</span><span class="cx">         }
</span><span class="cx">         auto& responseMap = decodedResponse->getMap();
</span><span class="cx"> 
</span><del>-        auto it = responseMap.find(CBOR(fido::kEntityIdMapKey));
</del><ins>+        auto it = responseMap.find(CBOR(kEntityIdMapKey));
</ins><span class="cx">         if (it == responseMap.end() || !it->second.isByteString()) {
</span><span class="cx">             ASSERT_NOT_REACHED();
</span><span class="cx">             return WTF::nullopt;
</span><span class="lines">@@ -155,7 +155,7 @@
</span><span class="cx">         }
</span><span class="cx">         auto& userHandle = it->second.getByteString();
</span><span class="cx"> 
</span><del>-        it = responseMap.find(CBOR(fido::kEntityNameMapKey));
</del><ins>+        it = responseMap.find(CBOR(kEntityNameMapKey));
</ins><span class="cx">         if (it == responseMap.end() || !it->second.isString()) {
</span><span class="cx">             ASSERT_NOT_REACHED();
</span><span class="cx">             return WTF::nullopt;
</span><span class="lines">@@ -299,8 +299,8 @@
</span><span class="cx">     const auto& secAttrLabel = creationOptions.rp.id;
</span><span class="cx"> 
</span><span class="cx">     cbor::CBORValue::MapValue userEntityMap;
</span><del>-    userEntityMap[cbor::CBORValue(fido::kEntityIdMapKey)] = cbor::CBORValue(creationOptions.user.idVector);
-    userEntityMap[cbor::CBORValue(fido::kEntityNameMapKey)] = cbor::CBORValue(creationOptions.user.name);
</del><ins>+    userEntityMap[cbor::CBORValue(kEntityIdMapKey)] = cbor::CBORValue(creationOptions.user.idVector);
+    userEntityMap[cbor::CBORValue(kEntityNameMapKey)] = cbor::CBORValue(creationOptions.user.name);
</ins><span class="cx">     auto userEntity = cbor::CBORWriter::write(cbor::CBORValue(WTFMove(userEntityMap)));
</span><span class="cx">     ASSERT(userEntity);
</span><span class="cx">     auto secAttrApplicationTag = toNSData(*userEntity);
</span></span></pre></div>
<a id="trunkSourceWebKitWebKitxcodeprojprojectpbxproj"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/WebKit.xcodeproj/project.pbxproj (269523 => 269524)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/WebKit.xcodeproj/project.pbxproj     2020-11-06 18:42:14 UTC (rev 269523)
+++ trunk/Source/WebKit/WebKit.xcodeproj/project.pbxproj        2020-11-06 18:44:09 UTC (rev 269524)
</span><span class="lines">@@ -1437,15 +1437,8 @@
</span><span class="cx">          9955A6F61C7986E300EB6A93 /* AutomationProtocolObjects.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 9955A6F21C79866400EB6A93 /* AutomationProtocolObjects.cpp */; };
</span><span class="cx">          9955A6F71C7986E500EB6A93 /* AutomationProtocolObjects.h in Headers */ = {isa = PBXBuildFile; fileRef = 9955A6F31C79866400EB6A93 /* AutomationProtocolObjects.h */; };
</span><span class="cx">          99788ACB1F421DDA00C08000 /* _WKAutomationSessionConfiguration.h in Headers */ = {isa = PBXBuildFile; fileRef = 99788AC91F421DCA00C08000 /* _WKAutomationSessionConfiguration.h */; settings = {ATTRIBUTES = (Private, ); }; };
</span><del>-               9979659E25310A4900B31AE3 /* WebInspectorUIExtensionControllerMessages.h in Headers */ = {isa = PBXBuildFile; fileRef = 9979659A25310A4800B31AE3 /* WebInspectorUIExtensionControllerMessages.h */; };
-               9979659F25310A4900B31AE3 /* WebInspectorUIExtensionControllerMessageReceiver.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 9979659B25310A4800B31AE3 /* WebInspectorUIExtensionControllerMessageReceiver.cpp */; };
-               997965A3253128C700B31AE3 /* _WKInspectorExtensionHost.h in Headers */ = {isa = PBXBuildFile; fileRef = 997965A2253128C700B31AE3 /* _WKInspectorExtensionHost.h */; settings = {ATTRIBUTES = (Private, ); }; };
</del><span class="cx">           9979CA58237F49F10039EC05 /* _WKInspectorPrivate.h in Headers */ = {isa = PBXBuildFile; fileRef = 9979CA57237F49F00039EC05 /* _WKInspectorPrivate.h */; settings = {ATTRIBUTES = (Private, ); }; };
</span><span class="cx">          99996A9F25004BCC004F7559 /* _WKInspectorPrivateForTesting.h in Headers */ = {isa = PBXBuildFile; fileRef = 99996A9D25004BCB004F7559 /* _WKInspectorPrivateForTesting.h */; settings = {ATTRIBUTES = (Private, ); }; };
</span><del>-               99B16757252BB7E10073140E /* _WKInspectorExtension.h in Headers */ = {isa = PBXBuildFile; fileRef = 99B16754252BB7E00073140E /* _WKInspectorExtension.h */; settings = {ATTRIBUTES = (Private, ); }; };
-               99B16758252BB7E10073140E /* _WKInspectorExtensionInternal.h in Headers */ = {isa = PBXBuildFile; fileRef = 99B16755252BB7E10073140E /* _WKInspectorExtensionInternal.h */; };
-               99B1675B252BBADD0073140E /* WebInspectorUIExtensionControllerProxy.h in Headers */ = {isa = PBXBuildFile; fileRef = 99B1675A252BBADD0073140E /* WebInspectorUIExtensionControllerProxy.h */; };
-               99B16764252BBE620073140E /* WebInspectorUIExtensionController.h in Headers */ = {isa = PBXBuildFile; fileRef = 99B16761252BBE610073140E /* WebInspectorUIExtensionController.h */; };
</del><span class="cx">           99C3AE2D1DADA6AD00AF5C16 /* WebAutomationSessionMacros.h in Headers */ = {isa = PBXBuildFile; fileRef = 99C3AE2C1DADA6A700AF5C16 /* WebAutomationSessionMacros.h */; };
</span><span class="cx">          99C81D5A1C20E7E2005C4C82 /* AutomationClient.h in Headers */ = {isa = PBXBuildFile; fileRef = 99C81D551C20DFBE005C4C82 /* AutomationClient.h */; };
</span><span class="cx">          99C81D5D1C21F38B005C4C82 /* APIAutomationClient.h in Headers */ = {isa = PBXBuildFile; fileRef = 99C81D5B1C20E817005C4C82 /* APIAutomationClient.h */; };
</span><span class="lines">@@ -4583,23 +4576,9 @@
</span><span class="cx">          9955A6F31C79866400EB6A93 /* AutomationProtocolObjects.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; name = AutomationProtocolObjects.h; path = DerivedSources/WebKit2/AutomationProtocolObjects.h; sourceTree = BUILT_PRODUCTS_DIR; };
</span><span class="cx">          99788AC91F421DCA00C08000 /* _WKAutomationSessionConfiguration.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = _WKAutomationSessionConfiguration.h; sourceTree = "<group>"; };
</span><span class="cx">          99788ACA1F421DCA00C08000 /* _WKAutomationSessionConfiguration.mm */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.objcpp; path = _WKAutomationSessionConfiguration.mm; sourceTree = "<group>"; };
</span><del>-               9979659A25310A4800B31AE3 /* WebInspectorUIExtensionControllerMessages.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; name = WebInspectorUIExtensionControllerMessages.h; path = DerivedSources/WebKit2/WebInspectorUIExtensionControllerMessages.h; sourceTree = BUILT_PRODUCTS_DIR; };
-               9979659B25310A4800B31AE3 /* WebInspectorUIExtensionControllerMessageReceiver.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; name = WebInspectorUIExtensionControllerMessageReceiver.cpp; path = DerivedSources/WebKit2/WebInspectorUIExtensionControllerMessageReceiver.cpp; sourceTree = BUILT_PRODUCTS_DIR; };
-               997965A2253128C700B31AE3 /* _WKInspectorExtensionHost.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = _WKInspectorExtensionHost.h; sourceTree = "<group>"; };
</del><span class="cx">           9979CA57237F49F00039EC05 /* _WKInspectorPrivate.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = _WKInspectorPrivate.h; sourceTree = "<group>"; };
</span><span class="cx">          99996A9D25004BCB004F7559 /* _WKInspectorPrivateForTesting.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = _WKInspectorPrivateForTesting.h; sourceTree = "<group>"; };
</span><span class="cx">          99996A9E25004BCB004F7559 /* _WKInspectorTesting.mm */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.objcpp; path = _WKInspectorTesting.mm; sourceTree = "<group>"; };
</span><del>-               999B7ED82550E4A800F450A4 /* InspectorExtensionTypes.cpp */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.cpp.cpp; path = InspectorExtensionTypes.cpp; sourceTree = "<group>"; };
-               99B16754252BB7E00073140E /* _WKInspectorExtension.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = _WKInspectorExtension.h; sourceTree = "<group>"; };
-               99B16755252BB7E10073140E /* _WKInspectorExtensionInternal.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = _WKInspectorExtensionInternal.h; sourceTree = "<group>"; };
-               99B16756252BB7E10073140E /* _WKInspectorExtension.mm */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.objcpp; path = _WKInspectorExtension.mm; sourceTree = "<group>"; };
-               99B1675A252BBADD0073140E /* WebInspectorUIExtensionControllerProxy.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = WebInspectorUIExtensionControllerProxy.h; sourceTree = "<group>"; };
-               99B1675D252BBE0A0073140E /* WebInspectorUIExtensionControllerProxy.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = WebInspectorUIExtensionControllerProxy.cpp; sourceTree = "<group>"; };
-               99B16760252BBE610073140E /* WebInspectorUIExtensionController.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = WebInspectorUIExtensionController.cpp; sourceTree = "<group>"; };
-               99B16761252BBE610073140E /* WebInspectorUIExtensionController.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = WebInspectorUIExtensionController.h; sourceTree = "<group>"; };
-               99B16762252BBE610073140E /* WebInspectorUIExtensionController.messages.in */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = text; path = WebInspectorUIExtensionController.messages.in; sourceTree = "<group>"; };
-               99BE3B1225422F4100C6551C /* APIInspectorExtension.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = APIInspectorExtension.h; sourceTree = "<group>"; };
-               99BE3B1625433B9400C6551C /* InspectorExtensionTypes.h */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.c.h; path = InspectorExtensionTypes.h; sourceTree = "<group>"; };
</del><span class="cx">           99C3AE231DAD8E3400AF5C16 /* WebAutomationSessionMac.mm */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.objcpp; path = WebAutomationSessionMac.mm; sourceTree = "<group>"; };
</span><span class="cx">          99C3AE261DAD948500AF5C16 /* WebAutomationSessionCocoa.mm */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.objcpp; path = WebAutomationSessionCocoa.mm; sourceTree = "<group>"; };
</span><span class="cx">          99C3AE2C1DADA6A700AF5C16 /* WebAutomationSessionMacros.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = WebAutomationSessionMacros.h; sourceTree = "<group>"; };
</span><span class="lines">@@ -6173,8 +6152,6 @@
</span><span class="cx">                          5C121E8324101F7000486F9B /* FrameTreeNodeData.h */,
</span><span class="cx">                          1AC75A1A1B3368270056745B /* HangDetectionDisabler.h */,
</span><span class="cx">                          BCCF6B2312C93E7A008F9C35 /* ImageOptions.h */,
</span><del>-                               999B7ED82550E4A800F450A4 /* InspectorExtensionTypes.cpp */,
-                               99BE3B1625433B9400C6551C /* InspectorExtensionTypes.h */,
</del><span class="cx">                           1A92DC1212F8BAB90017AF65 /* LayerTreeContext.cpp */,
</span><span class="cx">                          1A92DC1012F8BA460017AF65 /* LayerTreeContext.h */,
</span><span class="cx">                          2D10875E1D2C573E00B85F82 /* LoadParameters.cpp */,
</span><span class="lines">@@ -7393,10 +7370,6 @@
</span><span class="cx">                          991F492E23A812C60054642B /* _WKInspectorDebuggableInfo.mm */,
</span><span class="cx">                          99036AE123A949CE0000B06A /* _WKInspectorDebuggableInfoInternal.h */,
</span><span class="cx">                          9197940B23DBC50300257892 /* _WKInspectorDelegate.h */,
</span><del>-                               99B16754252BB7E00073140E /* _WKInspectorExtension.h */,
-                               99B16756252BB7E10073140E /* _WKInspectorExtension.mm */,
-                               997965A2253128C700B31AE3 /* _WKInspectorExtensionHost.h */,
-                               99B16755252BB7E10073140E /* _WKInspectorExtensionInternal.h */,
</del><span class="cx">                           5CAFDE442130843600B1F7E1 /* _WKInspectorInternal.h */,
</span><span class="cx">                          9979CA57237F49F00039EC05 /* _WKInspectorPrivate.h */,
</span><span class="cx">                          99996A9D25004BCB004F7559 /* _WKInspectorPrivateForTesting.h */,
</span><span class="lines">@@ -8706,8 +8679,6 @@
</span><span class="cx">                          1C8E28331275D73800BC7BD0 /* WebInspectorProxy.cpp */,
</span><span class="cx">                          1C8E28321275D73800BC7BD0 /* WebInspectorProxy.h */,
</span><span class="cx">                          1C77C1951288A872006A742F /* WebInspectorProxy.messages.in */,
</span><del>-                               99B1675D252BBE0A0073140E /* WebInspectorUIExtensionControllerProxy.cpp */,
-                               99B1675A252BBADD0073140E /* WebInspectorUIExtensionControllerProxy.h */,
</del><span class="cx">                           A55BA8271BA38E1E007CD33D /* WebInspectorUtilities.cpp */,
</span><span class="cx">                          A55BA8281BA38E1E007CD33D /* WebInspectorUtilities.h */,
</span><span class="cx">                          A5EC6AD22151BD6900677D17 /* WebPageDebuggable.cpp */,
</span><span class="lines">@@ -8770,9 +8741,6 @@
</span><span class="cx">                          1C891D6219B124FF00BA79DD /* WebInspectorUI.cpp */,
</span><span class="cx">                          1C891D6319B124FF00BA79DD /* WebInspectorUI.h */,
</span><span class="cx">                          1C891D6419B124FF00BA79DD /* WebInspectorUI.messages.in */,
</span><del>-                               99B16760252BBE610073140E /* WebInspectorUIExtensionController.cpp */,
-                               99B16761252BBE610073140E /* WebInspectorUIExtensionController.h */,
-                               99B16762252BBE610073140E /* WebInspectorUIExtensionController.messages.in */,
</del><span class="cx">                           A543E30A215C8A8400279CD9 /* WebPageInspectorTarget.cpp */,
</span><span class="cx">                          A543E308215C8A8300279CD9 /* WebPageInspectorTarget.h */,
</span><span class="cx">                          A543E309215C8A8300279CD9 /* WebPageInspectorTargetController.cpp */,
</span><span class="lines">@@ -9398,7 +9366,6 @@
</span><span class="cx">                          9197940923DBC4E000257892 /* APIInspectorClient.h */,
</span><span class="cx">                          994C6048253F820200BDF060 /* APIInspectorConfiguration.cpp */,
</span><span class="cx">                          994C6046253CACB800BDF060 /* APIInspectorConfiguration.h */,
</span><del>-                               99BE3B1225422F4100C6551C /* APIInspectorExtension.h */,
</del><span class="cx">                           31B362902141EABC007BFA53 /* APIInternalDebugFeature.cpp */,
</span><span class="cx">                          31B3628E2141EA4D007BFA53 /* APIInternalDebugFeature.h */,
</span><span class="cx">                          7CE4D2061A46775700C7F152 /* APILegacyContextHistoryClient.h */,
</span><span class="lines">@@ -10429,8 +10396,6 @@
</span><span class="cx">                          1C8E2A321277852400BC7BD0 /* WebInspectorMessages.h */,
</span><span class="cx">                          1CA8B943127C882A00576C2B /* WebInspectorProxyMessageReceiver.cpp */,
</span><span class="cx">                          1CA8B944127C882A00576C2B /* WebInspectorProxyMessages.h */,
</span><del>-                               9979659B25310A4800B31AE3 /* WebInspectorUIExtensionControllerMessageReceiver.cpp */,
-                               9979659A25310A4800B31AE3 /* WebInspectorUIExtensionControllerMessages.h */,
</del><span class="cx">                           1CBBE49E19B66C53006B7D81 /* WebInspectorUIMessageReceiver.cpp */,
</span><span class="cx">                          1CBBE49F19B66C53006B7D81 /* WebInspectorUIMessages.h */,
</span><span class="cx">                          51F060DD1654317500F3282F /* WebMDNSRegisterMessageReceiver.cpp */,
</span><span class="lines">@@ -10846,9 +10811,6 @@
</span><span class="cx">                          991F492F23A812C60054642B /* _WKInspectorDebuggableInfo.h in Headers */,
</span><span class="cx">                          99036AE223A949CF0000B06A /* _WKInspectorDebuggableInfoInternal.h in Headers */,
</span><span class="cx">                          9197940C23DBC50300257892 /* _WKInspectorDelegate.h in Headers */,
</span><del>-                               99B16757252BB7E10073140E /* _WKInspectorExtension.h in Headers */,
-                               997965A3253128C700B31AE3 /* _WKInspectorExtensionHost.h in Headers */,
-                               99B16758252BB7E10073140E /* _WKInspectorExtensionInternal.h in Headers */,
</del><span class="cx">                           5CAFDE472130846A00B1F7E1 /* _WKInspectorInternal.h in Headers */,
</span><span class="cx">                          9979CA58237F49F10039EC05 /* _WKInspectorPrivate.h in Headers */,
</span><span class="cx">                          99996A9F25004BCC004F7559 /* _WKInspectorPrivateForTesting.h in Headers */,
</span><span class="lines">@@ -11586,9 +11548,6 @@
</span><span class="cx">                          1C8E28341275D73800BC7BD0 /* WebInspectorProxy.h in Headers */,
</span><span class="cx">                          1CA8B946127C882A00576C2B /* WebInspectorProxyMessages.h in Headers */,
</span><span class="cx">                          1C891D6619B124FF00BA79DD /* WebInspectorUI.h in Headers */,
</span><del>-                               99B16764252BBE620073140E /* WebInspectorUIExtensionController.h in Headers */,
-                               9979659E25310A4900B31AE3 /* WebInspectorUIExtensionControllerMessages.h in Headers */,
-                               99B1675B252BBADD0073140E /* WebInspectorUIExtensionControllerProxy.h in Headers */,
</del><span class="cx">                           1CBBE4A119B66C53006B7D81 /* WebInspectorUIMessages.h in Headers */,
</span><span class="cx">                          A55BA82B1BA38E61007CD33D /* WebInspectorUtilities.h in Headers */,
</span><span class="cx">                          2DA944A01884E4F000ED86DB /* WebIOSEventFactory.h in Headers */,
</span><span class="lines">@@ -13408,7 +13367,6 @@
</span><span class="cx">                          FEDBDCD61E68D20000A59F8F /* WebInspectorInterruptDispatcherMessageReceiver.cpp in Sources */,
</span><span class="cx">                          1C8E2A351277852400BC7BD0 /* WebInspectorMessageReceiver.cpp in Sources */,
</span><span class="cx">                          1CA8B945127C882A00576C2B /* WebInspectorProxyMessageReceiver.cpp in Sources */,
</span><del>-                               9979659F25310A4900B31AE3 /* WebInspectorUIExtensionControllerMessageReceiver.cpp in Sources */,
</del><span class="cx">                           1CBBE4A019B66C53006B7D81 /* WebInspectorUIMessageReceiver.cpp in Sources */,
</span><span class="cx">                          2D92A78A212B6AB100F493FD /* WebKeyboardEvent.cpp in Sources */,
</span><span class="cx">                          1A6280F31919982A006AD9F9 /* WebKit.m in Sources */,
</span></span></pre></div>
<a id="trunkSourceWebKitWebProcessInspectorRemoteWebInspectorUIcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/WebProcess/Inspector/RemoteWebInspectorUI.cpp (269523 => 269524)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/WebProcess/Inspector/RemoteWebInspectorUI.cpp        2020-11-06 18:42:14 UTC (rev 269523)
+++ trunk/Source/WebKit/WebProcess/Inspector/RemoteWebInspectorUI.cpp   2020-11-06 18:44:09 UTC (rev 269524)
</span><span class="lines">@@ -39,10 +39,6 @@
</span><span class="cx"> #include <WebCore/InspectorController.h>
</span><span class="cx"> #include <WebCore/Settings.h>
</span><span class="cx"> 
</span><del>-#if ENABLE(INSPECTOR_EXTENSIONS)
-#include "WebInspectorUIExtensionController.h"
-#endif
-
</del><span class="cx"> #if !PLATFORM(MAC) && !PLATFORM(GTK) && !PLATFORM(WIN)
</span><span class="cx"> #include <WebCore/NotImplemented.h>
</span><span class="cx"> #endif
</span><span class="lines">@@ -62,14 +58,8 @@
</span><span class="cx">     WebInspectorUI::enableFrontendFeatures();
</span><span class="cx"> }
</span><span class="cx"> 
</span><del>-RemoteWebInspectorUI::~RemoteWebInspectorUI() = default;
-
</del><span class="cx"> void RemoteWebInspectorUI::initialize(DebuggableInfoData&& debuggableInfo, const String& backendCommandsURL)
</span><span class="cx"> {
</span><del>-#if ENABLE(INSPECTOR_EXTENSIONS)
-    m_extensionController = makeUnique<WebInspectorUIExtensionController>(*this);
-#endif
-
</del><span class="cx">     m_debuggableInfo = WTFMove(debuggableInfo);
</span><span class="cx">     m_backendCommandsURL = backendCommandsURL;
</span><span class="cx"> 
</span><span class="lines">@@ -119,8 +109,6 @@
</span><span class="cx"> 
</span><span class="cx">     m_frontendAPIDispatcher->dispatchCommandWithResultAsync("setIsVisible"_s, { JSON::Value::create(true) });
</span><span class="cx"> 
</span><del>-    WebProcess::singleton().parentProcessConnection()->send(Messages::RemoteWebInspectorProxy::FrontendLoaded(), m_page.identifier());
-
</del><span class="cx">     bringToFront();
</span><span class="cx"> }
</span><span class="cx"> 
</span><span class="lines">@@ -186,10 +174,6 @@
</span><span class="cx"> {
</span><span class="cx">     m_page.corePage()->inspectorController().setInspectorFrontendClient(nullptr);
</span><span class="cx"> 
</span><del>-#if ENABLE(INSPECTOR_EXTENSIONS)
-    m_extensionController = nullptr;
-#endif
-    
</del><span class="cx">     WebProcess::singleton().parentProcessConnection()->send(Messages::RemoteWebInspectorProxy::FrontendDidClose(), m_page.identifier());
</span><span class="cx"> }
</span><span class="cx"> 
</span><span class="lines">@@ -274,12 +258,6 @@
</span><span class="cx"> }
</span><span class="cx"> #endif
</span><span class="cx"> 
</span><del>-WebCore::Page* RemoteWebInspectorUI::frontendPage()
-{
-    return m_page.corePage();
-}
-
-
</del><span class="cx"> #if !PLATFORM(MAC) && !PLATFORM(GTK) && !PLATFORM(WIN)
</span><span class="cx"> String RemoteWebInspectorUI::localizedStringsURL() const
</span><span class="cx"> {
</span></span></pre></div>
<a id="trunkSourceWebKitWebProcessInspectorRemoteWebInspectorUIh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/WebProcess/Inspector/RemoteWebInspectorUI.h (269523 => 269524)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/WebProcess/Inspector/RemoteWebInspectorUI.h  2020-11-06 18:42:14 UTC (rev 269523)
+++ trunk/Source/WebKit/WebProcess/Inspector/RemoteWebInspectorUI.h     2020-11-06 18:44:09 UTC (rev 269524)
</span><span class="lines">@@ -39,16 +39,11 @@
</span><span class="cx"> 
</span><span class="cx"> namespace WebKit {
</span><span class="cx"> 
</span><del>-class WebInspectorUIExtensionController;
</del><span class="cx"> class WebPage;
</span><span class="cx"> 
</span><del>-class RemoteWebInspectorUI final
-    : public RefCounted<RemoteWebInspectorUI>
-    , public IPC::MessageReceiver
-    , public WebCore::InspectorFrontendClient {
</del><ins>+class RemoteWebInspectorUI final : public RefCounted<RemoteWebInspectorUI>, public IPC::MessageReceiver, public WebCore::InspectorFrontendClient {
</ins><span class="cx"> public:
</span><span class="cx">     static Ref<RemoteWebInspectorUI> create(WebPage&);
</span><del>-    ~RemoteWebInspectorUI();
</del><span class="cx"> 
</span><span class="cx">     // Implemented in generated RemoteWebInspectorUIMessageReceiver.cpp
</span><span class="cx">     void didReceiveMessage(IPC::Connection&, IPC::Decoder&) override;
</span><span class="lines">@@ -102,7 +97,6 @@
</span><span class="cx">     void showCertificate(const WebCore::CertificateInfo&) override;
</span><span class="cx">     void sendMessageToBackend(const String&) override;
</span><span class="cx">     WebCore::InspectorFrontendAPIDispatcher& frontendAPIDispatcher() override { return m_frontendAPIDispatcher; }
</span><del>-    WebCore::Page* frontendPage() final;
</del><span class="cx"> 
</span><span class="cx"> #if ENABLE(INSPECTOR_TELEMETRY)
</span><span class="cx">     bool supportsDiagnosticLogging() override;
</span><span class="lines">@@ -123,10 +117,6 @@
</span><span class="cx">     WebPage& m_page;
</span><span class="cx">     Ref<WebCore::InspectorFrontendAPIDispatcher> m_frontendAPIDispatcher;
</span><span class="cx">     RefPtr<WebCore::InspectorFrontendHost> m_frontendHost;
</span><del>-#if ENABLE(INSPECTOR_EXTENSIONS)
-    std::unique_ptr<WebInspectorUIExtensionController> m_extensionController;
-#endif
-
</del><span class="cx">     DebuggableInfoData m_debuggableInfo;
</span><span class="cx">     String m_backendCommandsURL;
</span><span class="cx"> 
</span></span></pre></div>
<a id="trunkSourceWebKitWebProcessInspectorWebInspectorUIcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/WebProcess/Inspector/WebInspectorUI.cpp (269523 => 269524)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/WebProcess/Inspector/WebInspectorUI.cpp      2020-11-06 18:42:14 UTC (rev 269523)
+++ trunk/Source/WebKit/WebProcess/Inspector/WebInspectorUI.cpp 2020-11-06 18:44:09 UTC (rev 269524)
</span><span class="lines">@@ -36,15 +36,10 @@
</span><span class="cx"> #include <WebCore/DOMWrapperWorld.h>
</span><span class="cx"> #include <WebCore/FloatRect.h>
</span><span class="cx"> #include <WebCore/InspectorController.h>
</span><del>-#include <WebCore/InspectorFrontendHost.h>
</del><span class="cx"> #include <WebCore/NotImplemented.h>
</span><span class="cx"> #include <WebCore/RuntimeEnabledFeatures.h>
</span><span class="cx"> #include <WebCore/Settings.h>
</span><span class="cx"> 
</span><del>-#if ENABLE(INSPECTOR_EXTENSIONS)
-#include "WebInspectorUIExtensionController.h"
-#endif
-
</del><span class="cx"> namespace WebKit {
</span><span class="cx"> using namespace WebCore;
</span><span class="cx"> 
</span><span class="lines">@@ -70,8 +65,6 @@
</span><span class="cx">     WebInspectorUI::enableFrontendFeatures();
</span><span class="cx"> }
</span><span class="cx"> 
</span><del>-WebInspectorUI::~WebInspectorUI() = default;
-
</del><span class="cx"> void WebInspectorUI::establishConnection(WebPageProxyIdentifier inspectedPageIdentifier, const DebuggableInfoData& debuggableInfo, bool underTest, unsigned inspectionLevel)
</span><span class="cx"> {
</span><span class="cx">     m_inspectedPageIdentifier = inspectedPageIdentifier;
</span><span class="lines">@@ -79,11 +72,8 @@
</span><span class="cx">     m_underTest = underTest;
</span><span class="cx">     m_inspectionLevel = inspectionLevel;
</span><span class="cx"> 
</span><del>-#if ENABLE(INSPECTOR_EXTENSIONS)
-    m_extensionController = makeUnique<WebInspectorUIExtensionController>(*this);
-#endif
</del><ins>+    m_frontendAPIDispatcher->reset();
</ins><span class="cx"> 
</span><del>-    m_frontendAPIDispatcher->reset();
</del><span class="cx">     m_frontendController = &m_page.corePage()->inspectorController();
</span><span class="cx">     m_frontendController->setInspectorFrontendClient(this);
</span><span class="cx"> 
</span><span class="lines">@@ -175,10 +165,6 @@
</span><span class="cx"> 
</span><span class="cx">     m_inspectedPageIdentifier = { };
</span><span class="cx">     m_underTest = false;
</span><del>-    
-#if ENABLE(INSPECTOR_EXTENSIONS)
-    m_extensionController = nullptr;
-#endif
</del><span class="cx"> }
</span><span class="cx"> 
</span><span class="cx"> void WebInspectorUI::reopen()
</span><span class="lines">@@ -426,12 +412,6 @@
</span><span class="cx">     return m_debuggableInfo.targetProductVersion;
</span><span class="cx"> }
</span><span class="cx"> 
</span><del>-WebCore::Page* WebInspectorUI::frontendPage()
-{
-    return m_page.corePage();
-}
-
-
</del><span class="cx"> #if !PLATFORM(MAC) && !PLATFORM(GTK) && !PLATFORM(WIN)
</span><span class="cx"> bool WebInspectorUI::canSave()
</span><span class="cx"> {
</span></span></pre></div>
<a id="trunkSourceWebKitWebProcessInspectorWebInspectorUIh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit/WebProcess/Inspector/WebInspectorUI.h (269523 => 269524)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/WebProcess/Inspector/WebInspectorUI.h        2020-11-06 18:42:14 UTC (rev 269523)
+++ trunk/Source/WebKit/WebProcess/Inspector/WebInspectorUI.h   2020-11-06 18:44:09 UTC (rev 269524)
</span><span class="lines">@@ -31,10 +31,10 @@
</span><span class="cx"> #include <WebCore/InspectorDebuggableType.h>
</span><span class="cx"> #include <WebCore/InspectorFrontendAPIDispatcher.h>
</span><span class="cx"> #include <WebCore/InspectorFrontendClient.h>
</span><ins>+#include <WebCore/InspectorFrontendHost.h>
</ins><span class="cx"> 
</span><span class="cx"> namespace WebCore {
</span><span class="cx"> class InspectorController;
</span><del>-class InspectorFrontendHost;
</del><span class="cx"> class CertificateInfo;
</span><span class="cx"> class FloatRect;
</span><span class="cx"> }
</span><span class="lines">@@ -42,17 +42,10 @@
</span><span class="cx"> namespace WebKit {
</span><span class="cx"> 
</span><span class="cx"> class WebPage;
</span><del>-#if ENABLE(INSPECTOR_EXTENSIONS)
-class WebInspectorUIExtensionController;
-#endif
</del><span class="cx"> 
</span><del>-class WebInspectorUI final
-    : public RefCounted<WebInspectorUI>
-    , private IPC::Connection::Client
-    , public WebCore::InspectorFrontendClient {
</del><ins>+class WebInspectorUI : public RefCounted<WebInspectorUI>, private IPC::Connection::Client, public WebCore::InspectorFrontendClient {
</ins><span class="cx"> public:
</span><span class="cx">     static Ref<WebInspectorUI> create(WebPage&);
</span><del>-    virtual ~WebInspectorUI();
</del><span class="cx"> 
</span><span class="cx">     static void enableFrontendFeatures();
</span><span class="cx"> 
</span><span class="lines">@@ -150,8 +143,7 @@
</span><span class="cx"> 
</span><span class="cx">     void sendMessageToBackend(const String&) override;
</span><span class="cx">     WebCore::InspectorFrontendAPIDispatcher& frontendAPIDispatcher() final { return m_frontendAPIDispatcher; }
</span><del>-    WebCore::Page* frontendPage() final;
-        
</del><ins>+
</ins><span class="cx">     void pagePaused() override;
</span><span class="cx">     void pageUnpaused() override;
</span><span class="cx"> 
</span><span class="lines">@@ -168,10 +160,6 @@
</span><span class="cx">     // corePage(), since we may need it after the frontend's page has started destruction.
</span><span class="cx">     WebCore::InspectorController* m_frontendController { nullptr };
</span><span class="cx"> 
</span><del>-#if ENABLE(INSPECTOR_EXTENSIONS)
-    std::unique_ptr<WebInspectorUIExtensionController> m_extensionController;
-#endif
-
</del><span class="cx">     WebPageProxyIdentifier m_inspectedPageIdentifier;
</span><span class="cx">     bool m_underTest { false };
</span><span class="cx">     DebuggableInfoData m_debuggableInfo;
</span></span></pre></div>
<a id="trunkSourceWebKitWebProcessInspectorWebInspectorUIExtensionControllercpp"></a>
<div class="delfile"><h4>Deleted: trunk/Source/WebKit/WebProcess/Inspector/WebInspectorUIExtensionController.cpp (269523 => 269524)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/WebProcess/Inspector/WebInspectorUIExtensionController.cpp   2020-11-06 18:42:14 UTC (rev 269523)
+++ trunk/Source/WebKit/WebProcess/Inspector/WebInspectorUIExtensionController.cpp      2020-11-06 18:44:09 UTC (rev 269524)
</span><span class="lines">@@ -1,134 +0,0 @@
</span><del>-/*
- * Copyright (C) 2020 Apple 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:
- * 1. Redistributions of source code must retain the above copyright
- *    notice, this list of conditions and the following disclaimer.
- * 2. Redistributions in binary form must reproduce the above copyright
- *    notice, this list of conditions and the following disclaimer in the
- *    documentation and/or other materials provided with the distribution.
- *
- * THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS''
- * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO,
- * THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
- * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR ITS CONTRIBUTORS
- * BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
- * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
- * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
- * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
- * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
- * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF
- * THE POSSIBILITY OF SUCH DAMAGE.
- */
-
-#include "config.h"
-#include "WebInspectorUIExtensionController.h"
-
-#if ENABLE(INSPECTOR_EXTENSIONS)
-
-#include "WebInspectorUI.h"
-#include "WebInspectorUIExtensionControllerMessages.h"
-#include "WebInspectorUIExtensionControllerMessagesReplies.h"
-#include "WebPage.h"
-#include "WebProcess.h"
-#include <WebCore/InspectorFrontendAPIDispatcher.h>
-
-namespace WebKit {
-
-WebInspectorUIExtensionController::WebInspectorUIExtensionController(WebCore::InspectorFrontendClient& inspectorFrontend)
-    : m_frontendClient(makeWeakPtr(inspectorFrontend))
-{
-    Page* page = inspectorFrontend.frontendPage();
-    ASSERT(page);
-
-    WebProcess::singleton().addMessageReceiver(Messages::WebInspectorUIExtensionController::messageReceiverName(), WebPage::fromCorePage(*page).identifier(), *this);
-}
-
-WebInspectorUIExtensionController::~WebInspectorUIExtensionController()
-{
-    WebProcess::singleton().removeMessageReceiver(*this);
-}
-
-Optional<InspectorExtensionError> WebInspectorUIExtensionController::parseInspectorExtensionErrorFromResult(JSC::JSValue result)
-{
-    ASSERT(m_frontendClient);
-    auto globalObject = m_frontendClient->frontendAPIDispatcher().frontendGlobalObject();
-    if (!globalObject)
-        return InspectorExtensionError::ContextDestroyed;
-
-    // If the evaluation result is a string, the frontend returned an error string.
-    // Anything else (falsy values, objects, arrays, DOM, etc.) is interpreted as success.
-    if (result.isString()) {
-        auto resultString = result.toWTFString(globalObject);
-        if (resultString == "ContextDestroyed"_s)
-            return InspectorExtensionError::ContextDestroyed;
-        if (resultString == "InternalError"_s)
-            return InspectorExtensionError::InternalError;
-        if (resultString == "InvalidRequest"_s)
-            return InspectorExtensionError::InvalidRequest;
-        if (resultString == "RegistrationFailed"_s)
-            return InspectorExtensionError::RegistrationFailed;
-
-        ASSERT_NOT_REACHED();
-        return InspectorExtensionError::InternalError;
-    }
-
-    return WTF::nullopt;
-}
-
-// WebInspectorUIExtensionController IPC messages.
-
-void WebInspectorUIExtensionController::registerExtension(const String& extensionID, const String& displayName, CompletionHandler<void(Expected<bool, InspectorExtensionError>)>&& completionHandler)
-{
-    if (!m_frontendClient) {
-        completionHandler(makeUnexpected(InspectorExtensionError::InvalidRequest));
-        return;
-    }
-
-    Vector<Ref<JSON::Value>> arguments {
-        JSON::Value::create(extensionID),
-        JSON::Value::create(displayName),
-    };
-    m_frontendClient->frontendAPIDispatcher().dispatchCommandWithResultAsync("registerExtension"_s, WTFMove(arguments), [weakThis = makeWeakPtr(this), completionHandler = WTFMove(completionHandler)](InspectorFrontendAPIDispatcher::EvaluationResult&& result) mutable {
-        if (!weakThis || !result) {
-            completionHandler(makeUnexpected(InspectorExtensionError::ContextDestroyed));
-            return;
-        }
-
-        if (auto parsedError = weakThis->parseInspectorExtensionErrorFromResult(result.value())) {
-            completionHandler(makeUnexpected(parsedError.value()));
-            return;
-        }
-
-        completionHandler(true);
-    });
-}
-
-void WebInspectorUIExtensionController::unregisterExtension(const String& extensionID, CompletionHandler<void(Expected<bool, InspectorExtensionError>)>&& completionHandler)
-{
-    if (!m_frontendClient) {
-        completionHandler(makeUnexpected(InspectorExtensionError::InvalidRequest));
-        return;
-    }
-
-    Vector<Ref<JSON::Value>> arguments { JSON::Value::create(extensionID) };
-    m_frontendClient->frontendAPIDispatcher().dispatchCommandWithResultAsync("unregisterExtension"_s, WTFMove(arguments), [weakThis = makeWeakPtr(this), completionHandler = WTFMove(completionHandler)](InspectorFrontendAPIDispatcher::EvaluationResult&& result) mutable {
-        if (!weakThis || !result) {
-            completionHandler(makeUnexpected(InspectorExtensionError::ContextDestroyed));
-            return;
-        }
-
-        if (auto parsedError = weakThis->parseInspectorExtensionErrorFromResult(result.value())) {
-            completionHandler(makeUnexpected(parsedError.value()));
-            return;
-        }
-
-        completionHandler(true);
-    });
-}
-
-} // namespace WebKit
-
-#endif // ENABLE(INSPECTOR_EXTENSIONS)
</del></span></pre></div>
<a id="trunkSourceWebKitWebProcessInspectorWebInspectorUIExtensionControllerh"></a>
<div class="delfile"><h4>Deleted: trunk/Source/WebKit/WebProcess/Inspector/WebInspectorUIExtensionController.h (269523 => 269524)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/WebProcess/Inspector/WebInspectorUIExtensionController.h     2020-11-06 18:42:14 UTC (rev 269523)
+++ trunk/Source/WebKit/WebProcess/Inspector/WebInspectorUIExtensionController.h        2020-11-06 18:44:09 UTC (rev 269524)
</span><span class="lines">@@ -1,74 +0,0 @@
</span><del>-/*
- * Copyright (C) 2020 Apple 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:
- * 1. Redistributions of source code must retain the above copyright
- *    notice, this list of conditions and the following disclaimer.
- * 2. Redistributions in binary form must reproduce the above copyright
- *    notice, this list of conditions and the following disclaimer in the
- *    documentation and/or other materials provided with the distribution.
- *
- * THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS''
- * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO,
- * THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
- * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR ITS CONTRIBUTORS
- * BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
- * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
- * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
- * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
- * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
- * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF
- * THE POSSIBILITY OF SUCH DAMAGE.
- */
-
-#pragma once
-
-#if ENABLE(INSPECTOR_EXTENSIONS)
-
-#include "Connection.h"
-#include "InspectorExtensionTypes.h"
-#include "MessageReceiver.h"
-#include <wtf/Forward.h>
-#include <wtf/Noncopyable.h>
-#include <wtf/URL.h>
-#include <wtf/WeakPtr.h>
-
-namespace JSC {
-class JSValue;
-}
-
-namespace WebCore {
-class InspectorFrontendClient;
-}
-
-namespace WebKit {
-
-class WebInspectorUI;
-
-class WebInspectorUIExtensionController
-    : public IPC::MessageReceiver
-    , public CanMakeWeakPtr<WebInspectorUIExtensionController> {
-    WTF_MAKE_FAST_ALLOCATED;
-    WTF_MAKE_NONCOPYABLE(WebInspectorUIExtensionController);
-public:
-    explicit WebInspectorUIExtensionController(WebCore::InspectorFrontendClient&);
-    ~WebInspectorUIExtensionController();
-
-    // Implemented in generated WebInspectorUIExtensionControllerMessageReceiver.cpp
-    void didReceiveMessage(IPC::Connection&, IPC::Decoder&) override;
-
-    // WebInspectorUIExtensionController IPC messages.
-    void registerExtension(const String& extensionID, const String& displayName, CompletionHandler<void(Expected<bool, InspectorExtensionError>)>&&);
-    void unregisterExtension(const String& extensionID, CompletionHandler<void(Expected<bool, InspectorExtensionError>)>&&);
-
-private:
-    Optional<InspectorExtensionError> parseInspectorExtensionErrorFromResult(JSC::JSValue result);
-
-    WeakPtr<WebCore::InspectorFrontendClient> m_frontendClient;
-};
-
-} // namespace WebKit
-
-#endif // ENABLE(INSPECTOR_EXTENSIONS)
</del></span></pre></div>
<a id="trunkSourceWebKitWebProcessInspectorWebInspectorUIExtensionControllermessagesin"></a>
<div class="delfile"><h4>Deleted: trunk/Source/WebKit/WebProcess/Inspector/WebInspectorUIExtensionController.messages.in (269523 => 269524)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit/WebProcess/Inspector/WebInspectorUIExtensionController.messages.in   2020-11-06 18:42:14 UTC (rev 269523)
+++ trunk/Source/WebKit/WebProcess/Inspector/WebInspectorUIExtensionController.messages.in      2020-11-06 18:44:09 UTC (rev 269524)
</span><span class="lines">@@ -1,30 +0,0 @@
</span><del>-# Copyright (C) 2020 Apple 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:
-# 1.  Redistributions of source code must retain the above copyright
-#     notice, this list of conditions and the following disclaimer.
-# 2.  Redistributions in binary form must reproduce the above copyright
-#     notice, this list of conditions and the following disclaimer in the
-#     documentation and/or other materials provided with the distribution.
-#
-# THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS'' AND
-# ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED
-# WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE
-# DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR ITS CONTRIBUTORS BE LIABLE FOR
-# ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
-# DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR
-# SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER
-# CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY,
-# OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
-# OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
-
-#if ENABLE(INSPECTOR_EXTENSIONS)
-
-messages -> WebInspectorUIExtensionController NotRefCounted {
-    RegisterExtension(String extensionID, String displayName) -> (Expected<bool, WebKit::InspectorExtensionError> result) Async
-    UnregisterExtension(String extensionID) -> (Expected<bool, WebKit::InspectorExtensionError> result) Async
-}
-
-#endif // ENABLE(INSPECTOR_EXTENSIONS)
</del></span></pre></div>
<a id="trunkToolsChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/Tools/ChangeLog (269523 => 269524)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Tools/ChangeLog    2020-11-06 18:42:14 UTC (rev 269523)
+++ trunk/Tools/ChangeLog       2020-11-06 18:44:09 UTC (rev 269524)
</span><span class="lines">@@ -1,3 +1,16 @@
</span><ins>+2020-11-06  Truitt Savell  <tsavell@apple.com>
+
+        Unreviewed, reverting r269486.
+
+        Caused 50+ timeouts on Mac Debug WK2
+
+        Reverted changeset:
+
+        "[Cocoa] Inspector Extensions: Add _WKInspectorExtension and
+        related plumbing"
+        https://bugs.webkit.org/show_bug.cgi?id=217783
+        https://trac.webkit.org/changeset/269486
+
</ins><span class="cx"> 2020-11-06  Jer Noble  <jer.noble@apple.com>
</span><span class="cx"> 
</span><span class="cx">         Do not allow pages to enter fullscreen while an exit fullscreen operation is ongoing.
</span></span></pre></div>
<a id="trunkToolsTestWebKitAPITestWebKitAPIxcodeprojprojectpbxproj"></a>
<div class="modfile"><h4>Modified: trunk/Tools/TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj (269523 => 269524)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Tools/TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj        2020-11-06 18:42:14 UTC (rev 269523)
+++ trunk/Tools/TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj   2020-11-06 18:44:09 UTC (rev 269524)
</span><span class="lines">@@ -835,7 +835,6 @@
</span><span class="cx">          9984FACC1CFFAF60008D198C /* WKWebViewTextInput.mm in Sources */ = {isa = PBXBuildFile; fileRef = 9984FACA1CFFAEEE008D198C /* WKWebViewTextInput.mm */; };
</span><span class="cx">          9984FACE1CFFB090008D198C /* editable-body.html in Copy Resources */ = {isa = PBXBuildFile; fileRef = 9984FACD1CFFB038008D198C /* editable-body.html */; };
</span><span class="cx">          9999108B1F393C96008AD455 /* Copying.mm in Sources */ = {isa = PBXBuildFile; fileRef = 9999108A1F393C8B008AD455 /* Copying.mm */; };
</span><del>-               999B7EE32551C63B00F450A4 /* WKInspectorExtensionHost.mm in Sources */ = {isa = PBXBuildFile; fileRef = 999B7EE22551C63B00F450A4 /* WKInspectorExtensionHost.mm */; };
</del><span class="cx">           99B4F9C624EDED9700022B82 /* WKInspectorDelegate.mm in Sources */ = {isa = PBXBuildFile; fileRef = 99B4F9C524EDED9600022B82 /* WKInspectorDelegate.mm */; };
</span><span class="cx">          9B02E0D6235FA47D004044B2 /* TextManipulation.mm in Sources */ = {isa = PBXBuildFile; fileRef = 9B02E0D5235FA47D004044B2 /* TextManipulation.mm */; };
</span><span class="cx">          9B0786A51C5885C300D159E3 /* InjectedBundleMakeAllShadowRootsOpen_Bundle.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 9B0786A41C5885C300D159E3 /* InjectedBundleMakeAllShadowRootsOpen_Bundle.cpp */; };
</span><span class="lines">@@ -2447,7 +2446,6 @@
</span><span class="cx">          9984FACA1CFFAEEE008D198C /* WKWebViewTextInput.mm */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.objcpp; path = WKWebViewTextInput.mm; sourceTree = "<group>"; };
</span><span class="cx">          9984FACD1CFFB038008D198C /* editable-body.html */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = text.html; path = "editable-body.html"; sourceTree = "<group>"; };
</span><span class="cx">          9999108A1F393C8B008AD455 /* Copying.mm */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.objcpp; path = Copying.mm; sourceTree = "<group>"; };
</span><del>-               999B7EE22551C63B00F450A4 /* WKInspectorExtensionHost.mm */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.objcpp; path = WKInspectorExtensionHost.mm; sourceTree = "<group>"; };
</del><span class="cx">           99B4F9C524EDED9600022B82 /* WKInspectorDelegate.mm */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.objcpp; path = WKInspectorDelegate.mm; sourceTree = "<group>"; };
</span><span class="cx">          9B02E0D5235FA47D004044B2 /* TextManipulation.mm */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.cpp.objcpp; path = TextManipulation.mm; sourceTree = "<group>"; };
</span><span class="cx">          9B0786A21C58830F00D159E3 /* InjectedBundleMakeAllShadowRootsOpen.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = InjectedBundleMakeAllShadowRootsOpen.cpp; sourceTree = "<group>"; };
</span><span class="lines">@@ -3411,7 +3409,6 @@
</span><span class="cx">                          370CE2291F57343400E7410B /* WKContentViewTargetForAction.mm */,
</span><span class="cx">                          51D124971E763AF8002B2820 /* WKHTTPCookieStore.mm */,
</span><span class="cx">                          99B4F9C524EDED9600022B82 /* WKInspectorDelegate.mm */,
</span><del>-                               999B7EE22551C63B00F450A4 /* WKInspectorExtensionHost.mm */,
</del><span class="cx">                           A5A729F01F622A9A00DE5A28 /* WKNavigationResponse.mm */,
</span><span class="cx">                          DF4B273821A47727009BD1CA /* WKNSDictionaryEmptyDictionaryCrash.mm */,
</span><span class="cx">                          375E0E151D66674400EFEC2C /* WKNSNumber.mm */,
</span><span class="lines">@@ -5545,7 +5542,6 @@
</span><span class="cx">                          51D124981E763B02002B2820 /* WKHTTPCookieStore.mm in Sources */,
</span><span class="cx">                          7CCE7F1D1A411AE600447C4C /* WKImageCreateCGImageCrash.cpp in Sources */,
</span><span class="cx">                          99B4F9C624EDED9700022B82 /* WKInspectorDelegate.mm in Sources */,
</span><del>-                               999B7EE32551C63B00F450A4 /* WKInspectorExtensionHost.mm in Sources */,
</del><span class="cx">                           A5A729F11F622AA700DE5A28 /* WKNavigationResponse.mm in Sources */,
</span><span class="cx">                          DF4B273921A47728009BD1CA /* WKNSDictionaryEmptyDictionaryCrash.mm in Sources */,
</span><span class="cx">                          375E0E171D66674400EFEC2C /* WKNSNumber.mm in Sources */,
</span></span></pre></div>
<a id="trunkToolsTestWebKitAPITestsWebKitCocoaWKInspectorExtensionHostmm"></a>
<div class="delfile"><h4>Deleted: trunk/Tools/TestWebKitAPI/Tests/WebKitCocoa/WKInspectorExtensionHost.mm (269523 => 269524)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Tools/TestWebKitAPI/Tests/WebKitCocoa/WKInspectorExtensionHost.mm  2020-11-06 18:42:14 UTC (rev 269523)
+++ trunk/Tools/TestWebKitAPI/Tests/WebKitCocoa/WKInspectorExtensionHost.mm     2020-11-06 18:44:09 UTC (rev 269524)
</span><span class="lines">@@ -1,171 +0,0 @@
</span><del>-/*
- * Copyright (C) 2020 Apple 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:
- * 1. Redistributions of source code must retain the above copyright
- *    notice, this list of conditions and the following disclaimer.
- * 2. Redistributions in binary form must reproduce the above copyright
- *    notice, this list of conditions and the following disclaimer in the
- *    documentation and/or other materials provided with the distribution.
- *
- * THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS''
- * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO,
- * THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
- * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR ITS CONTRIBUTORS
- * BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
- * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
- * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
- * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
- * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
- * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF
- * THE POSSIBILITY OF SUCH DAMAGE.
- */
-
-#import "config.h"
-
-#import "Test.h"
-#import "Utilities.h"
-#import <WebKit/WKPreferencesPrivate.h>
-#import <WebKit/WKWebViewPrivate.h>
-#import <WebKit/_WKInspector.h>
-#import <WebKit/_WKInspectorPrivateForTesting.h>
-#import <wtf/RetainPtr.h>
-
-#if ENABLE(INSPECTOR_EXTENSIONS)
-
-static bool didAttachLocalInspectorCalled = false;
-static bool pendingCallbackWasCalled = false;
-
-static void resetGlobalState()
-{
-    didAttachLocalInspectorCalled = false;
-    pendingCallbackWasCalled = false;
-}
-
-@interface UIDelegateForTestingInspectorExtensionHost : NSObject <WKUIDelegate>
-@end
-
-@implementation UIDelegateForTestingInspectorExtensionHost
-
-- (void)_webView:(WKWebView *)webView didAttachLocalInspector:(_WKInspector *)inspector
-{
-    EXPECT_EQ(webView._inspector, inspector);
-    didAttachLocalInspectorCalled = true;
-}
-
-@end
-
-TEST(WKInspectorExtensionHost, RegisterExtension)
-{
-    resetGlobalState();
-
-    auto webViewConfiguration = adoptNS([WKWebViewConfiguration new]);
-    webViewConfiguration.get().preferences._developerExtrasEnabled = YES;
-    auto webView = adoptNS([[WKWebView alloc] initWithFrame:CGRectMake(0, 0, 800, 600) configuration:webViewConfiguration.get()]);
-    auto uiDelegate = adoptNS([UIDelegateForTestingInspectorExtensionHost new]);
-
-    [webView setUIDelegate:uiDelegate.get()];
-    [webView loadHTMLString:@"<head><title>Test page to be inspected</title></head><body><p>Filler content</p></body>" baseURL:[NSURL URLWithString:@"http://example.com/"]];
-
-    [[webView _inspector] show];
-    TestWebKitAPI::Util::run(&didAttachLocalInspectorCalled);
-
-    auto firstID = [NSUUID UUID].UUIDString;
-    auto secondID = [NSUUID UUID].UUIDString;
-
-    // Normal registration.
-    pendingCallbackWasCalled = false;
-    [[webView _inspector] registerExtensionWithID:firstID displayName:@"FirstExtension" completionHandler:^(NSError * _Nullable error, _WKInspectorExtension * _Nullable extension) {
-        EXPECT_NULL(error);
-        EXPECT_NOT_NULL(extension);
-
-        pendingCallbackWasCalled = true;
-    }];
-    TestWebKitAPI::Util::run(&pendingCallbackWasCalled);
-
-    // Double registration.
-    pendingCallbackWasCalled = false;
-    [[webView _inspector] registerExtensionWithID:firstID displayName:@"FirstExtension" completionHandler:^(NSError * _Nullable error, _WKInspectorExtension * _Nullable extension) {
-        EXPECT_NOT_NULL(error);
-        EXPECT_NULL(extension);
-        EXPECT_TRUE([error.localizedFailureReason containsString:@"RegistrationFailed"]);
-
-        pendingCallbackWasCalled = true;
-    }];
-    TestWebKitAPI::Util::run(&pendingCallbackWasCalled);
-
-    // Two registrations.
-    pendingCallbackWasCalled = false;
-    [[webView _inspector] registerExtensionWithID:secondID displayName:@"SecondExtension" completionHandler:^(NSError * _Nullable error, _WKInspectorExtension * _Nullable extension) {
-        EXPECT_NULL(error);
-        EXPECT_NOT_NULL(extension);
-
-        pendingCallbackWasCalled = true;
-    }];
-    TestWebKitAPI::Util::run(&pendingCallbackWasCalled);
-}
-
-TEST(WKInspectorExtensionHost, UnregisterExtension)
-{
-    resetGlobalState();
-
-    auto webViewConfiguration = adoptNS([WKWebViewConfiguration new]);
-    webViewConfiguration.get().preferences._developerExtrasEnabled = YES;
-    auto webView = adoptNS([[WKWebView alloc] initWithFrame:CGRectMake(0, 0, 800, 600) configuration:webViewConfiguration.get()]);
-    auto uiDelegate = adoptNS([UIDelegateForTestingInspectorExtensionHost new]);
-
-    [webView setUIDelegate:uiDelegate.get()];
-    [webView loadHTMLString:@"<head><title>Test page to be inspected</title></head><body><p>Filler content</p></body>" baseURL:[NSURL URLWithString:@"http://example.com/"]];
-
-    [[webView _inspector] show];
-    TestWebKitAPI::Util::run(&didAttachLocalInspectorCalled);
-
-    auto firstID = [NSUUID UUID].UUIDString;
-    __block RetainPtr<_WKInspectorExtension> foundExtension;
-
-    // Unregister a known extension.
-    pendingCallbackWasCalled = false;
-    [[webView _inspector] registerExtensionWithID:firstID displayName:@"FirstExtension" completionHandler:^(NSError * _Nullable error, _WKInspectorExtension * _Nullable extension) {
-        EXPECT_NULL(error);
-        EXPECT_NOT_NULL(extension);
-        foundExtension = extension;
-
-        [[webView _inspector] unregisterExtension:foundExtension.get() completionHandler:^(NSError * _Nullable error) {
-            EXPECT_NULL(error);
-
-            pendingCallbackWasCalled = true;
-        }];
-    }];
-    TestWebKitAPI::Util::run(&pendingCallbackWasCalled);
-    EXPECT_NOT_NULL(foundExtension.get());
-
-    // Re-register an extension.
-    pendingCallbackWasCalled = false;
-    [[webView _inspector] registerExtensionWithID:firstID displayName:@"FirstExtension" completionHandler:^(NSError * _Nullable error, _WKInspectorExtension * _Nullable extension) {
-        EXPECT_NULL(error);
-        EXPECT_NOT_NULL(extension);
-        foundExtension = extension;
-
-        [[webView _inspector] unregisterExtension:foundExtension.get() completionHandler:^(NSError * _Nullable error) {
-            EXPECT_NULL(error);
-
-            pendingCallbackWasCalled = true;
-        }];
-    }];
-    TestWebKitAPI::Util::run(&pendingCallbackWasCalled);
-    EXPECT_NOT_NULL(foundExtension.get());
-
-    // Unregister an extension twice.
-    pendingCallbackWasCalled = false;
-    [[webView _inspector] unregisterExtension:foundExtension.get() completionHandler:^(NSError * _Nullable error) {
-        EXPECT_NOT_NULL(error);
-        EXPECT_TRUE([error.localizedFailureReason containsString:@"InvalidRequest"]);
-
-        pendingCallbackWasCalled = true;
-    }];
-    TestWebKitAPI::Util::run(&pendingCallbackWasCalled);
-}
-
-#endif // ENABLE(INSPECTOR_EXTENSIONS)
</del></span></pre>
</div>
</div>

</body>
</html>