Bug 170143

Summary: Web Inspector: Probe values not showing in sidebar
Product: WebKit Reporter: Joseph Pecoraro <joepeck>
Component: Web InspectorAssignee: Matt Baker <mattbaker>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, hi, inspector-bugzilla-changes, joepeck, mattbaker
Priority: P2    
Version: WebKit Nightly Build   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch none

Description Joseph Pecoraro 2017-03-27 15:05:01 PDT
Summary:
Probe values not showing in sidebar

Steps to Reproduce:
1. Inspect http://bogojoker.com/shell/
2. Resources > Select easySlider.min.js
3. Add a breakpoint on line 56
4. Right click > Edit Breakpoint
5. Add Action
6. Add a Probe Expression: "autoplay"
7. Automatically continue after evaluating
8. Dismiss popover
  => Should now have a Probes sidebar
9. Click the down arrow on the page
  => Should log a value in the Probe Datagrid, but nothing shows
Comment 1 Devin Rousso 2017-03-27 16:11:42 PDT
I think this is being caused by the way WebInspector.ProbeSetDataGrid is being used by WebInspector.ProbeSetDetailsSection.  Since WebInspector.DetailsSection is not yet a subclass of WebInspector.View, the check for `_isAttachedToRoot` fails as it's not a subview of the root, instead being added manually by calling `appendChild` on the actual element.
Comment 2 Matt Baker 2017-04-05 13:00:05 PDT
Created attachment 306304 [details]
Patch
Comment 3 Joseph Pecoraro 2017-04-05 13:46:51 PDT
Comment on attachment 306304 [details]
Patch

r=me
Comment 4 WebKit Commit Bot 2017-04-05 13:58:54 PDT
Comment on attachment 306304 [details]
Patch

Clearing flags on attachment: 306304

Committed r214967: <http://trac.webkit.org/changeset/214967>
Comment 5 WebKit Commit Bot 2017-04-05 13:58:55 PDT
All reviewed patches have been landed.  Closing bug.