Title: [235873] trunk/Source/WebInspectorUI
Revision
235873
Author
joep...@webkit.org
Date
2018-09-10 16:47:05 -0700 (Mon, 10 Sep 2018)

Log Message

Web Inspector: Address some ESLint warnings
https://bugs.webkit.org/show_bug.cgi?id=188824

Reviewed by Matt Baker.

* .eslintrc:
* UserInterface/Base/Platform.js:
* UserInterface/Controllers/AuditManager.js:
* UserInterface/Controllers/BasicBlockAnnotator.js:
* UserInterface/Controllers/CallFrameTreeController.js:
* UserInterface/Controllers/HARBuilder.js:
* UserInterface/Models/AuditReport.js:
* UserInterface/Models/AuditResult.js:
* UserInterface/Models/AuditTestCase.js:
* UserInterface/Models/AuditTestSuite.js:
* UserInterface/Models/Cookie.js:
* UserInterface/Models/SourceCodePosition.js:
* UserInterface/Test/FrontendTestHarness.js:
* UserInterface/Test/TestSuite.js:
* UserInterface/Views/CanvasContentView.js:
* UserInterface/Views/CollectionContentView.js:
* UserInterface/Views/ContentBrowser.js:
* UserInterface/Views/CookieStorageContentView.js:
* UserInterface/Views/DataGrid.js:
* UserInterface/Views/DatabaseTableContentView.js:
* UserInterface/Views/FindBanner.js:
* UserInterface/Views/JSONResourceContentView.js:
* UserInterface/Views/LogContentView.js:
* UserInterface/Views/ResourceSidebarPanel.js:
* UserInterface/Views/SourceCodeTextEditor.js:
* UserInterface/Views/SpreadsheetTextField.js:
* UserInterface/Workers/Formatter/FormatterUtilities.js:

Modified Paths

Diff

Modified: trunk/Source/WebInspectorUI/.eslintrc (235872 => 235873)


--- trunk/Source/WebInspectorUI/.eslintrc	2018-09-10 23:24:45 UTC (rev 235872)
+++ trunk/Source/WebInspectorUI/.eslintrc	2018-09-10 23:47:05 UTC (rev 235873)
@@ -54,6 +54,7 @@
         "RecordingAgent": true,
         "RuntimeAgent": true,
         "ScriptProfilerAgent": true,
+        "ServiceWorkerAgent": true,
         "TimelineAgent": true,
         "WorkerAgent": true,
 
@@ -98,6 +99,7 @@
         "CSSPrimitiveValue": true,
         "Exception": true,
         "WebKitPoint": true,
+        "SQLError": true,
 
         // Utilities
         "appendWebInspectorConsoleEvaluationSourceURL": true,
@@ -122,6 +124,7 @@
         "parseMIMEType": true,
         "resolveDotsInPath": true,
         "simpleGlobStringToRegExp": true,
+        "textToBlob": true,
         "timestamp": true,
         "zeroWidthSpace": true,
 

Modified: trunk/Source/WebInspectorUI/ChangeLog (235872 => 235873)


--- trunk/Source/WebInspectorUI/ChangeLog	2018-09-10 23:24:45 UTC (rev 235872)
+++ trunk/Source/WebInspectorUI/ChangeLog	2018-09-10 23:47:05 UTC (rev 235873)
@@ -1,3 +1,38 @@
+2018-09-10  Joseph Pecoraro  <pecor...@apple.com>
+
+        Web Inspector: Address some ESLint warnings
+        https://bugs.webkit.org/show_bug.cgi?id=188824
+
+        Reviewed by Matt Baker.
+
+        * .eslintrc:
+        * UserInterface/Base/Platform.js:
+        * UserInterface/Controllers/AuditManager.js:
+        * UserInterface/Controllers/BasicBlockAnnotator.js:
+        * UserInterface/Controllers/CallFrameTreeController.js:
+        * UserInterface/Controllers/HARBuilder.js:
+        * UserInterface/Models/AuditReport.js:
+        * UserInterface/Models/AuditResult.js:
+        * UserInterface/Models/AuditTestCase.js:
+        * UserInterface/Models/AuditTestSuite.js:
+        * UserInterface/Models/Cookie.js:
+        * UserInterface/Models/SourceCodePosition.js:
+        * UserInterface/Test/FrontendTestHarness.js:
+        * UserInterface/Test/TestSuite.js:
+        * UserInterface/Views/CanvasContentView.js:
+        * UserInterface/Views/CollectionContentView.js:
+        * UserInterface/Views/ContentBrowser.js:
+        * UserInterface/Views/CookieStorageContentView.js:
+        * UserInterface/Views/DataGrid.js:
+        * UserInterface/Views/DatabaseTableContentView.js:
+        * UserInterface/Views/FindBanner.js:
+        * UserInterface/Views/JSONResourceContentView.js:
+        * UserInterface/Views/LogContentView.js:
+        * UserInterface/Views/ResourceSidebarPanel.js:
+        * UserInterface/Views/SourceCodeTextEditor.js:
+        * UserInterface/Views/SpreadsheetTextField.js:
+        * UserInterface/Workers/Formatter/FormatterUtilities.js:
+
 2018-09-06  Andy Estes  <aes...@apple.com>
 
         [Apple Pay] Rename the -apple-pay-button-type value "checkout" to "check-out"

