Attachment 33179 Details for Bug 27503 – patch

[patch] patch

SyntaxHighlightJSON.patch (text/plain), 1.40 KB, created by Joseph Pecoraro on 2009-07-21 09:05:52 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Joseph Pecoraro

Created: 2009-07-21 09:05:52 PDT

Size: 1.40 KB

patch

obsolete

>diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog
>index 37351ad..c4420f9 100644
>--- a/WebCore/ChangeLog
>+++ b/WebCore/ChangeLog
>@@ -1,3 +1,13 @@
>+2009-07-21  Joseph Pecoraro  <joepeck02@gmail.com>
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Inspector: Should Syntax Highlight JSON
>+        https://bugs.webkit.org/show_bug.cgi?id=27503
>+
>+        * inspector/front-end/SourceView.js:
>+        (WebInspector.SourceView.prototype._contentLoaded):
>+
> 2009-07-20  Adam Langley  <agl@google.com>
> 
>         Reviewed by Eric Seidel.
>diff --git a/WebCore/inspector/front-end/SourceView.js b/WebCore/inspector/front-end/SourceView.js
>index 7510c8c..0b69b89 100644
>--- a/WebCore/inspector/front-end/SourceView.js
>+++ b/WebCore/inspector/front-end/SourceView.js
>@@ -104,8 +104,10 @@ WebInspector.SourceView.prototype = {
>     {
>         delete this._frameNeedsSetup;
>         this.sourceFrame.removeEventListener("content loaded", this._contentLoaded, this);
>-
>-        if (this.resource.type === WebInspector.Resource.Type.Script) {
>+        
>+        if (this.resource.type === WebInspector.Resource.Type.Script
>+            || this.resource.mimeType === 'application/json'
>+            || /\.js(on)?$/.test(this.resource.lastPathComponent) ) {
>             this.sourceFrame.addEventListener("syntax highlighting complete", this._syntaxHighlightingComplete, this);
>             this.sourceFrame.syntaxHighlightJavascript();
>         } else

Actions: View | Formatted Diff | Diff

Attachments on bug 27503: 33179 | 33180 | 33182