Web Inspector: Fix warnings about console.assert lines without semicolons

https://bugs.webkit.org/show_bug.cgi?id=174840

Patch by Joseph Pecoraro <pecoraro@apple.com> on 2017-07-25
Reviewed by Brian Burg.

* UserInterface/Views/SettingsTabContentView.js:
(WebInspector.SettingsTabContentView.prototype.set selectedSettingsView):
(WebInspector.SettingsTabContentView.prototype.setSettingsViewVisible):
Add semicolon to assert lines so they can be stripped in optimized builds.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@219894 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/Source/WebInspectorUI/ChangeLog b/Source/WebInspectorUI/ChangeLog
index af754d1023b57a8c0e4cd2f8d4baca9125b1ffdc..4dfcee467da93db32898e612004158bb00d33fd5 100644 (file)
--- a/Source/WebInspectorUI/ChangeLog
+++ b/Source/WebInspectorUI/ChangeLog
@@ -1,3 +1,15 @@
+2017-07-25  Joseph Pecoraro  <pecoraro@apple.com>
+
+        Web Inspector: Fix warnings about console.assert lines without semicolons
+        https://bugs.webkit.org/show_bug.cgi?id=174840
+
+        Reviewed by Brian Burg.
+
+        * UserInterface/Views/SettingsTabContentView.js:
+        (WebInspector.SettingsTabContentView.prototype.set selectedSettingsView):
+        (WebInspector.SettingsTabContentView.prototype.setSettingsViewVisible):
+        Add semicolon to assert lines so they can be stripped in optimized builds.
+
 2017-07-25  Joseph Pecoraro  <pecoraro@apple.com>
 
         Web Inspector: Inline multiple console log values if they are simple

diff --git a/Source/WebInspectorUI/UserInterface/Views/SettingsTabContentView.js b/Source/WebInspectorUI/UserInterface/Views/SettingsTabContentView.js
index 2ac1e431cc277f878b58fd313aa18596f507439b..cc34f587081b886c7adba7f2e60216e87f8f4fa7 100644 (file)
--- a/Source/WebInspectorUI/UserInterface/Views/SettingsTabContentView.js
+++ b/Source/WebInspectorUI/UserInterface/Views/SettingsTabContentView.js
@@ -80,7 +80,7 @@ WebInspector.SettingsTabContentView = class SettingsTabContentView extends WebIn
         this._selectedSettingsView.updateLayout();
 
         let navigationItem = this._navigationBar.findNavigationItem(settingsView.identifier);
-        console.assert(navigationItem, "Missing navigation item for settings view.", settingsView)
+        console.assert(navigationItem, "Missing navigation item for settings view.", settingsView);
         if (!navigationItem)
             return;
 
@@ -125,7 +125,7 @@ WebInspector.SettingsTabContentView = class SettingsTabContentView extends WebIn
             return;
 
         let index = this._settingsViews.indexOf(settingsView);
-        console.assert(index !== -1, "SettingsView not found.", settingsView)
+        console.assert(index !== -1, "SettingsView not found.", settingsView);
         if (index === -1)
             return;