Title: [226909] trunk/Source/WebInspectorUI
Revision
226909
Author
[email protected]
Date
2018-01-12 13:03:22 -0800 (Fri, 12 Jan 2018)

Log Message

Web Inspector: Support JSX (React) syntax highlighting
https://bugs.webkit.org/show_bug.cgi?id=181607
<rdar://problem/36442564>

Patch by Joseph Pecoraro <[email protected]> on 2018-01-12
Reviewed by Brian Burg.

* UserInterface/Base/MIMETypeUtilities.js:
(WI.mimeTypeForFileExtension):
(WI.fileExtensionForMIMEType):
* UserInterface/Models/Resource.js:
Support the jsx extension and mime types.

* UserInterface/Main.html:
* Scripts/update-codemirror-resources.rb:
* UserInterface/External/CodeMirror/jsx.js: Added.
Include new mode from CodeMirror@d8926768.

Modified Paths

Added Paths

Diff

Modified: trunk/Source/WebInspectorUI/ChangeLog (226908 => 226909)


--- trunk/Source/WebInspectorUI/ChangeLog	2018-01-12 21:01:02 UTC (rev 226908)
+++ trunk/Source/WebInspectorUI/ChangeLog	2018-01-12 21:03:22 UTC (rev 226909)
@@ -1,3 +1,22 @@
+2018-01-12  Joseph Pecoraro  <[email protected]>
+
+        Web Inspector: Support JSX (React) syntax highlighting
+        https://bugs.webkit.org/show_bug.cgi?id=181607
+        <rdar://problem/36442564>
+
+        Reviewed by Brian Burg.
+
+        * UserInterface/Base/MIMETypeUtilities.js:
+        (WI.mimeTypeForFileExtension):
+        (WI.fileExtensionForMIMEType):
+        * UserInterface/Models/Resource.js:
+        Support the jsx extension and mime types.
+
+        * UserInterface/Main.html:
+        * Scripts/update-codemirror-resources.rb:
+        * UserInterface/External/CodeMirror/jsx.js: Added.
+        Include new mode from CodeMirror@d8926768.
+
 2018-01-11  Joseph Pecoraro  <[email protected]>
 
         Web Inspector: Rename "Query String" section as "Query String Parameters" for clarity

Modified: trunk/Source/WebInspectorUI/Scripts/update-codemirror-resources.rb (226908 => 226909)


--- trunk/Source/WebInspectorUI/Scripts/update-codemirror-resources.rb	2018-01-12 21:01:02 UTC (rev 226908)
+++ trunk/Source/WebInspectorUI/Scripts/update-codemirror-resources.rb	2018-01-12 21:03:22 UTC (rev 226909)
@@ -47,6 +47,7 @@
   mode/css/css.js
   mode/htmlmixed/htmlmixed.js
   mode/_javascript_/_javascript_.js
+  mode/jsx/jsx.js
   mode/livescript/livescript.js
   mode/sass/sass.js
   mode/sql/sql.js

Modified: trunk/Source/WebInspectorUI/UserInterface/Base/MIMETypeUtilities.js (226908 => 226909)


--- trunk/Source/WebInspectorUI/UserInterface/Base/MIMETypeUtilities.js	2018-01-12 21:01:02 UTC (rev 226908)
+++ trunk/Source/WebInspectorUI/UserInterface/Base/MIMETypeUtilities.js	2018-01-12 21:03:22 UTC (rev 226909)
@@ -54,6 +54,7 @@
         "coffee": "text/x-coffeescript",
         "ls": "text/x-livescript",
         "ts": "text/typescript",
+        "jsx": "text/jsx",
 
         // Stylesheet types.
         "css": "text/css",
@@ -100,6 +101,7 @@
         "text/x-coffeescript": "coffee",
         "text/x-livescript": "ls",
         "text/typescript": "ts",
+        "text/jsx": "jsx",
 
         // Stylesheet types.
         "text/css": "css",

Added: trunk/Source/WebInspectorUI/UserInterface/External/CodeMirror/jsx.js (0 => 226909)


