DevTools: [Documentation] Add signature section for renderer

Source/devtools/front_end/documentation/DocumentationView.js

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 /**
  * @constructor
  * @extends {WebInspector.View}
  */
 WebInspector.DocumentationView = function()
 {
@@ skipping 84 matching lines @@
 }
 
 WebInspector.DocumentationView.Renderer.prototype = {
     /**
      * @return {!Element}
      */
     renderJSArticle: function()
     {
         this._createPageTitle(this._article.pageTitle, this._searchItem);
         this._createStandardizationStatus(this._article.standardizationStatus);
+        this._createSignatureSection(this._article.parameters, this._article.methods);
         this._createTextSectionWithTitle("Summary", this._article.summary);
-        this._createSignatureSection(this._article.parameters, this._article.methods);
+        this._createParametersSection(this._article.parameters);
         this._createReturnValueSection(this._article.methods);
         this._createExamplesSection(this._article.examples);
         this._createTextSectionWithTitle("Remarks", this._article.remarks);
 
         return this._element;
     },
 
     /**
      * @param {string} titleText
      * @param {string} searchItem
@@ skipping 24 matching lines @@
      */
     _createTextSectionWithTitle: function(titleText, article)
     {
         if (!article)
             return;
         var section = this._element.createChild("div", "documentation-section");
         var title = section.createChild("div", "documentation-section-title");
         title.textContent = titleText;
         var text = this._renderBlock(article);
         text.classList.add("documentation-text");
+        text.classList.add("documentation-section-content");
         section.appendChild(text);
     },
 
     /**
      * @param {!Array.<!WebInspector.JSArticle.Parameter>} parameters
      * @param {?WebInspector.JSArticle.Method} method
      */
     _createSignatureSection: function(parameters, method)
     {
+        if (!method)
+            return;
+        var section = this._element.createChild("p", "documentation-section");
+        var signature = section.createChild("span", "documentation-method-signature documentation-section-content monospace");
+        var methodName = signature.createChild("span", "documentation-method-name");
+        methodName.textContent = this._searchItem.split(".").peekLast() + "(";
+        for (var i = 0; i < parameters.length; ++i) {
+            if (i > 0)
+                signature.createTextChild(", ")
+            var parameterType = signature.createChild("span", "documentation-parameter-data-type-value");
+            parameterType.textContent = parameters[i].dataType + (parameters[i].optional ? "=" : "");;
+        }
+        signature.createTextChild("): ");
+        var returnTypeElement = signature.createChild("span", "documentation-parameter-data-type-value");
+        returnTypeElement.textContent = method.returnValueName;
+    },
+
+    /**
+     * @param {!Array.<!WebInspector.JSArticle.Parameter>} parameters
+     */
+    _createParametersSection: function(parameters)
+    {
         if (!parameters.length)
             return;
         var section = this._element.createChild("div", "documentation-section");
         var title = section.createChild("div", "documentation-section-title");
         title.textContent = "Parameters";
         for (var i = 0; i < parameters.length; ++i) {
-            var parameter = section.createChild("div", "documentation-parameter");
+            var parameter = section.createChild("div", "documentation-parameter documentation-section-content");
             var header = parameter.createChild("div", "documentation-parameter-header");
             var name = header.createChild("span", "documentation-parameter-name");
             name.textContent = parameters[i].name;
-            var dataTypeValue = header.createChild("span", "documentation-parameter-data-type-value");
+            var dataTypeValue = header.createChild("span", "documentation-parameter-data-type-value documentation-parameter-margin");
             dataTypeValue.textContent = parameters[i].dataType;
             if (parameters[i].optional) {
-                var optional = header.createChild("span", "documentation-parameter-optional");
+                var optional = header.createChild("span", "documentation-parameter-optional documentation-parameter-margin");
                 optional.textContent = WebInspector.UIString("Optional");
             }
             parameter.appendChild(this._renderBlock(parameters[i].description));
         }
     },
 
     /**
      * @param {?WebInspector.JSArticle.Method} method
      */
     _createReturnValueSection: function(method)
     {
         if (!method)
             return;
-
         var section = this._element.createChild("div", "documentation-section");
         var title = section.createChild("div", "documentation-section-title");
         title.textContent = "Return Value";
-        var returnValueName = section.createChild("div", "documentation-return-value");
+        var returnValueName = section.createChild("div", "documentation-section-content documentation-return-value");
         returnValueName.textContent = WebInspector.UIString("Returns an object of type " + method.returnValueName + ".");
-        var returnValueDescription = section.createChild("div", "documentation-return-value");
+        var returnValueDescription = section.createChild("div", "documentation-section-content documentation-text");
         returnValueDescription.textContent = WebInspector.UIString(method.returnValueDescription);
     },
 
     /**
      * @param {!Array.<!WebInspector.JSArticle.Example>} examples
      */
     _createExamplesSection: function(examples)
     {
         if (!examples.length)
             return;
 
         var section = this._element.createChild("div", "documentation-section");
         var title = section.createChild("div", "documentation-section-title");
         title.textContent = "Examples";
 
         for (var i = 0; i < examples.length; ++i) {
-            var example = section.createChild("div", "documentation-example");
+            var example = section.createChild("div", "documentation-example documentation-section-content");
             var exampleDescription = example.createChild("div", "documentation-example-description-section");
             if (examples[i].liveUrl) {
                 var liveUrl = exampleDescription.createChild("a", "documentation-example-link");
                 liveUrl.href = examples[i].liveUrl;
                 liveUrl.textContent = WebInspector.UIString("Example");
             }
-            exampleDescription.appendChild(this._renderBlock(examples[i].description));
-            var code = example.createChild("div", "documentation-example-code");
-            code.classList.add("source-code");
+            var description = this._renderBlock(examples[i].description);
+            description.classList.add("documentation-text");
+            exampleDescription.appendChild(description);
+            var code = example.createChild("div", "documentation-example-code source-code");
             code.textContent = examples[i].code;
             if (!examples[i].language)
                 continue;
             var syntaxHighlighter = new WebInspector.DOMSyntaxHighlighter(WebInspector.DocumentationView._languageToMimeType[examples[i].language.toLowerCase()], true);
             syntaxHighlighter.syntaxHighlightNode(code);
         }
     },
 
     /**
      * @param {!WebInspector.WikiParser.ArticleElement} article
@@ skipping 11 matching lines @@
         case elementTypes.Link:
             element = document.createElementWithClass("a", "documentation-link");
             element.href = article.url();
             if (!article.children().length)
                 element.textContent = article.url();
             break;
         case elementTypes.Code:
             element = document.createElementWithClass("span", "documentation-code-tag");
             break;
         case elementTypes.CodeBlock:
-            element = document.createElementWithClass("pre", "documentation-code");
-            element.classList.add("source-code");
+            element = document.createElementWithClass("pre", "documentation-code source-code");
             element.textContent = article.code();
             break;
         case elementTypes.PlainText:
             element = document.createElement("span");
             element.textContent = article.text();
             if (article.isHighlighted())
                 element.classList.add("documentation-highlighted-text");
             break;
         default:
             console.error("Unknown ArticleElement type " + article.type());
@@ skipping 31 matching lines @@
     appendApplicableItems: function(event, contextMenu, target)
     {
         if (!(target instanceof WebInspector.CodeMirrorTextEditor))
             return;
         var textEditor = /** @type {!WebInspector.CodeMirrorTextEditor} */ (target);
         var descriptors = this._determineDescriptors(textEditor);
         if (!descriptors.length)
             return;
         if (descriptors.length === 1) {
             var formatString = WebInspector.useLowerCaseMenuTitles() ? "Show documentation for %s.%s" : "Show Documentation for %s.%s";
-            contextMenu.appendItem(WebInspector.UIString(formatString, descriptors[0].name(), descriptors[0].searchItem()), WebInspector.DocumentationView.showDocumentationURL.bind(null, descriptors[0].url(), descriptors[0].searchItem()));
+            var methodName = String.sprintf("%s.%s", descriptors[0].name(), descriptors[0].searchItem());
+            contextMenu.appendItem(WebInspector.UIString(formatString, descriptors[0].name(), descriptors[0].searchItem()), WebInspector.DocumentationView.showDocumentationURL.bind(null, descriptors[0].url(), methodName));
             return;
         }
         var subMenuItem = contextMenu.appendSubMenuItem(WebInspector.UIString(WebInspector.useLowerCaseMenuTitles() ? "Show documentation for..." : "Show Documentation for..."));
-        for (var i = 0; i < descriptors.length; ++i)
-            subMenuItem.appendItem(String.sprintf("%s.%s", descriptors[i].name(), descriptors[i].searchItem()), WebInspector.DocumentationView.showDocumentationURL.bind(null, descriptors[i].url(), descriptors[i].searchItem()));
+        for (var i = 0; i < descriptors.length; ++i) {
+            var methodName = String.sprintf("%s.%s", descriptors[i].name(), descriptors[i].searchItem());
+            subMenuItem.appendItem(methodName, WebInspector.DocumentationView.showDocumentationURL.bind(null, descriptors[i].url(), methodName));
+        }
     },
 
     /**
      * @param {!WebInspector.CodeMirrorTextEditor} textEditor
      * @return {!Array.<!WebInspector.DocumentationURLProvider.ItemDescriptor>}
      */
     _determineDescriptors: function(textEditor)
     {
         var urlProvider = WebInspector.DocumentationURLProvider.instance();
         var textSelection = textEditor.selection().normalize();
@@ skipping 17 matching lines @@
         function computeDescriptors(column)
         {
             var token = textEditor.tokenAtTextPosition(textSelection.startLine, column);
             if (!token)
                 return [];
             var tokenText = textEditor.line(textSelection.startLine).substring(token.startColumn, token.endColumn);
             return urlProvider.itemDescriptors(tokenText);
         }
     }
 }