Title: [205425] trunk/Source/WebInspectorUI
Revision
205425
Author
joep...@webkit.org
Date
2016-09-04 22:02:23 -0700 (Sun, 04 Sep 2016)

Log Message

Web Inspector: Run ESLint over UserInterface/Views
https://bugs.webkit.org/show_bug.cgi?id=161564

Reviewed by Brian Burg.

* .eslintrc:
* UserInterface/Views/BoxModelDetailsSectionRow.js:
* UserInterface/Views/ButtonNavigationItem.js:
* UserInterface/Views/CSSStyleDeclarationSection.js:
* UserInterface/Views/CSSStyleDeclarationTextEditor.js:
* UserInterface/Views/CodeMirrorAdditions.js:
* UserInterface/Views/CodeMirrorEditor.js:
* UserInterface/Views/CodeMirrorFormatters.js:
* UserInterface/Views/ColorWheel.js:
* UserInterface/Views/ComputedStyleDetailsPanel.js:
* UserInterface/Views/ConsoleMessageView.js:
* UserInterface/Views/ContentViewContainer.js:
* UserInterface/Views/CookieStorageContentView.js:
* UserInterface/Views/DOMNodeDetailsSidebarPanel.js:
* UserInterface/Views/DOMStorageContentView.js:
* UserInterface/Views/DOMTreeElement.js:
* UserInterface/Views/DOMTreeOutline.js:
* UserInterface/Views/DashboardContainerView.js:
* UserInterface/Views/DataGrid.js:
* UserInterface/Views/DataGridNode.js:
* UserInterface/Views/DebuggerSidebarPanel.js:
* UserInterface/Views/DefaultDashboardView.js:
* UserInterface/Views/FontResourceContentView.js:
* UserInterface/Views/FormattedValue.js:
* UserInterface/Views/GradientEditor.js:
* UserInterface/Views/GradientSlider.js:
* UserInterface/Views/HeapSnapshotContentView.js:
* UserInterface/Views/HeapSnapshotDataGridTree.js:
* UserInterface/Views/HeapSnapshotInstanceDataGridNode.js:
* UserInterface/Views/HoverMenu.js:
* UserInterface/Views/LogContentView.js:
* UserInterface/Views/MemoryTimelineOverviewGraph.js:
* UserInterface/Views/MemoryTimelineView.js:
* UserInterface/Views/NavigationSidebarPanel.js:
* UserInterface/Views/NetworkTimelineOverviewGraph.js:
* UserInterface/Views/NewTabContentView.js:
* UserInterface/Views/ObjectPreviewView.js:
* UserInterface/Views/ObjectTreeBaseTreeElement.js:
* UserInterface/Views/ObjectTreeMapEntryTreeElement.js:
* UserInterface/Views/OverviewTimelineView.js:
* UserInterface/Views/Popover.js:
* UserInterface/Views/ProbeSetDetailsSection.js:
* UserInterface/Views/ProfileDataGridNode.js:
* UserInterface/Views/ProfileDataGridTree.js:
* UserInterface/Views/ProfileView.js:
* UserInterface/Views/ResourceSidebarPanel.js:
* UserInterface/Views/RulesStyleDetailsPanel.js:
* UserInterface/Views/ScopeRadioButtonNavigationItem.js:
* UserInterface/Views/ScriptClusterTimelineView.js:
* UserInterface/Views/ScriptDetailsTimelineView.js:
* UserInterface/Views/ScriptProfileTimelineView.js:
* UserInterface/Views/StackedLineChart.js:
* UserInterface/Views/StorageSidebarPanel.js:
* UserInterface/Views/TabBrowser.js:
* UserInterface/Views/TextEditor.js:
* UserInterface/Views/TimelineDataGridNode.js:
* UserInterface/Views/TimelineOverview.js:
* UserInterface/Views/TimelineRecordFrame.js:
* UserInterface/Views/TimelineRecordingContentView.js:
* UserInterface/Views/TimelineTabContentView.js:
* UserInterface/Views/TreeOutline.js:
* UserInterface/Views/VisualStyleCommaSeparatedKeywordEditor.js:
* UserInterface/Views/VisualStylePropertyEditor.js:
* UserInterface/Views/VisualStyleSelectorSection.js:

Modified Paths

Diff

Modified: trunk/Source/WebInspectorUI/.eslintrc (205424 => 205425)


--- trunk/Source/WebInspectorUI/.eslintrc	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/.eslintrc	2016-09-05 05:02:23 UTC (rev 205425)
@@ -101,6 +101,7 @@
         "appendWebInspectorConsoleEvaluationSourceURL": true,
         "timestamp": true,
         "handlePromiseException": true,
+        "insertObjectIntoSortedArray": true,
 
         // Image Utilities
         "platformImagePath": true,

Modified: trunk/Source/WebInspectorUI/ChangeLog (205424 => 205425)


--- trunk/Source/WebInspectorUI/ChangeLog	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/ChangeLog	2016-09-05 05:02:23 UTC (rev 205425)
@@ -1,5 +1,77 @@
 2016-09-04  Joseph Pecoraro  <pecor...@apple.com>
 