Modified: trunk/Source/WebInspectorUI/UserInterface/Base/Platform.js (235872 => 235873)


--- trunk/Source/WebInspectorUI/UserInterface/Base/Platform.js	2018-09-10 23:24:45 UTC (rev 235872)
+++ trunk/Source/WebInspectorUI/UserInterface/Base/Platform.js	2018-09-10 23:47:05 UTC (rev 235873)
@@ -49,7 +49,7 @@
         WI.Platform.version.base = 10;
         WI.Platform.version.release = parseInt(osVersionMatch[2]);
         switch (osVersionMatch[2]) {
-        case "12":
+        case "13":
             WI.Platform.version.name = "high-sierra";
             break;
         case "12":

Modified: trunk/Source/WebInspectorUI/UserInterface/Controllers/AuditManager.js (235872 => 235873)


--- trunk/Source/WebInspectorUI/UserInterface/Controllers/AuditManager.js	2018-09-10 23:24:45 UTC (rev 235872)
+++ trunk/Source/WebInspectorUI/UserInterface/Controllers/AuditManager.js	2018-09-10 23:47:05 UTC (rev 235873)
@@ -23,7 +23,7 @@
  * THE POSSIBILITY OF SUCH DAMAGE.
  */
 
- WI.AuditManager = class AuditManager extends WI.Object
+WI.AuditManager = class AuditManager extends WI.Object
 {
     constructor()
     {
@@ -31,12 +31,12 @@
 
         this._testSuiteConstructors = [];
         this._reports = new Map;
-        
+
         // Transforming all the constructors into AuditTestSuite instances.
-        this._testSuites = this._testSuiteConstructors.map(suite =>  {
+        this._testSuites = this._testSuiteConstructors.map(suite => {
             let newTestSuite = new suite;
 
-            if (!newTestSuite instanceof WI.AuditTestSuite)
+            if (!(newTestSuite instanceof WI.AuditTestSuite))
                 throw new Error("Audit test suites must be of instance WI.AuditTestSuite.");
 
             return newTestSuite;
@@ -56,11 +56,11 @@
             let auditResult = await this._runTestCase(representedObject);
             auditReport.addResult(auditResult);
         } else if (representedObject instanceof WI.AuditTestSuite) {
-            let testCases = representedObject.testCases;            
+            let testCases = representedObject.testCases;
             // Start reducing from testCases[0].
             let result = testCases.slice(1).reduce((chain, testCase, index) => {
                 if (testCase.setup) {
-                    let setup = testCase.setup.call(testCase, testCase.suite)
+                    let setup = testCase.setup.call(testCase, testCase.suite);
                     if (testCase.setup[Symbol.toStringTag] === "AsyncFunction")
                         return setup;
                     else
@@ -73,7 +73,7 @@
                 });
 
                 if (testCase.tearDown) {
-                    let tearDown = testCase.tearDown.call(testCase, testCase.suite)
+                    let tearDown = testCase.tearDown.call(testCase, testCase.suite);
                     if (testCase.tearDown[Symbol.toStringTag] === "AsyncFunction")
                         return tearDown;
                     else
@@ -131,9 +131,9 @@
         this.dispatchEventToListeners(WI.AuditManager.Event.TestEnded, {test: testCase});
         return new WI.AuditResult(testCase, {result}, didRaiseException);
     }
-}
+};
 
 WI.AuditManager.Event = {
     TestStarted: Symbol("test-started"),
     TestEnded: Symbol("test-ended")
-}
+};

Modified: trunk/Source/WebInspectorUI/UserInterface/Controllers/BasicBlockAnnotator.js (235872 => 235873)


--- trunk/Source/WebInspectorUI/UserInterface/Controllers/BasicBlockAnnotator.js	2018-09-10 23:24:45 UTC (rev 235872)
+++ trunk/Source/WebInspectorUI/UserInterface/Controllers/BasicBlockAnnotator.js	2018-09-10 23:47:05 UTC (rev 235873)
@@ -63,7 +63,7 @@
         while (match) {
             lineEndings.push(match.index);
             lineEndingLengths.push(match[0].length);
-            match = pattern.exec(content)
+            match = pattern.exec(content);
         }
 
         function offsetToPosition(offset) {

Modified: trunk/Source/WebInspectorUI/UserInterface/Controllers/CallFrameTreeController.js (235872 => 235873)


--- trunk/Source/WebInspectorUI/UserInterface/Controllers/CallFrameTreeController.js	2018-09-10 23:24:45 UTC (rev 235872)
+++ trunk/Source/WebInspectorUI/UserInterface/Controllers/CallFrameTreeController.js	2018-09-10 23:47:05 UTC (rev 235873)
@@ -59,7 +59,7 @@
         this._treeOutline.removeChildren();
 
         for (let callFrame of this._callFrames)
-            this._treeOutline.appendChild(new WI.CallFrameTreeElement(callFrame))
+            this._treeOutline.appendChild(new WI.CallFrameTreeElement(callFrame));
     }
 
     disconnect()

Modified: trunk/Source/WebInspectorUI/UserInterface/Controllers/HARBuilder.js (235872 => 235873)


--- trunk/Source/WebInspectorUI/UserInterface/Controllers/HARBuilder.js	2018-09-10 23:24:45 UTC (rev 235872)
+++ trunk/Source/WebInspectorUI/UserInterface/Controllers/HARBuilder.js	2018-09-10 23:47:05 UTC (rev 235873)
@@ -308,4 +308,4 @@
         console.assert(false);
         return undefined;
     }
-}
+};

Modified: trunk/Source/WebInspectorUI/UserInterface/Models/AuditReport.js (235872 => 235873)


--- trunk/Source/WebInspectorUI/UserInterface/Models/AuditReport.js	2018-09-10 23:24:45 UTC (rev 235872)
+++ trunk/Source/WebInspectorUI/UserInterface/Models/AuditReport.js	2018-09-10 23:47:05 UTC (rev 235873)
@@ -23,12 +23,12 @@
  * THE POSSIBILITY OF SUCH DAMAGE.
  */
 
- WI.AuditReport = class AuditReport
+WI.AuditReport = class AuditReport
 {
     constructor(representedTest)
     {
         console.assert(representedTest instanceof WI.AuditTestCase || representedTest instanceof WI.AuditTestSuite);
-        
+
         this._results = [];
         this._isWritable = true;
         this._representedTestCases = (representedTest instanceof WI.AuditTestCase) ? [representedTest] : [...representedTest.testCases];
@@ -59,4 +59,4 @@
     {
         this._isWritable = false;
     }
-}
+};

Modified: trunk/Source/WebInspectorUI/UserInterface/Models/AuditResult.js (235872 => 235873)


--- trunk/Source/WebInspectorUI/UserInterface/Models/AuditResult.js	2018-09-10 23:24:45 UTC (rev 235872)
+++ trunk/Source/WebInspectorUI/UserInterface/Models/AuditResult.js	2018-09-10 23:47:05 UTC (rev 235873)
@@ -23,7 +23,7 @@
  * THE POSSIBILITY OF SUCH DAMAGE.
  */
 
- WI.AuditResult = class AuditResult
+WI.AuditResult = class AuditResult
 {
     constructor(testInstance, testResult, failed)
     {
@@ -43,10 +43,10 @@
     get name() { return this._testName; }
     get logLevel() { return this._logLevel; }
     get failed() { return this._failed; }
-}
+};
 
 WI.AuditResult.LogLevel = {
     Error: "error",
     Warning: "warning",
     Passed: "passed"
-}
+};

Modified: trunk/Source/WebInspectorUI/UserInterface/Models/AuditTestCase.js (235872 => 235873)


--- trunk/Source/WebInspectorUI/UserInterface/Models/AuditTestCase.js	2018-09-10 23:24:45 UTC (rev 235872)
+++ trunk/Source/WebInspectorUI/UserInterface/Models/AuditTestCase.js	2018-09-10 23:47:05 UTC (rev 235873)
@@ -23,13 +23,13 @@
  * THE POSSIBILITY OF SUCH DAMAGE.
  */
 
- WI.AuditTestCase = class AuditTestCase extends WI.Object
+WI.AuditTestCase = class AuditTestCase extends WI.Object
 {
     constructor(suite, name, test, setup, tearDown, errorDetails = {})
     {
         console.assert(suite instanceof WI.AuditTestSuite);
-        console.assert(typeof(name) === "string");
-        
+        console.assert(typeof name === "string");
+
         if (setup)
             console.assert(setup instanceof Function);
 
@@ -41,7 +41,7 @@
 
         super();
         this._id = Symbol(name);
-        
+
         this._suite = suite;
         this._name = name;
         this._test = test;
@@ -59,4 +59,4 @@
     get setup() { return this._setup; }
     get tearDown() { return this._tearDown; }
     get errorDetails() { return this._errorDetails; }
-}
+};

Modified: trunk/Source/WebInspectorUI/UserInterface/Models/AuditTestSuite.js (235872 => 235873)


--- trunk/Source/WebInspectorUI/UserInterface/Models/AuditTestSuite.js	2018-09-10 23:24:45 UTC (rev 235872)
+++ trunk/Source/WebInspectorUI/UserInterface/Models/AuditTestSuite.js	2018-09-10 23:47:05 UTC (rev 235873)
@@ -23,7 +23,7 @@
  * THE POSSIBILITY OF SUCH DAMAGE.
  */
 
- WI.AuditTestSuite = class AuditTestSuite extends WI.Object
+WI.AuditTestSuite = class AuditTestSuite extends WI.Object
 {
     constructor(identifier, name)
     {
@@ -50,12 +50,12 @@
     _buildTestCasesFromDescriptors()
     {
         for (let descriptor of this.constructor.testCaseDescriptors()) {
-            if (typeof(descriptor.name) !== "string" || !descriptor.name)
+            if (typeof descriptor.name !== "string" || !descriptor.name)
                 throw new Error("Test name must be a valid string.");
-            
+
             let {name, test, setup, tearDown, errorDetails} = descriptor;
 
-            if (!test instanceof Function || test[Symbol.toStringTag] !== "AsyncFunction")
+            if (!(test instanceof Function) || test[Symbol.toStringTag] !== "AsyncFunction")
                 throw new Error("Test function must be an async function.");
 
             let testCaseInstance = new WI.AuditTestCase(this, name, test, setup, tearDown, errorDetails);
@@ -63,8 +63,8 @@
             this._testCases.set(testCaseInstance.id, testCaseInstance);
         }
     }
-}
+};
 
 WI.AuditTestSuite.Event = {
     NewAuditResultAvailable: Symbol("new-audit-result-available")
-}   
+};

