https://bugs.webkit.org/show_bug.cgi?id=74648 introduced Node.prototype.append, among a few other new DOM methods. Convert the following code: element.appendChild(document.createTextNode(aString)) to element.append(aString)
<rdar://problem/22000352>
Created attachment 257527 [details] Patch
Comment on attachment 257527 [details] Patch Clearing flags on attachment: 257527 Committed r187402: <http://trac.webkit.org/changeset/187402>
All reviewed patches have been landed. Closing bug.
Comment on attachment 257527 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=257527&action=review Nice! > Source/WebInspectorUI/UserInterface/Models/Breakpoint.js:399 > checkboxLabel.appendChild(checkboxElement); > - checkboxLabel.appendChild(document.createTextNode(this._sourceCodeLocation.displayLocationString())); > + checkboxLabel.append(this._sourceCodeLocation.displayLocationString()); Cases like this, where we have multiple appends in a row, we can reduce to a single append call: From: checkboxLabel.appendChild(checkboxElement); checkboxLabel.append(text); To: checkboxLabel.append(checkboxElement, text); > Source/WebInspectorUI/UserInterface/Views/BoxModelDetailsSectionRow.js:219 > + boxElement.append(widthElement, " \u00D7 ", heightElement); Which I see you did here. > Source/WebInspectorUI/UserInterface/Views/CSSStyleDetailsSidebarPanel.js:59 > labelElement.appendChild(checkboxElement); > - labelElement.appendChild(document.createTextNode(label)); > + labelElement.append(label); Another case.