+        Web Inspector: Run ESLint over UserInterface/Views
+        https://bugs.webkit.org/show_bug.cgi?id=161564
+
+        Reviewed by Brian Burg.
+
+        * .eslintrc:
+        * UserInterface/Views/BoxModelDetailsSectionRow.js:
+        * UserInterface/Views/ButtonNavigationItem.js:
+        * UserInterface/Views/CSSStyleDeclarationSection.js:
+        * UserInterface/Views/CSSStyleDeclarationTextEditor.js:
+        * UserInterface/Views/CodeMirrorAdditions.js:
+        * UserInterface/Views/CodeMirrorEditor.js:
+        * UserInterface/Views/CodeMirrorFormatters.js:
+        * UserInterface/Views/ColorWheel.js:
+        * UserInterface/Views/ComputedStyleDetailsPanel.js:
+        * UserInterface/Views/ConsoleMessageView.js:
+        * UserInterface/Views/ContentViewContainer.js:
+        * UserInterface/Views/CookieStorageContentView.js:
+        * UserInterface/Views/DOMNodeDetailsSidebarPanel.js:
+        * UserInterface/Views/DOMStorageContentView.js:
+        * UserInterface/Views/DOMTreeElement.js:
+        * UserInterface/Views/DOMTreeOutline.js:
+        * UserInterface/Views/DashboardContainerView.js:
+        * UserInterface/Views/DataGrid.js:
+        * UserInterface/Views/DataGridNode.js:
+        * UserInterface/Views/DebuggerSidebarPanel.js:
+        * UserInterface/Views/DefaultDashboardView.js:
+        * UserInterface/Views/FontResourceContentView.js:
+        * UserInterface/Views/FormattedValue.js:
+        * UserInterface/Views/GradientEditor.js:
+        * UserInterface/Views/GradientSlider.js:
+        * UserInterface/Views/HeapSnapshotContentView.js:
+        * UserInterface/Views/HeapSnapshotDataGridTree.js:
+        * UserInterface/Views/HeapSnapshotInstanceDataGridNode.js:
+        * UserInterface/Views/HoverMenu.js:
+        * UserInterface/Views/LogContentView.js:
+        * UserInterface/Views/MemoryTimelineOverviewGraph.js:
+        * UserInterface/Views/MemoryTimelineView.js:
+        * UserInterface/Views/NavigationSidebarPanel.js:
+        * UserInterface/Views/NetworkTimelineOverviewGraph.js:
+        * UserInterface/Views/NewTabContentView.js:
+        * UserInterface/Views/ObjectPreviewView.js:
+        * UserInterface/Views/ObjectTreeBaseTreeElement.js:
+        * UserInterface/Views/ObjectTreeMapEntryTreeElement.js:
+        * UserInterface/Views/OverviewTimelineView.js:
+        * UserInterface/Views/Popover.js:
+        * UserInterface/Views/ProbeSetDetailsSection.js:
+        * UserInterface/Views/ProfileDataGridNode.js:
+        * UserInterface/Views/ProfileDataGridTree.js:
+        * UserInterface/Views/ProfileView.js:
+        * UserInterface/Views/ResourceSidebarPanel.js:
+        * UserInterface/Views/RulesStyleDetailsPanel.js:
+        * UserInterface/Views/ScopeRadioButtonNavigationItem.js:
+        * UserInterface/Views/ScriptClusterTimelineView.js:
+        * UserInterface/Views/ScriptDetailsTimelineView.js:
+        * UserInterface/Views/ScriptProfileTimelineView.js:
+        * UserInterface/Views/StackedLineChart.js:
+        * UserInterface/Views/StorageSidebarPanel.js:
+        * UserInterface/Views/TabBrowser.js:
+        * UserInterface/Views/TextEditor.js:
+        * UserInterface/Views/TimelineDataGridNode.js:
+        * UserInterface/Views/TimelineOverview.js:
+        * UserInterface/Views/TimelineRecordFrame.js:
+        * UserInterface/Views/TimelineRecordingContentView.js:
+        * UserInterface/Views/TimelineTabContentView.js:
+        * UserInterface/Views/TreeOutline.js:
+        * UserInterface/Views/VisualStyleCommaSeparatedKeywordEditor.js:
+        * UserInterface/Views/VisualStylePropertyEditor.js:
+        * UserInterface/Views/VisualStyleSelectorSection.js:
+
+2016-09-04  Joseph Pecoraro  <pecor...@apple.com>
+
         Web Inspector: Update ESLint rules and run it over most of UserInterface
         https://bugs.webkit.org/show_bug.cgi?id=161561
 

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/BoxModelDetailsSectionRow.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/BoxModelDetailsSectionRow.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/BoxModelDetailsSectionRow.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -113,7 +113,7 @@
         {
             // Check if the value is a float and whether it should be rounded.
             let floatValue = parseFloat(value);
-            let shouldRoundValue = (!isNaN(floatValue) && floatValue % 1 !== 0);
+            let shouldRoundValue = !isNaN(floatValue) && (floatValue % 1 !== 0);
 
             if (isNaN(floatValue))
                 value = figureDash;
@@ -222,7 +222,7 @@
                 var heightElement = createContentAreaHeightElement.call(this, style);
                 boxElement.append(widthElement, " \u00D7 ", heightElement);
             } else {
-                var suffix = (name === "border" ? "-width" : "");
+                var suffix = name === "border" ? "-width" : "";
 
                 var labelElement = document.createElement("div");
                 labelElement.className = "label";
@@ -276,7 +276,7 @@
 
     _alteredFloatNumber(number, event)
     {
-        var arrowKeyPressed = (event.keyIdentifier === "Up" || event.keyIdentifier === "Down");
+        var arrowKeyPressed = event.keyIdentifier === "Up" || event.keyIdentifier === "Down";
 
         // Jump by 10 when shift is down or jump by 0.1 when Alt/Option is down.
         // Also jump by 10 for page up and down, or by 100 if shift is held with a page key.

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/ButtonNavigationItem.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/ButtonNavigationItem.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/ButtonNavigationItem.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -130,7 +130,7 @@
             return;
         this.dispatchEventToListeners(WebInspector.ButtonNavigationItem.Event.Clicked);
     }
-}
+};
 
 WebInspector.ButtonNavigationItem.DisabledStyleClassName = "disabled";
 WebInspector.ButtonNavigationItem.TextOnlyClassName = "text-only";

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/CSSStyleDeclarationSection.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/CSSStyleDeclarationSection.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/CSSStyleDeclarationSection.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -390,7 +390,7 @@
         var selectorText = this._selectorElement.textContent;
         if (!selectorText || !selectorText.length) {
             if (!this._style.ownerRule)
-                return;
+                return "";
 
             selectorText = this._style.ownerRule.selectorText;
         }

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/CSSStyleDeclarationTextEditor.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/CSSStyleDeclarationTextEditor.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/CSSStyleDeclarationTextEditor.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -270,7 +270,7 @@
         for (var property of propertiesList) {
             if (property.__filterResultClassName) {
                 property.__filterResultClassName = null;
-                this._updateTextMarkerForPropertyIfNeeded(property)
+                this._updateTextMarkerForPropertyIfNeeded(property);
             }
         }
     }
@@ -567,7 +567,7 @@
                     if (trimmedPreviousLine.includes(";"))
                         previousHead = trimmedPreviousLine.lastIndexOf(";");
                 }
-                
+
                 codeMirror.setSelection({line: cursor.line, ch: previousAnchor}, {line: cursor.line, ch: previousHead});
                 return;
             }

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/CodeMirrorAdditions.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/CodeMirrorAdditions.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/CodeMirrorAdditions.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -487,7 +487,7 @@
             // selectionStart/End may the same object if there is no selection. If that is the case
             // make only one modification to prevent a double adjustment, and keep it a single object
             // to avoid CodeMirror inadvertently creating an actual selection range.
-            let diff = (newLength - previousLength);
+            let diff = newLength - previousLength;
             if (selectionStart === selectionEnd)
                 selectionStart.ch += diff;
             else {
@@ -632,7 +632,7 @@
 
     let lineCount = cm.lineCount();
     let abort = false;
-    for (lineNumber = initialPosition.line; !abort && lineNumber < lineCount; ++lineNumber) {
+    for (let lineNumber = initialPosition.line; !abort && lineNumber < lineCount; ++lineNumber) {
         let line = cm.getLine(lineNumber);
         let stream = new CodeMirror.StringStream(line);
         if (lineNumber === initialPosition.line)
@@ -639,7 +639,6 @@
             stream.start = stream.pos = initialPosition.ch;
 
         while (!stream.eol()) {
-            let innerMode = CodeMirror.innerMode(mode, state);
             let tokenType = mode.token(stream, state);
             if (!callback(tokenType, stream.current())) {
                 abort = true;
@@ -651,4 +650,4 @@
 
     if (!abort)
         callback(null);
-}
+};

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/CodeMirrorEditor.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/CodeMirrorEditor.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/CodeMirrorEditor.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -46,4 +46,4 @@
 
         return codeMirror;
     }
-}
+};

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/CodeMirrorFormatters.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/CodeMirrorFormatters.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/CodeMirrorFormatters.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -35,7 +35,7 @@
             if (content === "(") // Most keywords like "if (" but not "function(" or "typeof(".
                 return lastToken && /\bkeyword\b/.test(lastToken) && (lastContent !== "function" && lastContent !== "typeof" && lastContent !== "instanceof");
             if (content === ":") // Ternary.
-                return (state.lexical.type === "stat" || state.lexical.type === ")" || state.lexical.type === "]");
+                return state.lexical.type === "stat" || state.lexical.type === ")" || state.lexical.type === "]";
             return false;
         }
 

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/ColorWheel.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/ColorWheel.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/ColorWheel.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -203,7 +203,6 @@
         var ctx = this._rawCanvas.getContext("2d");
 
         var dimension = this._dimension * window.devicePixelRatio;
-        var center = dimension / 2;
 
         ctx.fillStyle = "white";
         ctx.fillRect(0, 0, dimension, dimension);

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/ComputedStyleDetailsPanel.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/ComputedStyleDetailsPanel.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/ComputedStyleDetailsPanel.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -30,7 +30,7 @@
         super(delegate, WebInspector.ComputedStyleDetailsPanel.StyleClassName, "computed", WebInspector.UIString("Styles \u2014 Computed"));
 
         this._computedStyleShowAllSetting = new WebInspector.Setting("computed-style-show-all", false);
