Also move the creation of the NetworkGridContentView to right before it is shown.
Created attachment 310910 [details]
Created attachment 310912 [details]
[Image] Timeline Before
Created attachment 310913 [details]
[Image] Timeline After
Comment on attachment 310910 [details]
View in context: https://bugs.webkit.org/attachment.cgi?id=310910&action=review
r=me, but with concerns
> this.contentBrowser.addEventListener(WebInspector.ContentBrowser.Event.CurrentContentViewDidChange, this._contentBrowserCurrentContentViewDidChange, this);
As was mentioned previously, it doesn't make sense to me why this is not in initialLayout. It seems potentially problematic keeping it separate. It is quite small right now but I do worry about future changes adding code in there that is unsafe unless performed after initial layout.
> + var scopeItemPrefix = "network-sidebar-";
Style: Lets upgrade these to `let` now.
Created attachment 311042 [details]
Comment on attachment 311042 [details]
Clearing flags on attachment: 311042
Committed r217295: <http://trac.webkit.org/changeset/217295>
All reviewed patches have been landed. Closing bug.