Bug 170104

Summary: REGRESSION (r214354): Web Inspector: WebSocket connection icon obscures close content view icon
Product: WebKit Reporter: Nikita Vasilyev <nvasilyev>
Component: Web InspectorAssignee: Devin Rousso <hi>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, inspector-bugzilla-changes, joepeck, mattbaker
Priority: P2    
Version: WebKit Nightly Build   
Hardware: All   
OS: All   
Attachments:
Description Flags
[Image] Bug
none
Patch none

Description Nikita Vasilyev 2017-03-25 19:50:03 PDT
Created attachment 305412 [details]
[Image] Bug

Regressed when this icon was introduced.

I still like the Matt's idea of indicating the connection status by replacing or perhaps augmenting the document icon.
Comment 1 Devin Rousso 2017-03-27 14:52:53 PDT
Created attachment 305513 [details]
Patch
Comment 2 Devin Rousso 2017-03-27 14:53:44 PDT
(In reply to Devin Rousso from comment #1)
> Created attachment 305513 [details]
> Patch

This patch removes any indicator from the Network tab.  I feel this is fine since there is more information (such as the increasing size or the waterfall) that a Socket is still alive.
Comment 3 Matt Baker 2017-03-27 15:13:38 PDT
Could we display the connection status icon in the subtitle part of the tree element? (we'd want to hide the em-dash separator if we tried this)
Comment 4 Joseph Pecoraro 2017-04-03 11:28:26 PDT
Comment on attachment 305513 [details]
Patch

r=me, This sounds fine to me.
Comment 5 WebKit Commit Bot 2017-04-03 14:26:13 PDT
Comment on attachment 305513 [details]
Patch

Clearing flags on attachment: 305513

Committed r214839: <http://trac.webkit.org/changeset/214839>
Comment 6 WebKit Commit Bot 2017-04-03 14:26:14 PDT
All reviewed patches have been landed.  Closing bug.