-        
+
         this.cssStyleDeclarationTextEditorShouldAddPropertyGoToArrows = true;
     }
 

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/ConsoleMessageView.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/ConsoleMessageView.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/ConsoleMessageView.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -322,7 +322,7 @@
             if (this._message.url) {
                 var anchor = WebInspector.linkifyURLAsNode(this._message.url, this._message.url, "console-message-url");
                 anchor.classList.add("console-message-location");
-                this._element.appendChild(anchor);                
+                this._element.appendChild(anchor);
             }
             return;
         }

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/ContentViewContainer.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/ContentViewContainer.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/ContentViewContainer.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -184,7 +184,7 @@
         if (newContentView.parentContainer && newContentView.parentContainer !== this)
             return;
 
-        var currentlyShowing = (this.currentContentView === oldContentView);
+        var currentlyShowing = this.currentContentView === oldContentView;
         if (currentlyShowing)
             this._hideEntry(this.currentBackForwardEntry);
 
@@ -492,7 +492,7 @@
         }
 
         if (!this.subviews.includes(entry.contentView))
-            this.addSubview(entry.contentView)
+            this.addSubview(entry.contentView);
 
         entry.prepareToShow(shouldCallShown);
     }
@@ -508,7 +508,7 @@
 
         entry.prepareToHide();
         if (this.subviews.includes(entry.contentView))
-            this.removeSubview(entry.contentView)
+            this.removeSubview(entry.contentView);
     }
 
     _tombstoneContentViewContainersForContentView(contentView)

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/CookieStorageContentView.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/CookieStorageContentView.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/CookieStorageContentView.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -154,7 +154,7 @@
         let resourceMatchesStorageDomain = (resource) => {
             let urlComponents = resource.urlComponents;
             return urlComponents && urlComponents.host && urlComponents.host === this.representedObject.host;
-        }
+        };
 
         let allResources = [];
         for (let frame of WebInspector.frameResourceManager.frames) {

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/DOMNodeDetailsSidebarPanel.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/DOMNodeDetailsSidebarPanel.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/DOMNodeDetailsSidebarPanel.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -302,7 +302,7 @@
         }
 
         function linkListForNodeIds(nodeIds) {
-            if (!nodeIds) 
+            if (!nodeIds)
                 return null;
 
             const itemsToShow = 5;
@@ -309,8 +309,8 @@
             let hasLinks = false;
             let listItemCount = 0;
             let container = document.createElement("div");
-            container.classList.add("list-container")
-            let linkList = container.createChild("ul", "node-link-list");            
+            container.classList.add("list-container");
+            let linkList = container.createChild("ul", "node-link-list");
             let initiallyHiddenItems = [];
             for (let nodeId of nodeIds) {
                 let node = WebInspector.domTreeManager.nodeForId(nodeId);
@@ -320,10 +320,10 @@
                 hasLinks = true;
                 let li = linkList.createChild("li");
                 li.appendChild(link);
-                if (listItemCount >= itemsToShow) {  
+                if (listItemCount >= itemsToShow) {
                     li.hidden = true;
                     initiallyHiddenItems.push(li);
-                } 
+                }
                 listItemCount++;
             }
             container.appendChild(linkList);
@@ -331,11 +331,11 @@
                 let moreNodesButton = container.createChild("button", "expand-list-button");
                 moreNodesButton.textContent = WebInspector.UIString("%d More\u2026").format(listItemCount - itemsToShow);
                 moreNodesButton.addEventListener("click", () => {
-                    initiallyHiddenItems.forEach((element) => element.hidden = false);
+                    initiallyHiddenItems.forEach((element) => { element.hidden = false; });
                     moreNodesButton.remove();
                 });
             }
-            if (hasLinks) 
+            if (hasLinks)
                 return container;
 
             return null;
@@ -365,9 +365,8 @@
 
                 // Accessibility tree children are not a 1:1 mapping with DOM tree children.
                 var childNodeLinkList = linkListForNodeIds(accessibilityProperties.childNodeIds);
-                
                 var controlledNodeLinkList = linkListForNodeIds(accessibilityProperties.controlledNodeIds);
-                
+
                 var current = "";
                 switch (accessibilityProperties.current) {
                 case DOMAgent.AccessibilityPropertiesCurrent.True:
@@ -391,12 +390,12 @@
                 default:
                     current = "";
                 }
-                
+
                 var disabled = booleanValueToLocalizedStringIfTrue("disabled");
                 var expanded = booleanValueToLocalizedStringIfPropertyDefined("expanded");
                 var flowedNodeLinkList = linkListForNodeIds(accessibilityProperties.flowedNodeIds);
                 var focused = booleanValueToLocalizedStringIfPropertyDefined("focused");
