1260380 - "Parsed value" section gets duplicated when editing a local storage entry

Status: RESOLVED FIXED

(DevTools :: Storage Inspector, defect)

Description

[Patrick Brosset <:pbro>]

STR:
- (this depends on bug 1231155)
- open the mozilla.org site
- open the console
- enter localStorage.setItem('test', '{"foo": "bar"}')
- open the storage inspector
- switch to the local storage item in the tree-view
- click on the 'test' item
=> Notice that the sidebar opens and shows the JSON value as a parsed JS object

- now, double-click on the '{"foo": "bar"}' value in the table
- edit bar to bar2 instead
- press enter
=> Notice that the "Parsed Value" section in the sidebar is now duplicated, there are 2 parsed JS objects in there.

Comment 1

[Jarda Snajdr [:jsnajdr]]

Attachment: 'Parsed value' section gets duplicated when editing.

This bug happens because:
1. After editing a value, displayObjectSidebar() is called twice in a quick succession. This is because the row get reselected [1] and updated [2] as the result of the edit.
2. In displayObjectSidebar(), the UI update is broken into two parts - first it's cleared and a part of the content is added, then it waits for an async action (getting value of a string grip), and then finishes the UI update. Race conditions happen that cause the "Parsed Value" section to be duplicated.

Fixed this by refactoring the displayObjectSidebar() method to first do any async actions, and then do the whole UI update synchronously.

Comment 2

[Jarda Snajdr [:jsnajdr]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=5a92796172f0

Comment 3

[Mike Ratcliffe [:miker] [:mratcliffe] [:mikeratcliffe]]

Comment on attachment 8743781 [details] [diff] [review]
'Parsed value' section gets duplicated when editing.

Review of attachment 8743781 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good... r+ on green try.

Comment 4

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/b6f71c9b42b9

Comment 5

[Carsten Book [:Tomcat]]

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