Modified: trunk/Source/WebInspectorUI/UserInterface/Models/Cookie.js (235872 => 235873)


--- trunk/Source/WebInspectorUI/UserInterface/Models/Cookie.js	2018-09-10 23:24:45 UTC (rev 235872)
+++ trunk/Source/WebInspectorUI/UserInterface/Models/Cookie.js	2018-09-10 23:47:05 UTC (rev 235873)
@@ -214,7 +214,7 @@
 
         return new WI.Cookie(WI.Cookie.Type.Response, name, value, header, expires, maxAge, path, domain, secure, httpOnly, sameSite);
     }
-}
+};
 
 WI.Cookie.Type = {
     Request: "request",

Modified: trunk/Source/WebInspectorUI/UserInterface/Models/SourceCodePosition.js (235872 => 235873)


--- trunk/Source/WebInspectorUI/UserInterface/Models/SourceCodePosition.js	2018-09-10 23:24:45 UTC (rev 235872)
+++ trunk/Source/WebInspectorUI/UserInterface/Models/SourceCodePosition.js	2018-09-10 23:47:05 UTC (rev 235873)
@@ -42,12 +42,6 @@
         return new WI.SourceCodePosition(this._lineNumber, this._columnNumber + delta);
     }
 
-    offsetColumn(delta)
-    {
-        console.assert(this._columnNumber + delta >= 0);
-        return new WI.SourceCodePosition(this._lineNumber, this._columnNumber + delta);
-    }
-
     equals(position)
     {
         return this._lineNumber === position.lineNumber && this._columnNumber === position.columnNumber;

Modified: trunk/Source/WebInspectorUI/UserInterface/Test/FrontendTestHarness.js (235872 => 235873)


--- trunk/Source/WebInspectorUI/UserInterface/Test/FrontendTestHarness.js	2018-09-10 23:24:45 UTC (rev 235872)
+++ trunk/Source/WebInspectorUI/UserInterface/Test/FrontendTestHarness.js	2018-09-10 23:47:05 UTC (rev 235873)
@@ -94,7 +94,7 @@
             return (!remoteObjectOnly && remoteObject.hasValue()) ? remoteObject.value : remoteObject;
         }
 
