RESOLVED FIXED198004
Web Inspector: Storage tab crashes easily when adding new local/session storage entries. 
https://bugs.webkit.org/show_bug.cgi?id=198004
Summary Web Inspector: Storage tab crashes easily when adding new local/session stora...
Ross Kirsling
Reported 2019-05-17 15:01:20 PDT
(Evidently somebody noticed this a year ago on our end but it never got reported upstream... :sad_panda:) Steps: 1. Open Storage tab 2. Double-click an empty row 3. Enter text for key 4. Press tab 5. Enter text for value 6. Press tab Result: > TypeError:​ undefined is not an object (evaluating 'this._editingNode._element.children')​ (at DataGrid.js:​523:​49)​ > _startEditingNodeAtColumnIndex @ DataGrid.js:​523:​49 > moveToNextCell @ DataGrid.js:​596:​52 > _editingCommitted @ DataGrid.js:​606:​28 > _editingCommitted @ [native code]​ > editingCommitted @ EditingSupport.js:​197:​26 > blurEventListener @ EditingSupport.js:​143:​30 Notes: - After step 4, the text for the key disappears. - If you press enter instead of tab in step 6, the crash does not occur and the text for the key re-appears, but now the text for the value disappears.
Attachments
Patch (3.74 KB, patch)
2019-05-20 13:05 PDT, Devin Rousso 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Devin Rousso
Comment 1 2019-05-20 13:05:14 PDT
Created attachment 370268 [details] Patch
Ross Kirsling
Comment 2 2019-05-20 13:18:56 PDT
Comment on attachment 370268 [details] Patch Woohoo! r=me
WebKit Commit Bot
Comment 3 2019-05-20 13:45:28 PDT
Comment on attachment 370268 [details] Patch Clearing flags on attachment: 370268 Committed r245535: <https://trac.webkit.org/changeset/245535>
WebKit Commit Bot
Comment 4 2019-05-20 13:45:29 PDT
All reviewed patches have been landed. Closing bug.
Radar WebKit Bug Importer
Comment 5 2019-05-20 13:46:19 PDT
<rdar://problem/50958823>
Note You need to log in before you can comment on or make changes to this bug.