1130392 - The list of numeric properties on an object in the debugger should always be sorted in incrementing order

Status: RESOLVED DUPLICATE of bug 967319

(DevTools :: Debugger, defect, P3)

Description

[Jared Wein [:jaws] (please needinfo? me)]

+++ This bug was initially created as a clone of Bug #883702 +++

The debugger sorts properties in alphabetical ordering, but when it comes to numerical properties it sorts them in the following order:

0:
1:
10:
100:
101 through 108:
109:
11:
110:
etc.

It is treating those numbers as strings.

Comment 1

[Jared Wein [:jaws] (please needinfo? me)]

Simpler steps to reproduce:
In the console, type the following commands:
let a = {1: 1, 2: 2, 10:10};
a;

Notice how the console shows that the value of `a` is `Object { 1: 1, 2: 2, 10: 10 }`.
Click on the "Object" value.
The variables view that appears will show the properties in a vertical column, in the order of: 1, 10, 2.

To fix this bug, you will need to edit http://mxr.mozilla.org/mozilla-central/source/browser/devtools/shared/widgets/VariablesView.jsm?rev=0f7b2674c64a#1398 and http://mxr.mozilla.org/mozilla-central/source/browser/devtools/shared/widgets/VariablesView.jsm?rev=0f7b2674c64a#2235 to take a custom sorting function that will order properties that parse as numbers numerically (in increasing order), and properties that do not parse as numbers lexicographically.

Comment 2

[Panos Astithas (he/him) [:past] (please ni?)]

This is a duplicate of bug 967319 (six digits!), but I will leave it up to you which way to dupe.

Comment 3

[Jared Wein [:jaws] (please needinfo? me)]

We can dupe to the older bug. Brandon, I'm going to close this bug and re-assign you to bug 967319 since it was filed first :)