-        let response = RuntimeAgent.evaluate.invoke({_expression_, objectGroup: "test", includeCommandLineAPI: false})
+        let response = RuntimeAgent.evaluate.invoke({_expression_, objectGroup: "test", includeCommandLineAPI: false});
         if (callback && typeof callback === "function") {
             response = response.then(({result, wasThrown}) => callback(null, translateResult(result), wasThrown));
             response = response.catch((error) => callback(error, null, false));

Modified: trunk/Source/WebInspectorUI/UserInterface/Test/TestSuite.js (235872 => 235873)


--- trunk/Source/WebInspectorUI/UserInterface/Test/TestSuite.js	2018-09-10 23:24:45 UTC (rev 235872)
+++ trunk/Source/WebInspectorUI/UserInterface/Test/TestSuite.js	2018-09-10 23:47:05 UTC (rev 235873)
@@ -182,7 +182,7 @@
     addTestCase(testcase)
     {
         if ([testcase.setup, testcase.teardown, testcase.test].some((fn) => fn && fn[Symbol.toStringTag] === "AsyncFunction"))
-            throw new Error("Tried to pass a test case with an async `setup`, `test`, or `teardown` function, but this is a synchronous test suite.")
+            throw new Error("Tried to pass a test case with an async `setup`, `test`, or `teardown` function, but this is a synchronous test suite.");
 
         super.addTestCase(testcase);
     }

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/CanvasContentView.js (235872 => 235873)


--- trunk/Source/WebInspectorUI/UserInterface/Views/CanvasContentView.js	2018-09-10 23:24:45 UTC (rev 235872)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/CanvasContentView.js	2018-09-10 23:47:05 UTC (rev 235873)
@@ -410,7 +410,7 @@
             let formatString = frameCount === 1 ? WI.UIString("%d Frame") : WI.UIString("%d Frames");
             title = formatString.format(frameCount);
         } else
