Bug 152039 – Web Inspector: console.dir should expand objects

• Status: RESOLVED
• Resolution: FIXED
• Component: Web Inspector

[Nikita Vasilyev]

Created attachment 266970 [details]
[Image] Actual/Expected

See the attached image.

console.log and console.dir of the following object shows a preview that isn't expandable (since it's lossless) and not very readable.

[
    {
        startOffset: 26,
        endOffset: 65,
        executionCount: 1
    },
    {
        startOffset: 66,
        endOffset: 76,
        executionCount: 1000
    },
    {
        startOffset: 77,
        endOffset: 84,
        executionCount: 1
    },
    {
        startOffset: 85,
        endOffset: 112,
        executionCount: 1000
    },
    {
        startOffset: 113,
        endOffset: 151,
        executionCount: 0
    },
    {
        startOffset: 152,
        endOffset: 161,
        executionCount: 1
    },
    {
        startOffset: 162,
        endOffset: 171,
        executionCount: 1000
    },
    {
        startOffset: 172,
        endOffset: 173,
        executionCount: 1
    }
])


Arrays with less than 10 items seem to be always logged as unexpandable (lossless) preview.
I had to add leading zeros to force it to be expandable.

I suggest always expand objects logged with console.dir and keep console.log as is.
This way it would be up to developers to decide which output they want.

[Radar WebKit Bug Importer]

<rdar://problem/23816853>

[Nikita Vasilyev]

Hm, I just realized that I should have used console.table for this particular example.

Regardless of that, I often log objects with 10+ properties and there is no way to have them expanded by default.

[Joseph Pecoraro]

console.dir should always allow expanding regardless of the property of the object/array. So it seems that broke somewhere.

[Nikita Vasilyev]

*** This bug has been marked as a duplicate of bug 152328 ***

[Nikita Vasilyev]

Not a duplicate, reopening.

console.dir should EXPAND objects, not just allowing them to be expanded.

For example, console.dir(window) should show window expanded.

[Nikita Vasilyev]

Created attachment 377061 [details]
Patch

[Joseph Pecoraro]

Comment on attachment 377061 [details]
Patch

r=me!

[WebKit Commit Bot]

Comment on attachment 377061 [details]
Patch

Clearing flags on attachment: 377061

Committed r249034: <https://trac.webkit.org/changeset/249034>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.