-                
+
                 var ignored = "";
                 if (accessibilityProperties.ignored) {
                     ignored = WebInspector.UIString("Yes");
@@ -419,7 +418,7 @@
                 var liveRegionStatus = "";
                 var liveRegionStatusNode = null;
                 var liveRegionStatusToken = accessibilityProperties.liveRegionStatus;
-                switch(liveRegionStatusToken) {
+                switch (liveRegionStatusToken) {
                 case DOMAgent.AccessibilityPropertiesLiveRegionStatus.Assertive:
                     liveRegionStatus = WebInspector.UIString("Assertive");
                     break;
@@ -435,7 +434,7 @@
                     if (liveRegionRelevant && liveRegionRelevant.length) {
                         // @aria-relevant="all" is exposed as ["additions","removals","text"], in order.
                         // This order is controlled in WebCore and expected in WebInspectorUI.
-                        if (liveRegionRelevant.length === 3 
+                        if (liveRegionRelevant.length === 3
                             && liveRegionRelevant[0] === DOMAgent.LiveRegionRelevant.Additions
                             && liveRegionRelevant[1] === DOMAgent.LiveRegionRelevant.Removals
                             && liveRegionRelevant[2] === DOMAgent.LiveRegionRelevant.Text)
@@ -517,7 +516,7 @@
                 this._accessibilityNodeInvalidRow.value = invalid;
                 this._accessibilityNodeLabelRow.value = label;
                 this._accessibilityNodeLiveRegionStatusRow.value = liveRegionStatusNode || liveRegionStatus;
-                
+
                 // Row label changes based on whether the value is a delegate node link.
                 this._accessibilityNodeMouseEventRow.label = mouseEventNodeLink ? WebInspector.UIString("Click Listener") : WebInspector.UIString("Clickable");
                 this._accessibilityNodeMouseEventRow.value = mouseEventNodeLink || mouseEventTextValue;
@@ -533,7 +532,7 @@
                 this._accessibilityNodeSelectedChildrenRow.label = WebInspector.UIString("Selected Items");
                 this._accessibilityNodeSelectedChildrenRow.value = selectedChildNodeLinkList || "";
                 if (selectedChildNodeLinkList && accessibilityProperties.selectedChildNodeIds.length === 1)
-                    this._accessibilityNodeSelectedChildrenRow.label = WebInspector.UIString("Selected Item");                
+                    this._accessibilityNodeSelectedChildrenRow.label = WebInspector.UIString("Selected Item");
 
                 // Display order, not alphabetical as above.
                 this._accessibilityGroup.rows = [

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/DOMStorageContentView.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/DOMStorageContentView.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/DOMStorageContentView.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -78,6 +78,8 @@
             if (node.data.key === event.data.key)
                 return this._dataGrid.removeChild(node);
         }
+
+        return null;
     }
 
     itemAdded(event)
@@ -199,14 +201,6 @@
             editingNode.__originalValue = undefined;
         }
 
-        function restoreOriginalValues()
-        {
-            editingNode.data.key = editingNode.__originalKey;
-            editingNode.data.value = editingNode.__originalValue;
-            editingNode.refresh();
-            cleanup();
-        }
-
         // If the key/value field was cleared, add "missing" style.
         if (isEditingKey) {
             if (key.length)
@@ -243,7 +237,7 @@
         if (!key.length && !value.length && !editingNode.isPlaceholderNode) {
             this._dataGrid.removeChild(editingNode);
             domStorage.removeItem(editingNode.__originalKey);
-            return;                
+            return;
         }
 
         // Done editing but leaving the row in an invalid state. Leave in uncommitted state.

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/DOMTreeElement.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/DOMTreeElement.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/DOMTreeElement.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -398,7 +398,7 @@
         // Remove any tree elements that no longer have this node (or this node's contentDocument) as their parent.
         // Keep a list of existing tree elements for nodes that we can use later.
         var existingChildTreeElements = new Map;
-        for (var i = (this.children.length - 1); i >= 0; --i) {
+        for (var i = this.children.length - 1; i >= 0; --i) {
             var currentChildTreeElement = this.children[i];
             var currentNode = currentChildTreeElement.representedObject;
             var currentParentNode = currentNode.parentNode;
@@ -411,7 +411,6 @@
         }
 
         // Move / create TreeElements for our visible children.
-        var childIndex = 0;
         var elementToSelect = null;
         var visibleChildren = this._visibleChildren();
         for (var i = 0; i < visibleChildren.length && i < this.expandedChildrenLimit; ++i) {
@@ -459,7 +458,6 @@
         if (this.expandAllButtonElement && this.expandAllButtonElement.__treeElement.parent)
             this.removeChild(this.expandAllButtonElement.__treeElement);
 
-        var node = this.representedObject;
         if (!this._hasVisibleChildren())
             return;
 
@@ -497,7 +495,6 @@
     handleLoadAllChildren()
     {
         var visibleChildren = this._visibleChildren();
-        var totalChildrenCount = visibleChildren.length;
         this.expandedChildrenLimit = Math.max(visibleChildren.length, this.expandedChildrenLimit + WebInspector.DOMTreeElement.InitialChildrenLimit);
     }
 
@@ -1101,7 +1098,7 @@
         // tag, or HTML elements without a closing tag (such as <br>). Return
         // null in the case where there isn't a closing tag.
         var tags = this.listItemElement.getElementsByClassName("html-tag");
-        return (tags.length === 1 ? null : tags[tags.length - 1]);
+        return tags.length === 1 ? null : tags[tags.length - 1];
     }
 
     updateTitle(onlySearchQueryChanged)
@@ -1129,7 +1126,7 @@
 
     _buildAttributeDOM(parentElement, name, value, node)
     {
-        let hasText = (value.length > 0);
+        let hasText = value.length > 0;
         let attrSpanElement = parentElement.createChild("span", "html-attribute");
         let attrNameElement = attrSpanElement.createChild("span", "html-attribute-name");
         attrNameElement.textContent = name;
@@ -1177,7 +1174,7 @@
                     linkElement.href = ""
                     linkElement.textContent = linkText.insertWordBreakCharacters();
                     let descriptorElement = attrValueElement.appendChild(document.createElement("span"));
-                    descriptorElement.textContent = string.substring(spaceIndex);
+                    descriptorElement.textContent = descriptorText;
                 }
 
                 if (i < groups.length - 1) {
@@ -1526,7 +1523,6 @@
         var text = this.title.textContent;
         var searchRegex = new RegExp(this._searchQuery.escapeForRegExp(), "gi");
 
-        var offset = 0;
         var match = searchRegex.exec(text);
         var matchRanges = [];
         while (match) {

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/DOMTreeOutline.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/DOMTreeOutline.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/DOMTreeOutline.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -561,7 +561,7 @@
             console.assert(effectiveNode);
             if (!effectiveNode)
                 return;
-        }            
+        }
 
         if (effectiveNode.nodeType() !== Node.ELEMENT_NODE)
             return;

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/DashboardContainerView.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/DashboardContainerView.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/DashboardContainerView.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -68,7 +68,7 @@
             return null;
 
         if (this.currentDashboardView === dashboardView)
-            return;
+            return dashboardView;
 
         var index = this._dashboardStack.indexOf(dashboardView);
         this._showDashboardAtIndex(index);
@@ -92,7 +92,7 @@
         var _onlyReturnExistingViews_ = true;
         var dashboardView = this._dashboardViewForRepresentedObject(representedObject, onlyReturnExistingViews);
         if (!dashboardView)
-            return null;
+            return;
 
         this._closeDashboardView(dashboardView);
     }

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/DataGrid.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/DataGrid.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/DataGrid.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -726,7 +726,7 @@
         if (column["collapsesGroup"]) {
             console.assert(column["group"] !== column["collapsesGroup"]);
 
-            var dividerElement = headerCellElement.createChild("div", "divider");
+            headerCellElement.createChild("div", "divider");
 
             var collapseDiv = headerCellElement.createChild("div", "collapser-button");
             collapseDiv.title = this._collapserButtonCollapseColumnsToolTip();
@@ -1763,7 +1763,7 @@
     resizerDragging(resizer, positionDelta)
     {
         console.assert(resizer === this._currentResizer, resizer, this._currentResizer);
-        if (resizer != this._currentResizer)
+        if (resizer !== this._currentResizer)
             return;
 
         // Constrain the dragpoint to be within the containing div of the
@@ -1812,7 +1812,7 @@
     resizerDragEnded(resizer)
     {
         console.assert(resizer === this._currentResizer, resizer, this._currentResizer);
-        if (resizer != this._currentResizer)
+        if (resizer !== this._currentResizer)
             return;
 
         this._currentResizer = null;

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/DataGridNode.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/DataGridNode.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/DataGridNode.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -254,7 +254,7 @@
 
     get leftPadding()
     {
-        if (typeof(this._leftPadding) === "number")
+        if (typeof this._leftPadding === "number")
             return this._leftPadding;
 
         this._leftPadding = this.depth * this.dataGrid.indentWidth;
@@ -405,7 +405,7 @@
         if (!this.parent)
             return;
 
-        var previousChild = (myIndex > 0 ? this.parent.children[myIndex - 1] : null);
+        var previousChild = myIndex > 0 ? this.parent.children[myIndex - 1] : null;
 
         if (previousChild) {
             previousChild.nextSibling = this;

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/DebuggerSidebarPanel.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/DebuggerSidebarPanel.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/DebuggerSidebarPanel.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -123,7 +123,7 @@
                 }
             }
             return false;
-        };
+        }
 
         this.filterBar.addFilterBarButton("debugger-show-resources-with-issues-only", showResourcesWithIssuesOnlyFilterFunction.bind(this), true, WebInspector.UIString("Only show resources with issues"), WebInspector.UIString("Show all resources"), "Images/Errors.svg", 15, 15);
 
@@ -377,7 +377,7 @@
 
         if (!treeElement) {
             console.error("Unknown sourceCode instance", sourceCode);
-            return;
+            return null;
         }
 
         if (!treeElement.parent) {
@@ -750,7 +750,7 @@
     {
         if (!a.debuggerObject || !b.debuggerObject)
             return 0;
-        
+
         var aLocation = a.debuggerObject.sourceCodeLocation;
         var bLocation = b.debuggerObject.sourceCodeLocation;
 
@@ -865,6 +865,8 @@
 
         if (debuggerObject instanceof WebInspector.IssueMessage)
             return this._addIssue(debuggerObject);
+
+        return null;
     }
 
     _addIssue(issueMessage)

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/DefaultDashboardView.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/DefaultDashboardView.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/DefaultDashboardView.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -29,7 +29,7 @@
     {
         super(representedObject, "default");
 
-        representedObject.addEventListener(WebInspector.DefaultDashboard.Event.DataDidChange, () => { this._updateDisplaySoon() });
+        representedObject.addEventListener(WebInspector.DefaultDashboard.Event.DataDidChange, () => { this._updateDisplaySoon(); });
         this._scheduledUpdateIdentifier = undefined;
 
         this._items = {
@@ -62,7 +62,7 @@
         for (var name in this._items)
             this._appendElementForNamedItem(name);
     }
-    
+
     // Private
 
     _updateDisplaySoon()

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/FontResourceContentView.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/FontResourceContentView.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/FontResourceContentView.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -142,7 +142,7 @@
     {
         this.sizeToFit();
     }
-}
+};
 
 WebInspector.FontResourceContentView._uniqueFontIdentifier = 0;
 

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/FormattedValue.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/FormattedValue.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/FormattedValue.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -159,7 +159,7 @@
     span.classList.add("formatted-function");
     span.textContent = description.substring(0, description.indexOf("("));
     return span;
-}
+};
 
 WebInspector.FormattedValue.createElementForTypesAndValue = function(type, subtype, displayString, size, isPreview, hadException)
 {
@@ -225,7 +225,7 @@
         return WebInspector.FormattedValue.createElementForFunctionWithName(objectPreview.description);
 
     return new WebInspector.ObjectPreviewView(objectPreview, previewViewMode).element;
-}
+};
 
 WebInspector.FormattedValue.createObjectPreviewOrFormattedValueForRemoteObject = function(object, previewViewMode)
 {

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/GradientEditor.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/GradientEditor.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/GradientEditor.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -221,7 +221,7 @@
 
         this.dispatchEventToListeners(WebInspector.GradientEditor.Event.GradientChanged, {gradient: this._gradient});
     }
-}
+};
 
 WebInspector.GradientEditor.Event = {
     GradientChanged: "gradient-editor-gradient-changed",

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/GradientSlider.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/GradientSlider.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/GradientSlider.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -125,7 +125,7 @@
     knobWillDetach(knob)
     {
         knob.element.classList.add(WebInspector.GradientSlider.DetachingClassName);
-        
+
         this._stops.remove(knob.stop);
         this._knobs.remove(knob);
         this._sortStops();
@@ -194,7 +194,7 @@
             this._shadowKnob.element.classList.add(WebInspector.GradientSlider.ShadowClassName);
             this.element.appendChild(this._shadowKnob.element);
         }
-        
+
         this._shadowKnob.x = window.webkitConvertPointFromPageToNode(this.element, new WebKitPoint(event.pageX, event.pageY)).x;
 
         var colorData = this._canvas.getContext("2d").getImageData(this._shadowKnob.x - 1, 0, 1, 1).data;
@@ -218,7 +218,7 @@
     {
         var w = WebInspector.GradientSlider.Width;
         var h = WebInspector.GradientSlider.Height;
-        
+
         this._canvas.width = w;
         this._canvas.height = h;
 
@@ -238,7 +238,7 @@
     _updateKnobs()
     {
         var selectedStop = this._selectedKnob ? this._selectedKnob.stop : null;
-        
+
         while (this._knobs.length > this._stops.length)
             this._knobs.pop().element.remove();
 
@@ -311,7 +311,7 @@
     {
         return this._x;
     }
-    
+
     set x(x) {
         this._x = x;
         this._updateTransform();
@@ -321,7 +321,7 @@
     {
         return this._x;
     }
-    
+
     set y(y) {
         this._y = y;
         this._updateTransform();
@@ -349,7 +349,7 @@
             return;
 
         this._element.classList.toggle(WebInspector.GradientSliderKnob.SelectedClassName, selected);
-        
+
         if (this.delegate && typeof this.delegate.knobSelectionChanged === "function")
             this.delegate.knobSelectionChanged(this);
     }

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/HeapSnapshotContentView.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/HeapSnapshotContentView.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/HeapSnapshotContentView.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -103,7 +103,7 @@
                 tooltip: WebInspector.UIString("Size of the current object plus the size of all objects it keeps alive."),
                 width: "140px",
                 aligned: "right",
-                sortable: true, 
+                sortable: true,
             },
             size: {
                 title: WebInspector.UIString("Self Size"),
@@ -138,7 +138,7 @@
                 tooltip: WebInspector.UIString("Size of the current object plus the size of all objects it keeps alive."),
                 width: "140px",
                 aligned: "right",
-                sortable: true, 
+                sortable: true,
             },
             size: {
                 title: WebInspector.UIString("Self Size"),

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/HeapSnapshotDataGridTree.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/HeapSnapshotDataGridTree.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/HeapSnapshotDataGridTree.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -53,7 +53,6 @@
         let numberCompare = (columnIdentifier, a, b) => multiplier * (a.data[columnIdentifier] - b.data[columnIdentifier]);
         let localeCompare = (columnIdentifier, a, b) => multiplier * (a.data[columnIdentifier].localeCompare(b.data[columnIdentifier]));
 
-        let comparator;
         switch (columnIdentifier) {
         case "retainedSize":
             return numberCompare.bind(this, "retainedSize");
@@ -210,7 +209,7 @@
             this.appendChild(new WebInspector.HeapSnapshotClassDataGridNode({className, size, retainedSize, count: liveCount}, this));
         }
 
-        this.didPopulate()
+        this.didPopulate();
     }
 
     removeCollectedNodes(collectedNodes)

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/HeapSnapshotInstanceDataGridNode.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/HeapSnapshotInstanceDataGridNode.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/HeapSnapshotInstanceDataGridNode.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -247,7 +247,7 @@
     {
         HeapAgent.getRemoteObject(this._node.id, (error, remoteObjectPayload) => {
             if (error) {
-                this._populateError(containerElement)
+                this._populateError(containerElement);
                 return;
             }
 
@@ -427,7 +427,7 @@
         }
 
         function stringifyEdge(edge) {
-            switch(edge.type) {
+            switch (edge.type) {
             case WebInspector.HeapSnapshotEdgeProxy.EdgeType.Property:
             case WebInspector.HeapSnapshotEdgeProxy.EdgeType.Variable:
                 if (/^(?![0-9])\w+$/.test(edge.data))

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/HoverMenu.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/HoverMenu.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/HoverMenu.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -103,7 +103,7 @@
     {
         var buttonWidth = this._button.width;
         var buttonHeight = this._button.height;
-        
+
         // Add room for the button on the last line.
         var lastRect = rects.pop();
         lastRect.size.width += buttonWidth;
@@ -177,7 +177,7 @@
             "q", 0, r, r, r,
             "H", rect.maxX()
         ], tx, ty);
-        
+
         rect = secondRect;
         this._addPath([
             "M", rect.minX(), rect.minY(),
@@ -188,7 +188,7 @@
             "H", rect.minX()
         ], tx, ty);
     }
-    
+
     _drawOverlappingLines(rects)
     {
         var PADDING = 2;
@@ -208,10 +208,10 @@
         var maxY = rects.lastValue.maxY() + PADDING;
         var firstLineMinX = rects[0].minX() - PADDING;
         var lastLineMaxX = rects.lastValue.maxX() + PADDING;
-        
+
         if (firstLineMinX === minX && lastLineMaxX === maxX)
             return this._addRect(new WebInspector.Rect(minX, minY, maxX - minX, maxY - minY));
-        
+
         var lastLineMinY = rects.lastValue.minY() + PADDING;
         if (rects[0].minX() === minX + PADDING) {
             return this._addPath([

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/LogContentView.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/LogContentView.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/LogContentView.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -296,7 +296,7 @@
     {
         var messageLevel;
 
-        switch(level) {
+        switch (level) {
         case WebInspector.ConsoleMessage.MessageLevel.Warning:
             messageLevel = WebInspector.LogContentView.Scopes.Warnings;
             break;
@@ -680,7 +680,7 @@
     _scopeBarSelectionDidChange(event)
     {
         var item = this._scopeBar.selectedItems[0];
-        
+
         if (item.id === WebInspector.LogContentView.Scopes.All) {
             for (var item of this._scopeBar.items)
                 item.element.classList.remove("unread");
@@ -850,6 +850,7 @@
             if (this._isMessageVisible(messages[i]))
                 return messages[i];
         }
+        return null;
     }
 
     _nextMessage(message)
@@ -859,6 +860,7 @@
             if (this._isMessageVisible(messages[i]))
                 return messages[i];
         }
+        return null;
     }
 
     _clearFocusableChildren()

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/MemoryTimelineOverviewGraph.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/MemoryTimelineOverviewGraph.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/MemoryTimelineOverviewGraph.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -35,7 +35,7 @@
 
         this._memoryTimeline = timeline;
         this._memoryTimeline.addEventListener(WebInspector.Timeline.Event.RecordAdded, this._memoryTimelineRecordAdded, this);
-        this._memoryTimeline.addEventListener(WebInspector.MemoryTimeline.Event.MemoryPressureEventAdded, this._memoryTimelineMemoryPressureEventAdded, this)
+        this._memoryTimeline.addEventListener(WebInspector.MemoryTimeline.Event.MemoryPressureEventAdded, this._memoryTimelineMemoryPressureEventAdded, this);
 
         this._didInitializeCategories = false;
 
@@ -94,8 +94,6 @@
             this._chart.size = new WebInspector.Size(graphWidth, this.height);
 
         let graphStartTime = this.startTime;
-        let graphEndTime = this.endTime;
-        let graphCurrentTime = this.currentTime;
         let visibleEndTime = Math.min(this.endTime, this.currentTime);
 
         let secondsPerPixel = this.timelineOverview.secondsPerPixel;

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/MemoryTimelineView.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/MemoryTimelineView.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/MemoryTimelineView.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -174,7 +174,6 @@
 
         let graphStartTime = this.startTime;
         let graphEndTime = this.endTime;
-        let graphCurrentTime = this.currentTime;
         let visibleEndTime = Math.min(this.endTime, this.currentTime);
 
         let discontinuities = this._recording.discontinuitiesInTimeRange(graphStartTime, visibleEndTime);

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/NavigationSidebarPanel.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/NavigationSidebarPanel.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/NavigationSidebarPanel.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -589,8 +589,6 @@
                 break;
         }
 
-        let selectionWasHidden = selectedTreeElement && selectedTreeElement.hidden;
-
         let filters = this._filterBar.filters;
         this._textFilterRegex = simpleGlobStringToRegExp(filters.text, "i");
         this._filtersSetting.value = filters;

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/NetworkTimelineOverviewGraph.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/NetworkTimelineOverviewGraph.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/NetworkTimelineOverviewGraph.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -114,7 +114,7 @@
             return a.startTime - b.startTime;
         }
 
-        let minimumBarPaddingTime = WebInspector.TimelineOverview.MinimumDurationPerPixel * (WebInspector.TimelineRecordBar.MinimumWidthPixels + WebInspector.TimelineRecordBar.MinimumMarginPixels)
+        let minimumBarPaddingTime = WebInspector.TimelineOverview.MinimumDurationPerPixel * (WebInspector.TimelineRecordBar.MinimumWidthPixels + WebInspector.TimelineRecordBar.MinimumMarginPixels);
 
         // Try to find a row that has room and does not overlap a previous record.
         var foundRowForRecord = false;

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/NewTabContentView.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/NewTabContentView.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/NewTabContentView.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -124,7 +124,7 @@
             referencedView: this,
             shouldReplaceTab: !canCreateAdditionalTabs || !WebInspector.modifierKeys.metaKey,
             shouldShowNewTab: !WebInspector.modifierKeys.metaKey
-        }
+        };
         WebInspector.createNewTabWithType(tabType, options);
     }
 

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/ObjectPreviewView.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/ObjectPreviewView.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/ObjectPreviewView.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -273,7 +273,6 @@
         event.__addedObjectPreviewContextMenuItems = true;
 
         contextMenu.appendItem(WebInspector.UIString("Log Value"), () => {
-            let remoteObject = this._remoteObject;
             let isImpossible = !this._propertyPath || this._propertyPath.isFullPathImpossible();
             let text = isImpossible ? WebInspector.UIString("Selected Value") : this._propertyPath.displayPath(WebInspector.PropertyPath.Type.Value);
 

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/ObjectTreeBaseTreeElement.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/ObjectTreeBaseTreeElement.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/ObjectTreeBaseTreeElement.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -194,7 +194,7 @@
             this.treeOutline.objectTreeElementAddContextMenuItems(this, contextMenu);
             if (!contextMenu.isEmpty())
                 contextMenu.appendSeparator();
-        }             
+        }
 
         let resolvedValue = this.resolvedValue();
         if (!resolvedValue)

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/ObjectTreeMapEntryTreeElement.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/ObjectTreeMapEntryTreeElement.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/ObjectTreeMapEntryTreeElement.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -112,7 +112,7 @@
         super(object, propertyPath);
 
         this._key = key;
-        
+
         this.mainTitle = this.titleFragment();
 
         this.addClassName("value");

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/OverviewTimelineView.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/OverviewTimelineView.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/OverviewTimelineView.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -205,7 +205,7 @@
 
         let parentFrame = resource.parentFrame;
         if (!parentFrame)
-            return;
+            return null;
 
         let resourceTimelineRecord = this._networkTimeline ? this._networkTimeline.recordForResource(resource) : null;
         if (!resourceTimelineRecord)
@@ -231,7 +231,7 @@
             parentDataGridNode = this._addResourceToDataGridIfNeeded(parentResource);
             console.assert(parentDataGridNode);
             if (!parentDataGridNode)
-                return;
+                return null;
         }
 
         this._insertDataGridNode(resourceDataGridNode, parentDataGridNode);

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/Popover.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/Popover.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/Popover.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -513,7 +513,7 @@
 
         // Prevent the arrow from being positioned against one of the popover's rounded corners.
         let arrowPadding = cornerRadius + arrowHalfLength;
-        if (anchorEdge === WebInspector.RectEdge.MIN_Y ||anchorEdge === WebInspector.RectEdge.MAX_Y)
+        if (anchorEdge === WebInspector.RectEdge.MIN_Y || anchorEdge === WebInspector.RectEdge.MAX_Y)
             anchorPoint.x = Number.constrain(anchorPoint.x, bounds.minX() + arrowPadding, bounds.maxX() - arrowPadding);
         else
             anchorPoint.y = Number.constrain(anchorPoint.y, bounds.minY() + arrowPadding, bounds.maxY() - arrowPadding);

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/ProbeSetDetailsSection.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/ProbeSetDetailsSection.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/ProbeSetDetailsSection.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -133,7 +133,7 @@
         popover.present(target, [WebInspector.RectEdge.MAX_Y, WebInspector.RectEdge.MIN_Y, WebInspector.RectEdge.MAX_X]);
         popover.windowResizeHandler = () => {
             let target = WebInspector.Rect.rectFromClientRect(event.target.element.getBoundingClientRect());
-            popover.present(target, [WebInspector.RectEdge.MAX_Y, WebInspector.RectEdge.MIN_Y, WebInspector.RectEdge.MAX_X]);            
+            popover.present(target, [WebInspector.RectEdge.MAX_Y, WebInspector.RectEdge.MIN_Y, WebInspector.RectEdge.MAX_X]);
         };
         textBox.select();
     }

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/ProfileDataGridNode.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/ProfileDataGridNode.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/ProfileDataGridNode.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -174,8 +174,8 @@
                 for (let {type, source} of this._tree.modifiers) {
                     if (type === WebInspector.ProfileDataGridTree.ModifierType.ChargeToCaller) {
                         if (child.equals(source)) {
-                            this._childrenToChargeToSelf.add(child);                            
-                            this._extraSelfTimeFromChargedChildren += child.filteredTimestampsAndDuration(this._tree.startTime, this._tree.endTime).duration;                            
+                            this._childrenToChargeToSelf.add(child);
+                            this._extraSelfTimeFromChargedChildren += child.filteredTimestampsAndDuration(this._tree.startTime, this._tree.endTime).duration;
                             continue;
                         }
                     }
@@ -266,4 +266,4 @@
 
         this.removeEventListener("populate", this._populate, this);
     }
-}
+};

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/ProfileDataGridTree.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/ProfileDataGridTree.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/ProfileDataGridTree.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -50,11 +50,11 @@
 
     static buildSortComparator(columnIdentifier, sortOrder)
     {
-        let ascending = sortOrder == WebInspector.DataGrid.SortOrder.Ascending;
+        let ascending = sortOrder === WebInspector.DataGrid.SortOrder.Ascending;
         return function(a, b) {
             let result = a.data[columnIdentifier] - b.data[columnIdentifier];
             return ascending ? result : -result;
-        }
+        };
     }
 
     // Public
@@ -279,7 +279,7 @@
     {
         this.dispatchEventToListeners(WebInspector.ProfileDataGridTree.Event.ModifiersChanged);
     }
-}
+};
 
 WebInspector.ProfileDataGridTree.Event = {
     FocusChanged: "profile-data-grid-tree-focus-changed",

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/ProfileView.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/ProfileView.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/ProfileView.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -102,7 +102,7 @@
     {
         if (!this._profileDataGridTree)
             return false;
-        return this._profileDataGridTree.focusNodes.length > 0
+        return this._profileDataGridTree.focusNodes.length > 0;
     }
 
     clearFocusNodes()

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/ResourceSidebarPanel.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/ResourceSidebarPanel.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/ResourceSidebarPanel.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -68,7 +68,7 @@
 
         if (WebInspector.debuggableType === WebInspector.DebuggableType._javascript_) {
             this.contentTreeOutline.disclosureButtons = false;
-            WebInspector.SourceCode.addEventListener(WebInspector.SourceCode.Event.SourceMapAdded, () => { this.contentTreeOutline.disclosureButtons = true }, this);
+            WebInspector.SourceCode.addEventListener(WebInspector.SourceCode.Event.SourceMapAdded, () => { this.contentTreeOutline.disclosureButtons = true; }, this);
         }
 
         if (WebInspector.frameResourceManager.mainFrame)
@@ -331,7 +331,7 @@
     {
         const suppressOnDeselect = true;
         const suppressSelectSibling = true;
-        
+
         if (this._extensionScriptsFolderTreeElement) {
             if (this._extensionScriptsFolderTreeElement.parent)
                 this._extensionScriptsFolderTreeElement.parent.removeChild(this._extensionScriptsFolderTreeElement, suppressOnDeselect, suppressSelectSibling);

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/RulesStyleDetailsPanel.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/RulesStyleDetailsPanel.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/RulesStyleDetailsPanel.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -370,7 +370,7 @@
                         label.classList.remove(WebInspector.CSSStyleDetailsSidebarPanel.NoFilterMatchInSectionClassName);
                 } else
                     section.element.classList.add(WebInspector.CSSStyleDetailsSidebarPanel.FilterMatchingSectionHasLabelClassName);
-                
+
                 matchFound = true;
             }
         }

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/ScopeRadioButtonNavigationItem.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/ScopeRadioButtonNavigationItem.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/ScopeRadioButtonNavigationItem.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -87,7 +87,7 @@
         this._element.title = selectedItemIdentifier.label;
         this.dispatchEventToListeners(WebInspector.ScopeRadioButtonNavigationItem.Event.SelectedItemChanged);
     }