-            title = WI.UIString("Waiting for frames…")
+            title = WI.UIString("Waiting for frames…");
 
         this._progressView.title = title;
         this._progressView.subtitle = bufferUsed ? Number.bytesToString(bufferUsed) : "";

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/CollectionContentView.js (235872 => 235873)


--- trunk/Source/WebInspectorUI/UserInterface/Views/CollectionContentView.js	2018-09-10 23:24:45 UTC (rev 235872)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/CollectionContentView.js	2018-09-10 23:47:05 UTC (rev 235873)
@@ -275,7 +275,7 @@
             if (typeof this._contentPlaceholder === "string")
                 this._contentPlaceholderElement = WI.createMessageTextView(this._contentPlaceholder);
             else if (this._contentPlaceholder instanceof HTMLElement)
-                this._contentPlaceholderElement =  this._contentPlaceholder;
+                this._contentPlaceholderElement = this._contentPlaceholder;
         }
 
         if (!this._contentPlaceholderElement.parentNode)

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/ContentBrowser.js (235872 => 235873)


--- trunk/Source/WebInspectorUI/UserInterface/Views/ContentBrowser.js	2018-09-10 23:24:45 UTC (rev 235872)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/ContentBrowser.js	2018-09-10 23:47:05 UTC (rev 235873)
@@ -230,7 +230,7 @@
     shown()
     {
         this._updateContentViewSelectionPathNavigationItem(this.currentContentView);
-        this.updateHierarchicalPathForCurrentContentView()
+        this.updateHierarchicalPathForCurrentContentView();
 
         this._contentViewContainer.shown();
     }

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/CookieStorageContentView.js (235872 => 235873)