--- trunk/Source/WebInspectorUI/UserInterface/External/CodeMirror/jsx.js	                        (rev 0)
+++ trunk/Source/WebInspectorUI/UserInterface/External/CodeMirror/jsx.js	2018-01-12 21:03:22 UTC (rev 226909)
@@ -0,0 +1,148 @@
+// CodeMirror, copyright (c) by Marijn Haverbeke and others
+// Distributed under an MIT license: http://codemirror.net/LICENSE
+
+(function(mod) {
+  if (typeof exports == "object" && typeof module == "object") // CommonJS
+    mod(require("../../lib/codemirror"), require("../xml/xml"), require("../_javascript_/_javascript_"))
+  else if (typeof define == "function" && define.amd) // AMD
+    define(["../../lib/codemirror", "../xml/xml", "../_javascript_/_javascript_"], mod)
+  else // Plain browser env
+    mod(CodeMirror)
+})(function(CodeMirror) {
+  "use strict"
+
+  // Depth means the amount of open braces in JS context, in XML
+  // context 0 means not in tag, 1 means in tag, and 2 means in tag
+  // and js block comment.
+  function Context(state, mode, depth, prev) {
+    this.state = state; this.mode = mode; this.depth = depth; this.prev = prev
+  }
+
+  function copyContext(context) {
+    return new Context(CodeMirror.copyState(context.mode, context.state),
+                       context.mode,
+                       context.depth,
+                       context.prev && copyContext(context.prev))
+  }
+
+  CodeMirror.defineMode("jsx", function(config, modeConfig) {
+    var xmlMode = CodeMirror.getMode(config, {name: "xml", allowMissing: true, multilineTagIndentPastTag: false, allowMissingTagName: true})
+    var jsMode = CodeMirror.getMode(config, modeConfig && modeConfig.base || "_javascript_")
+
+    function flatXMLIndent(state) {
+      var tagName = state.tagName
+      state.tagName = null
+      var result = xmlMode.indent(state, "")
+      state.tagName = tagName
+      return result
+    }
+
+    function token(stream, state) {
+      if (state.context.mode == xmlMode)
+        return xmlToken(stream, state, state.context)
+      else
+        return jsToken(stream, state, state.context)
+    }
+
+    function xmlToken(stream, state, cx) {
+      if (cx.depth == 2) { // Inside a JS /* */ comment
+        if (stream.match(/^.*?\*\//)) cx.depth = 1
+        else stream.skipToEnd()
+        return "comment"
+      }
+
+      if (stream.peek() == "{") {
+        xmlMode.skipAttribute(cx.state)
+
+        var indent = flatXMLIndent(cx.state), xmlContext = cx.state.context
+        // If JS starts on same line as tag
+        if (xmlContext && stream.match(/^[^>]*>\s*$/, false)) {
+          while (xmlContext.prev && !xmlContext.startOfLine)
+            xmlContext = xmlContext.prev
+          // If tag starts the line, use XML indentation level
+          if (xmlContext.startOfLine) indent -= config.indentUnit
+          // Else use JS indentation level
+          else if (cx.prev.state.lexical) indent = cx.prev.state.lexical.indented
+        // Else if inside of tag
+        } else if (cx.depth == 1) {
+          indent += config.indentUnit
+        }
+
+        state.context = new Context(CodeMirror.startState(jsMode, indent),
+                                    jsMode, 0, state.context)
+        return null
+      }
+
+      if (cx.depth == 1) { // Inside of tag
+        if (stream.peek() == "<") { // Tag inside of tag
+          xmlMode.skipAttribute(cx.state)
+          state.context = new Context(CodeMirror.startState(xmlMode, flatXMLIndent(cx.state)),
+                                      xmlMode, 0, state.context)
+          return null
+        } else if (stream.match("//")) {
+          stream.skipToEnd()
+          return "comment"
+        } else if (stream.match("/*")) {
+          cx.depth = 2
+          return token(stream, state)
+        }
+      }
+
+      var style = xmlMode.token(stream, cx.state), cur = stream.current(), stop
+      if (/\btag\b/.test(style)) {
+        if (/>$/.test(cur)) {
+          if (cx.state.context) cx.depth = 0
+          else state.context = state.context.prev
+        } else if (/^</.test(cur)) {
+          cx.depth = 1
+        }
+      } else if (!style && (stop = cur.indexOf("{")) > -1) {
+        stream.backUp(cur.length - stop)
+      }
+      return style
+    }
+
+    function jsToken(stream, state, cx) {
+      if (stream.peek() == "<" && jsMode.expressionAllowed(stream, cx.state)) {
+        jsMode.skipExpression(cx.state)
+        state.context = new Context(CodeMirror.startState(xmlMode, jsMode.indent(cx.state, "")),
+                                    xmlMode, 0, state.context)
+        return null
+      }
+
+      var style = jsMode.token(stream, cx.state)
+      if (!style && cx.depth != null) {
+        var cur = stream.current()
+        if (cur == "{") {
+          cx.depth++
+        } else if (cur == "}") {
+          if (--cx.depth == 0) state.context = state.context.prev
+        }
+      }
+      return style
+    }
+
+    return {
+      startState: function() {
+        return {context: new Context(CodeMirror.startState(jsMode), jsMode)}
+      },
+
+      copyState: function(state) {
+        return {context: copyContext(state.context)}
+      },
+
+      token: token,
+
+      indent: function(state, textAfter, fullLine) {
+        return state.context.mode.indent(state.context.state, textAfter, fullLine)
+      },
+
+      innerMode: function(state) {
+        return state.context
+      }
+    }
+  }, "xml", "_javascript_")
+
+  CodeMirror.defineMIME("text/jsx", "jsx")
+  CodeMirror.defineMIME("text/typescript-jsx", {name: "jsx", base: {name: "_javascript_", typescript: true}})
+});

Modified: trunk/Source/WebInspectorUI/UserInterface/Main.html (226908 => 226909)


--- trunk/Source/WebInspectorUI/UserInterface/Main.html	2018-01-12 21:01:02 UTC (rev 226908)
+++ trunk/Source/WebInspectorUI/UserInterface/Main.html	2018-01-12 21:03:22 UTC (rev 226909)
@@ -249,6 +249,7 @@
     <script src=""
     <script src=""
     <script src=""
+    <script src=""
     <script src=""
     <script src=""
     <script src=""

Modified: trunk/Source/WebInspectorUI/UserInterface/Models/Resource.js (226908 => 226909)


--- trunk/Source/WebInspectorUI/UserInterface/Models/Resource.js	2018-01-12 21:01:02 UTC (rev 226908)
+++ trunk/Source/WebInspectorUI/UserInterface/Models/Resource.js	2018-01-12 21:03:22 UTC (rev 226909)
@@ -1165,6 +1165,8 @@
     "text/livescript": WI.Resource.Type.Script,
     "text/x-livescript": WI.Resource.Type.Script,
     "text/typescript": WI.Resource.Type.Script,
+    "text/typescript-jsx": WI.Resource.Type.Script,
+    "text/jsx": WI.Resource.Type.Script,
     "text/x-clojure": WI.Resource.Type.Script,
-    "text/x-coffeescript": WI.Resource.Type.Script
+    "text/x-coffeescript": WI.Resource.Type.Script,
 };
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to