Devtools: [WIP] Implement enhanced devtools extension language APIs

Source/devtools/front_end/main/Main.js

 /*
  * Copyright (C) 2006, 2007, 2008 Apple Inc.  All rights reserved.
  * Copyright (C) 2007 Matt Lilek (pewtermoose@gmail.com).
  * Copyright (C) 2009 Joseph Pecoraro
  *
  * 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
@@ skipping 192 matching lines @@
         WebInspector.fileManager = new WebInspector.FileManager();
         WebInspector.isolatedFileSystemManager = new WebInspector.IsolatedFileSystemManager();
         WebInspector.workspace = new WebInspector.Workspace(WebInspector.isolatedFileSystemManager.mapping());
         WebInspector.networkMapping = new WebInspector.NetworkMapping(WebInspector.workspace, WebInspector.isolatedFileSystemManager.mapping());
         WebInspector.networkProjectManager = new WebInspector.NetworkProjectManager(WebInspector.targetManager, WebInspector.workspace, WebInspector.networkMapping);
         WebInspector.presentationConsoleMessageHelper = new WebInspector.PresentationConsoleMessageHelper(WebInspector.workspace);
         WebInspector.cssWorkspaceBinding = new WebInspector.CSSWorkspaceBinding(WebInspector.targetManager, WebInspector.workspace, WebInspector.networkMapping);
         WebInspector.debuggerWorkspaceBinding = new WebInspector.DebuggerWorkspaceBinding(WebInspector.targetManager, WebInspector.workspace, WebInspector.networkMapping);
         WebInspector.fileSystemWorkspaceBinding = new WebInspector.FileSystemWorkspaceBinding(WebInspector.isolatedFileSystemManager, WebInspector.workspace, WebInspector.networkMapping);
         WebInspector.breakpointManager = new WebInspector.BreakpointManager(null, WebInspector.workspace, WebInspector.networkMapping, WebInspector.targetManager, WebInspector.debuggerWorkspaceBinding);
-        WebInspector.extensionServer = new WebInspector.ExtensionServer();
+        WebInspector.extensionServer = new WebInspector.ExtensionServer(new WebInspector.ExtensionServer.UIDelegateImpl());
 
         new WebInspector.OverlayController();
         new WebInspector.ContentScriptProjectDecorator();
         new WebInspector.ExecutionContextSelector(WebInspector.targetManager, WebInspector.context);
 
         var autoselectPanel = WebInspector.UIString("a panel chosen automatically");
         var openAnchorLocationSetting = WebInspector.settings.createSetting("openLinkHandler", autoselectPanel);
         WebInspector.openAnchorLocationRegistry = new WebInspector.HandlerRegistry(openAnchorLocationSetting);
         WebInspector.openAnchorLocationRegistry.registerHandler(autoselectPanel, function() { return false; });
         WebInspector.Linkifier.setLinkHandler(new WebInspector.HandlerRegistry.LinkHandler());
@@ skipping 613 matching lines @@
     var p = this.helpContentElement.createChild("p");
     p.classList.add("help-section");
     p.textContent = WebInspector.UIString("Inspected worker has terminated. Once it restarts we will attach to it automatically.");
 }
 
 WebInspector.WorkerTerminatedScreen.prototype = {
 
     __proto__: WebInspector.HelpScreen.prototype
 }
 
+// TODO: Expose the 'sources' panel features more cleanly; files are

[pfeldman, 2015/08/13 21:15:46]

This needs to use runtime DI 

[wes, 2015/08/14 00:55:05]

It can also probably be moved out of Main and back into extension server.
It doesn't really need to be a delegate.

+// opened via the `WebInspector.Revealer` construct (which does similar things
+// to the below) - we need something similar for each of these, probably.
+/**
+ * @constructor
+ * @implements {WebInspector.ExtensionServer.UIDelegate}
+ */
+WebInspector.ExtensionServer.UIDelegateImpl = function() {}
+WebInspector.ExtensionServer.UIDelegateImpl.prototype = {
+    /**
+     * @override
+     * @param {!WebInspector.UISourceCode} code
+     * @param {!Array<{kind: string, text: string, location: {startLine: number, startColumn: number, endLine: number, endColumn: number}}>} messages
+     * @suppress {missingProperties}
+     */
+    setResourceLineMessages: function(code, messages) {
+        self.runtime.loadModulePromise("sources").then(function() { //sources isn't autoloaded, but it tracks line messages - load it

[pfeldman, 2015/08/13 21:15:46]

For example check out UISourceCode revealers (Revealer.reveal)

[wes, 2015/08/14 00:55:05]

I actually mentioned that class in my TODO above. ;3
So this would be like a revealer but rather than opening the view, it should
simply
call setMessagesForSource on it.

So would I create a construct which simply functions like a Revealer; 
or hijack the existing revealer construct and re-purpose it to 'reveal'
resource line messages?

[pfeldman, 2015/08/17 21:15:51]

I think we need to introduce proper markers the UISourceCodes themselves. The
ones that would bind messages to lines and allow listening to them.

[wes, 2015/08/25 18:13:18]

IMO, binding messages to lines is very limiting. If we're overhauling them, we
want to be able to bind messages to arbitrary spans (which may span multiple
lines or only part of a line).

+            var frame = WebInspector.SourcesPanel.instance().sourcesView().viewForFile(code);
+            frame.setMessagesForSource(messages.map(function(m){
+                return new WebInspector.SourceFrameMessage(m.text, WebInspector.SourceFrameMessage.Level[m.kind], {
+                    line: m.location.startLine,
+                    column: m.location.startColumn
+                }, {
+                    line: m.location.endLine,
+                    column: m.location.endColumn
+                });
+            }));
+        });
+    },
+
+    /**
+     * @override
+     * @param {!WebInspector.UISourceCode} code
+     * @return {!Promise<!SourceFrameMessageGetter>}
+     * @suppress {missingProperties}
+     */
+    getResourceLineMessages: function(code) {

[wes, 2015/08/14 00:55:05]

We mentioned removing the `get` line messages (and the change event) at our
meeting. 
I just want to record that - this will likely be gone in the next patch and
that'll be why.

+        return self.runtime.loadModulePromise("sources").then(function() {
+            var frame = WebInspector.SourcesPanel.instance().sourcesView().viewForFile(code);
+            return /** @type {!SourceFrameMessageGetter} */ (frame.getMessagesFromSource());
+        });
+    },
+
+    /**
+     * @override
+     * @param {string} highlightText
+     * @param {!Array<!SearchResultArray>} groupedResults
+     * @suppress {missingProperties}
+     */
+    displaySearchResults: function(highlightText, groupedResults) {
+        //TODO: remove private refs, replace with (likely new) public ones
+        self.runtime.loadModulePromise("sources").then(function() { //sources isn't autoloaded - ensure it exists since we're about to swap to it!
+            var searchResultsPanel = WebInspector.AdvancedSearchView._instance;
+            if (!searchResultsPanel) {
+                searchResultsPanel = new WebInspector.AdvancedSearchView();
+            }
+
+            searchResultsPanel._nothingFound();
+            searchResultsPanel._searchConfig = new WebInspector.SearchConfig(highlightText, true, false); //TODO: HACK - really should be able to highlight arbitrary spans
+            searchResultsPanel._searchResultsPane = searchResultsPanel._searchScope.createSearchResultsPane(searchResultsPanel._searchConfig);
+            searchResultsPanel._resetResults();
+            searchResultsPanel._searchResultsElement.appendChild(searchResultsPanel._searchResultsPane.element);
+
+
+            WebInspector.inspectorView.setCurrentPanel(WebInspector.SourcesPanel.instance());
+            WebInspector.inspectorView.showViewInDrawer("sources.search");
+
+            groupedResults.map(function(sourceResult) { //map to file search result
+                var code = WebInspector.networkMapping.uiSourceCodeFromURL(sourceResult.source);
+                if (!code) {
+                    return;
+                }
+                return new WebInspector.FileBasedSearchResult(
+                    code,
+                    sourceResult.map(function(item){
+                        return new WebInspector.ContentProvider.SearchMatch(item.line, item.lineContent);
+                    })
+                );
+            }).forEach(function(searchResult) {
+                if (!searchResult) return;
+                searchResultsPanel._addSearchResult(searchResult);
+                searchResultsPanel._searchResultsPane.addSearchResult(searchResult);
+            });
+        });
+     },
+
+    /**
+     * @override
+     * @param {!Array<string>} mimes
+     * @param {?} mode
+     */
+     addSimpleCodeMirrorMode: function(mimes, mode) {
+         self.runtime.loadModulePromise("source_frame").then(function() { //also not autoloaded (Though TBH this could just add waiting ones on start, rather than loading it when we want to add one)
+            for (var i=0; i<mimes.length; i++) {
+                CodeMirror.defineSimpleMode(mimes[i], mode);
+            }
+         });
+     }
+}
+
 new WebInspector.Main();