--- trunk/Source/WebInspectorUI/UserInterface/Views/CookieStorageContentView.js	2018-09-10 23:24:45 UTC (rev 235872)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/CookieStorageContentView.js	2018-09-10 23:47:05 UTC (rev 235873)
@@ -73,7 +73,7 @@
     {
         // FIXME <https://webkit.org/b/151400>: If there are no cookies, add placeholder explanatory text.
         if (!this._dataGrid) {
-            var columns = {name: {}, value: {}, domain: {}, path: {}, expires: {}, size: {}, http: {}, secure: {}, sameSite: {} };
+            var columns = {name: {}, value: {}, domain: {}, path: {}, expires: {}, size: {}, http: {}, secure: {}, sameSite: {}};
 
             columns.name.title = WI.UIString("Name");
             columns.name.sortable = true;

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/DataGrid.js (235872 => 235873)


--- trunk/Source/WebInspectorUI/UserInterface/Views/DataGrid.js	2018-09-10 23:24:45 UTC (rev 235872)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/DataGrid.js	2018-09-10 23:47:05 UTC (rev 235873)
@@ -1573,7 +1573,6 @@
         if (!gridNode) {
             if (this.selectedNode)
                 this.selectedNode.deselect();
-            
             return;
         }
 

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/DatabaseTableContentView.js (235872 => 235873)


--- trunk/Source/WebInspectorUI/UserInterface/Views/DatabaseTableContentView.js	2018-09-10 23:24:45 UTC (rev 235872)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/DatabaseTableContentView.js	2018-09-10 23:47:05 UTC (rev 235873)
@@ -69,7 +69,7 @@
 
     _escapeTableName(name)
     {
-        return name.replace(/\"/g, "\"\"");
+        return name.replace(/"/g, "\"\"");
     }
 
     _queryFinished(columnNames, values)

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/FindBanner.js (235872 => 235873)


--- trunk/Source/WebInspectorUI/UserInterface/Views/FindBanner.js	2018-09-10 23:24:45 UTC (rev 235872)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/FindBanner.js	2018-09-10 23:47:05 UTC (rev 235873)
@@ -53,7 +53,7 @@
         this._previousResultButton = document.createElement("button");
         this._previousResultButton.classList.add("segmented", "previous-result");
         this._previousResultButton.disabled = true;
-        this._previousResultButton.title = WI.UIString("Find Previous (%s)").format(WI.findPreviousKeyboardShortcut.displayName);;
+        this._previousResultButton.title = WI.UIString("Find Previous (%s)").format(WI.findPreviousKeyboardShortcut.displayName);
         this._previousResultButton.addEventListener("click", this._previousResultButtonClicked.bind(this));
         this.element.appendChild(this._previousResultButton);
 
@@ -64,7 +64,7 @@
         this._nextResultButton = document.createElement("button");
         this._nextResultButton.classList.add("segmented", "next-result");
         this._nextResultButton.disabled = true;
-        this._nextResultButton.title = WI.UIString("Find Next (%s)").format(WI.findNextKeyboardShortcut.displayName);;
+        this._nextResultButton.title = WI.UIString("Find Next (%s)").format(WI.findNextKeyboardShortcut.displayName);
         this._nextResultButton.addEventListener("click", this._nextResultButtonClicked.bind(this));
         this.element.appendChild(this._nextResultButton);
 

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/JSONResourceContentView.js (235872 => 235873)


--- trunk/Source/WebInspectorUI/UserInterface/Views/JSONResourceContentView.js	2018-09-10 23:24:45 UTC (rev 235872)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/JSONResourceContentView.js	2018-09-10 23:47:05 UTC (rev 235873)
@@ -56,7 +56,7 @@
             doNotPauseOnExceptionsAndMuteConsole: true,
             contextId: undefined,
             returnByValue: false,
-            generatePreview: true,    
+            generatePreview: true,
         };
         this.resource.target.RuntimeAgent.evaluate.invoke(options, (error, result, wasThrown) => {
             if (error || wasThrown) {
@@ -80,6 +80,6 @@
         if (this._remoteObject) {
             this._remoteObject.release();
             this._remoteObject = null;
-        }   
+        }
     }
 };

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/LogContentView.js (235872 => 235873)


--- trunk/Source/WebInspectorUI/UserInterface/Views/LogContentView.js	2018-09-10 23:24:45 UTC (rev 235872)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/LogContentView.js	2018-09-10 23:47:05 UTC (rev 235873)
@@ -79,7 +79,7 @@
             new WI.ScopeBarItem(WI.LogContentView.Scopes.Warnings, WI.UIString("Warnings"), false, "warnings"),
             new WI.ScopeBarItem(WI.LogContentView.Scopes.Logs, WI.UIString("Logs"), false, "logs"),
             new WI.ScopeBarItem(WI.LogContentView.Scopes.Infos, WI.UIString("Infos"), false, "infos", true),
-            new WI.ScopeBarItem(WI.LogContentView.Scopes.Debugs, WI.UIString("Debugs"), false, "debugs", true),            
+            new WI.ScopeBarItem(WI.LogContentView.Scopes.Debugs, WI.UIString("Debugs"), false, "debugs", true),
         ];
 
         this._scopeBar = new WI.ScopeBar("log-scope-bar", scopeBarItems, scopeBarItems[0]);

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/ResourceSidebarPanel.js (235872 => 235873)


