- modified the problems panel to support multiple code inspector

src/extensions/default/JSLint/unittests.js

@@ -36,8 +36,9 @@ define(function (require, exports, module) {
             $,
             brackets,
             CodeInspection,
-            EditorManager;
-
+            EditorManager,
+            previousState;
+
         var toggleJSLintResults = function (visible) {
             $("#status-inspection").triggerHandler("click");
             expect($("#problems-panel").is(":visible")).toBe(visible);
@@ -50,9 +51,12 @@ define(function (require, exports, module) {
                     // Load module instances from brackets.test
                     $ = testWindow.$;
                     brackets = testWindow.brackets;
-                    EditorManager = testWindow.brackets.test.EditorManager;
-                    CodeInspection = testWindow.brackets.test.CodeInspection;
+                    EditorManager = brackets.test.EditorManager;
+                    CodeInspection = brackets.test.CodeInspection;
                     CodeInspection.toggleEnabled(true);
+                    // enable JSLint and preserve the previous state before we enable it for testing
+                    previousState = CodeInspection._getProviderState({name: "JSLint"});
+                    CodeInspection._setProviderState({name: "JSLint"}, true);
                 });
             });
 
@@ -67,17 +71,16 @@ define(function (require, exports, module) {
             brackets      = null;
             EditorManager = null;
             SpecRunnerUtils.closeTestWindow();
+
+            // revert to previous state
+            CodeInspection._setProviderState({name: 'JSLint'}, previousState);
         });
 
         it("should run JSLint linter when a JavaScript document opens", function () {
-            runs(function () {
-                spyOn(testWindow, "JSLINT").andCallThrough();
-            });
-
             waitsForDone(SpecRunnerUtils.openProjectFiles(["errors.js"]), "open test file");
 
             runs(function () {
-                expect(testWindow.JSLINT).toHaveBeenCalled();
+                expect($("#problems-panel").is(":visible")).toBe(true);
             });
         });
 
@@ -98,6 +101,5 @@ define(function (require, exports, module) {
                 toggleJSLintResults(false);
             });
         });
-
     });
 });

src/htmlContent/problems-panel-table.html

@@ -1,11 +1,16 @@
 <table class="bottom-panel-table table table-striped table-condensed row-highlight">
     <tbody>
         {{#reportList}}
+        <tr class="inspector-section" data-provider="{{providerName}}">
+            <td colspan="3"><span class="disclosure-triangle expanded"></span>{{providerName}} ({{numProblems}})</td>
+        </tr>
+        {{#results}}
         <tr>
             <td class="line-number" data-character="{{pos.ch}}">{{friendlyLine}}</td>
-            <td>{{message}}</td>
-            <td>{{codeSnippet}}</td>
+            <td class="line-text">{{message}}</td>
+            <td class="line-snippet">{{codeSnippet}}</td>
         </tr>
+        {{/results}}
         {{/reportList}}
     </tbody>
 </table>

src/htmlContent/problems-panel.html

@@ -1,6 +1,6 @@
 <div id="problems-panel" class="bottom-panel vert-resizable top-resizer">
     <div class="toolbar simple-toolbar-layout">
-        <div class="title"><!-- filled in programmatically with active linter's name --></div>
+        <div class="title"><!-- filled in programmatically --></div>
         <a href="#" class="close">&times;</a>
     </div>
     <div class="table-container resizable-content"></div>