1401972 - Devtools Reps: update bundle to v0.14.0

Status: VERIFIED FIXED

(DevTools :: Shared Components, defect)

Comment 1

[Nicolas Chevobbe [:nchevobbe]]

Attachment: Bug 1401972 - Release: Update reps bundle to 0.14.0; .

Review commit: https://reviewboard.mozilla.org/r/182278/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/182278/

Comment 2

[Nicolas Chevobbe [:nchevobbe]]

Comment on attachment 8911060 [details]
Bug 1401972 - Release: Update reps bundle to 0.14.0; .

https://reviewboard.mozilla.org/r/182278/#review187856

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/af867bbd41ee
Release: Update reps bundle to 0.14.0; r=nchevobbe.

Comment 4

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/af867bbd41ee

Comment 5

[Nicolas Chevobbe [:nchevobbe]]

Comment on attachment 8911060 [details]
Bug 1401972 - Release: Update reps bundle to 0.14.0; .

This is an update to the Reps bundle, the underlying sets of components the devtools use to display variables. The bundle is used in the console, the debugger, the dom panel, the json viewer and the inspector.
This patches fixes an important perf regression as well as an issue that prevented to expand some object in the console and the debugger

[Feature/Bug causing the regression]:
https://bugzilla.mozilla.org/show_bug.cgi?id=1399460

[User impact if declined]:
Serious slow-downs when using the console with a lot of logged object. Impossibility to expand some objects in the console.

[Is this code covered by automated tests?]:
Yes

[Has the fix been verified in Nightly?]:
Yes

[Needs manual test from QE? If yes, steps to reproduce]: 
Yes,
STR: 
1. Open the console
2. Evaluate
```
var map1 = new Map([]); 
map1.set(map1, 1); map1;
```
3. Expand the object, then the `<entries>` node, then the <key> one

Expected results:
It shows child nodes of the key node (which should be `Map → 1`)

[List of other uplifts needed for the feature/fix]:

[Is the change risky?]: No
[Why is the change risky/not risky?]: Because we added some guards to not fail when expanding objects (and added tests to make sure everything is fine), and remove unecessary event listeners that we didn't used at all and were causing the performance issue.

[String changes made/needed]:

Comment 6

[Sylvestre Ledru [:Sylvestre]]

Comment on attachment 8911060 [details]
Bug 1401972 - Release: Update reps bundle to 0.14.0; .

Improve performance. We usually don't like to take this kind of patches in beta because we know that performance improvements are usually causing regressions.
However, it is early in the beta cycle, so, taking it.
Should be in 57b4 (gtb tomorrow Thursday)

Comment 7

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-beta/rev/e420ad572fe0

Comment 8

[roxleitan]

Verified fixed using the latest Nightly 58.0a1 (2017-11-06)and FF Beta 57.0b14 on Windows 10 x64, Ubuntu 16.04 x64 and Mac OS X 10.12.