--- trunk/Source/WebInspectorUI/UserInterface/Views/ResourceSidebarPanel.js	2018-09-10 23:24:45 UTC (rev 235872)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/ResourceSidebarPanel.js	2018-09-10 23:47:05 UTC (rev 235873)
@@ -317,7 +317,7 @@
                     this._addTargetWithMainResource(script.target);
                 this.contentTreeOutline.disclosureButtons = true;
                 return;
-            }            
+            }
         }
 
         // If the script URL matches a resource we can assume it is part of that resource and does not need added.

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/SourceCodeTextEditor.js (235872 => 235873)


--- trunk/Source/WebInspectorUI/UserInterface/Views/SourceCodeTextEditor.js	2018-09-10 23:24:45 UTC (rev 235872)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/SourceCodeTextEditor.js	2018-09-10 23:47:05 UTC (rev 235873)
@@ -1432,7 +1432,7 @@
             startPosition = fromInlineScriptPosition(startPosition).toCodeMirror();
             endPosition = fromInlineScriptPosition(endPosition).toCodeMirror();
             callback({startPosition, endPosition});
-        }
+        };
 
         script.requestScriptSyntaxTree((syntaxTree) => {
             // Convert to the position within the inline script before querying the AST.

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/SpreadsheetTextField.js (235872 => 235873)


--- trunk/Source/WebInspectorUI/UserInterface/Views/SpreadsheetTextField.js	2018-09-10 23:24:45 UTC (rev 235872)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/SpreadsheetTextField.js	2018-09-10 23:47:05 UTC (rev 235873)
@@ -396,7 +396,7 @@
         let selection = window.getSelection();
 
         let isHidden = (clientRect) => {
-            return clientRect.x === 0 && clientRect.y === 0
+            return clientRect.x === 0 && clientRect.y === 0;
         };
 
         if (selection.rangeCount) {

Modified: trunk/Source/WebInspectorUI/UserInterface/Workers/Formatter/FormatterUtilities.js (235872 => 235873)


--- trunk/Source/WebInspectorUI/UserInterface/Workers/Formatter/FormatterUtilities.js	2018-09-10 23:24:45 UTC (rev 235872)
+++ trunk/Source/WebInspectorUI/UserInterface/Workers/Formatter/FormatterUtilities.js	2018-09-10 23:47:05 UTC (rev 235873)
@@ -42,7 +42,7 @@
         let match = pattern.exec(this);
         while (match) {
             lineEndings.push(match.index);
-            match = pattern.exec(this)
+            match = pattern.exec(this);
         }
 
         lineEndings.push(this.length);
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to