-}
+};
 
 WebInspector.ScopeRadioButtonNavigationItem.Event = {
     SelectedItemChanged: "scope-radio-button-navigation-item-selected-item-changed"

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/ScriptClusterTimelineView.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/ScriptClusterTimelineView.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/ScriptClusterTimelineView.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -55,7 +55,7 @@
 
         this._showContentViewForIdentifier(this._currentContentViewSetting.value);
 
-        this.contentViewContainer.addEventListener(WebInspector.ContentViewContainer.Event.CurrentContentViewDidChange, this._scriptClusterViewCurrentContentViewDidChange, this)
+        this.contentViewContainer.addEventListener(WebInspector.ContentViewContainer.Event.CurrentContentViewDidChange, this._scriptClusterViewCurrentContentViewDidChange, this);
     }
 
     // TimelineView

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/ScriptDetailsTimelineView.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/ScriptDetailsTimelineView.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/ScriptDetailsTimelineView.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -128,10 +128,6 @@
             if (dataGridNode.hidden)
                 return null;
 
-            let representedObject = null;
-            if (dataGridNode instanceof WebInspector.ProfileNodeDataGridNode)
-                representedObject = dataGridNode.profileNode;
-
             let pathComponent = new WebInspector.TimelineDataGridNodePathComponent(dataGridNode);
             pathComponent.addEventListener(WebInspector.HierarchicalPathComponent.Event.SiblingWasSelected, this.dataGridNodePathComponentSelected, this);
             pathComponents.unshift(pathComponent);

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/ScriptProfileTimelineView.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/ScriptProfileTimelineView.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/ScriptProfileTimelineView.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -57,7 +57,7 @@
 
         this._profileOrientationButton = new WebInspector.TextToggleButtonNavigationItem("profile-orientation", WebInspector.UIString("Inverted"));
         this._profileOrientationButton.addEventListener(WebInspector.ButtonNavigationItem.Event.Clicked, this._profileOrientationButtonClicked, this);
