| commit | a0d40624e1b649552ff86aff851ba1d3e5be223a | [log] [tgz] |
|---|---|---|
| author | Simon Zünd <[email protected]> | Wed Feb 12 13:16:42 2020 |
| committer | Commit Bot <[email protected]> | Wed Feb 12 16:45:22 2020 |
| tree | 114960bf16f299db51a9167bc193dc698f48e717 | |
| parent | cd1a491dd59faea768145c3f1cf07b83c1f4b073 [diff] [blame] |
[utilities] Replace {valuesArray} with {values} and/or spread operator
For {Set}s we omit the call to Set#values, since spreading a set
{foo} "[...foo]" is equivalent to "[...foo.values()]".
Drive-by: Remove the now unused Map#valuesArray and Set#valuesArray.
Bug: chromium:1050549
Change-Id: Ic9b51f957d895265a79a7fa3471ecb6cb734a1f0
Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/2050770
Commit-Queue: Simon Zünd <[email protected]>
Reviewed-by: Tim van der Lippe <[email protected]>
diff --git a/front_end/elements/ClassesPaneWidget.js b/front_end/elements/ClassesPaneWidget.js index 919c967..62f758d 100644 --- a/front_end/elements/ClassesPaneWidget.js +++ b/front_end/elements/ClassesPaneWidget.js
@@ -216,8 +216,7 @@ activeClasses.add(className); } - const newClasses = activeClasses.valuesArray(); - newClasses.sort(); + const newClasses = [...activeClasses.values()].sort(); this._pendingNodeClasses.set(node, newClasses.join(' ')); this._updateNodeThrottler.schedule(this._flushPendingClasses.bind(this)); @@ -314,7 +313,7 @@ .classNamesPromise(selectedNode.ownerDocument.id) .then(classes => completions.addAll(classes)); promises.push(domPromise); - return Promise.all(promises).then(() => completions.valuesArray()); + return Promise.all(promises).then(() => [...completions]); } /**