-        if (WebInspector.ScriptProfileTimelineView.profileOrientationSetting.value == WebInspector.ScriptProfileTimelineView.ProfileOrientation.TopDown)
+        if (WebInspector.ScriptProfileTimelineView.profileOrientationSetting.value === WebInspector.ScriptProfileTimelineView.ProfileOrientation.TopDown)
             this._profileOrientationButton.activated = false;
         else
             this._profileOrientationButton.activated = true;
@@ -64,7 +64,7 @@
 
         this._topFunctionsButton = new WebInspector.TextToggleButtonNavigationItem("top-functions", WebInspector.UIString("Top Functions"));
         this._topFunctionsButton.addEventListener(WebInspector.ButtonNavigationItem.Event.Clicked, this._topFunctionsButtonClicked, this);
-        if (WebInspector.ScriptProfileTimelineView.profileTypeSetting.value == WebInspector.ScriptProfileTimelineView.ProfileViewType.Hierarchy)
+        if (WebInspector.ScriptProfileTimelineView.profileTypeSetting.value === WebInspector.ScriptProfileTimelineView.ProfileViewType.Hierarchy)
             this._topFunctionsButton.activated = false;
         else
             this._topFunctionsButton.activated = true;

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/StackedLineChart.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/StackedLineChart.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/StackedLineChart.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -87,7 +87,7 @@
     initializeSections(sectionClassNames)
     {
         console.assert(!this._pathElements.length, "Should not initialize multiple times");
-        
+
         sectionClassNames.reverse();
 
         for (let i = 0; i < sectionClassNames.length; ++i) {

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/StorageSidebarPanel.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/StorageSidebarPanel.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/StorageSidebarPanel.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -189,7 +189,6 @@
 
     _domStorageObjectWasAdded(event)
     {
-        var domStorage = event.data.domStorage;
         this._addDOMStorageObject(event.data.domStorage);
     }
 
@@ -212,7 +211,6 @@
 
     _databaseWasAdded(event)
     {
-        var database = event.data.database;
         this._addDatabase(event.data.database);
     }
 

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/TabBrowser.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/TabBrowser.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/TabBrowser.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -49,13 +49,13 @@
         this._contentViewContainer = new WebInspector.ContentViewContainer;
         this.addSubview(this._contentViewContainer);
 
-        let showNextTab = () => { this._showNextTab(); }
-        let showPreviousTab = () => { this._showPreviousTab(); }
+        let showNextTab = () => { this._showNextTab(); };
+        let showPreviousTab = () => { this._showPreviousTab(); };
         let closeCurrentTab = () => {
             let selectedTabBarItem = this._tabBar.selectedTabBarItem;
             if (this._tabBar.tabBarItems.length > 2 || !selectedTabBarItem.isDefaultTab)
                 this._tabBar.removeTabBarItem(selectedTabBarItem);
-        }
+        };
 
         this._closeCurrentTabKeyboardShortuct = new WebInspector.KeyboardShortcut(WebInspector.KeyboardShortcut.Modifier.CommandOrControl | WebInspector.KeyboardShortcut.Modifier.Shift, "W", closeCurrentTab);
 

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/TextEditor.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/TextEditor.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/TextEditor.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -826,7 +826,7 @@
     _updateAfterFormatting(pretty, beforePrettyPrintState)
     {
         let oldSelectionAnchor = beforePrettyPrintState.selectionAnchor;
-        let oldSelectionHead = beforePrettyPrintState.selectionHead;        
+        let oldSelectionHead = beforePrettyPrintState.selectionHead;
         let newSelectionAnchor, newSelectionHead;
         let newExecutionLocation = null;
 
@@ -891,7 +891,7 @@
             this.searchCleared();
             // Set timeout so that this happens after the current CodeMirror operation.
             // The editor has to update for the value and selection changes.
-            setTimeout(() => { this.performSearch(searchQuery) }, 0);
+            setTimeout(() => { this.performSearch(searchQuery); }, 0);
         }
 
         if (this._delegate && typeof this._delegate.textEditorUpdatedFormatting === "function")

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/TimelineDataGridNode.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/TimelineDataGridNode.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/TimelineDataGridNode.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -44,7 +44,7 @@
 
     get record()
     {
-        return this.records && this.records.length ? this.records[0] : null;;
+        return this.records && this.records.length ? this.records[0] : null;
     }
 
     get records()

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/TimelineOverview.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/TimelineOverview.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/TimelineOverview.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -106,7 +106,7 @@
         this._cachedScrollContainerWidth = NaN;
         this._timelineRulerSelectionChanged = false;
         this._viewMode = WebInspector.TimelineOverview.ViewMode.Timelines;
-        this._selectedTimeline = null;;
+        this._selectedTimeline = null;
 
         for (let instrument of this._recording.instruments)
             this._instrumentAdded(instrument);
@@ -590,7 +590,7 @@
     _handleGestureChange(event)
     {
         // Cap zooming out at 5x.
-        let scale = Math.max(1/5, event.scale);
+        let scale = Math.max(1 / 5, event.scale);
 
         let mouseOffset = event.pageX - this.element.totalOffsetLeft;
         let newSecondsPerPixel = this._gestureStartDurationPerPixel / scale;

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/TimelineRecordFrame.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/TimelineRecordFrame.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/TimelineRecordFrame.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -96,7 +96,7 @@
 
         this._element.style.width = (1 / graphDataSource.timelineOverview.secondsPerPixel) + "px";
 
-        var graphDuration = graphDataSource.endTime - graphDataSource.startTime
+        var graphDuration = graphDataSource.endTime - graphDataSource.startTime;
         var recordLeftPosition = (frameIndex - graphDataSource.startTime) / graphDuration;
         this._updateElementPosition(this._element, recordLeftPosition, "left");
         this._updateChildElements(graphDataSource);

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/TimelineRecordingContentView.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/TimelineRecordingContentView.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/TimelineRecordingContentView.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -307,7 +307,7 @@
         if (selectedPathComponent === this._selectedTimeRangePathComponent)
             return;
 
-        let timelineRuler = this._timelineOverview.timelineRuler
+        let timelineRuler = this._timelineOverview.timelineRuler;
         if (selectedPathComponent === this._entireRecordingPathComponent)
             timelineRuler.selectEntireRange();
         else {

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/TimelineTabContentView.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/TimelineTabContentView.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/TimelineTabContentView.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -435,7 +435,7 @@
 
     _recordingCreated(event)
     {
-        this._addRecording(event.data.recording)
+        this._addRecording(event.data.recording);
         this._recordingCountChanged();
     }
 

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/TreeOutline.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/TreeOutline.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/TreeOutline.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -325,7 +325,6 @@
         }
 
         for (let child of childrenToRemove) {
-            child = childrenToRemove[i];
             child.deselect(suppressOnDeselect);
 
             let treeOutline = child.treeOutline;
@@ -363,7 +362,7 @@
     _forgetTreeElement(element)
     {
         if (this.selectedTreeElement === element) {
-            element.deselect(true)
+            element.deselect(true);
             this.selectedTreeElement = null;
         }
         if (this._knownTreeElements[element.identifier])

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/VisualStyleCommaSeparatedKeywordEditor.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/VisualStyleCommaSeparatedKeywordEditor.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/VisualStyleCommaSeparatedKeywordEditor.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -69,7 +69,7 @@
     get value()
     {
         if (!this._commaSeparatedKeywords.hasChildren)
-            return;
+            return "";
 
         let value = "";
         for (let treeItem of this._commaSeparatedKeywords.children) {
@@ -288,4 +288,4 @@
 WebInspector.VisualStyleCommaSeparatedKeywordEditor.Event = {
     TreeItemSelected: "visual-style-comma-separated-keyword-editor-tree-item-selected",
     NoRemainingTreeItems: "visual-style-comma-separated-keyword-editor-no-remaining-tree-items"
-}
+};

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/VisualStylePropertyEditor.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/VisualStylePropertyEditor.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/VisualStylePropertyEditor.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -52,7 +52,7 @@
             else {
                 this._possibleValues.basic = canonicalizeValues(possibleValues.basic);
                 this._possibleValues.advanced = canonicalizeValues(possibleValues.advanced);
-            } 
+            }
         }
         this._possibleUnits = null;
         if (possibleUnits) {
@@ -105,7 +105,7 @@
                 name,
                 textContainsNameRegExp: new RegExp("(?:(?:^|;)\\s*" + name + "\\s*:)"),
                 replacementRegExp: new RegExp("((?:^|;)\\s*)(" + name + ")(.+?(?:;|$))")
-            })
+            });
         }
 
         this._propertyReferenceName = propertyNames[0];
@@ -615,4 +615,4 @@
 
 WebInspector.VisualStylePropertyEditor.Event = {
     ValueDidChange: "visual-style-property-editor-value-changed"
-}
+};

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/VisualStyleSelectorSection.js (205424 => 205425)


--- trunk/Source/WebInspectorUI/UserInterface/Views/VisualStyleSelectorSection.js	2016-09-05 05:01:57 UTC (rev 205424)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/VisualStyleSelectorSection.js	2016-09-05 05:02:23 UTC (rev 205425)
@@ -327,4 +327,4 @@
 
 WebInspector.VisualStyleSelectorSection.Event = {
     SelectorChanged: "visual-style-selector-section-selector-changed"
-}
+};
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to