[Devtools] Regression fix for double toolbar for images in network

third_party/WebKit/Source/devtools/front_end/network/RequestPreviewView.js

Index: third_party/WebKit/Source/devtools/front_end/network/RequestPreviewView.js
diff --git a/third_party/WebKit/Source/devtools/front_end/network/RequestPreviewView.js b/third_party/WebKit/Source/devtools/front_end/network/RequestPreviewView.js
index a316ec37b3998595b8294d609638079f1f38e944..0d2362ecbb0bbe8b90279f6e1b6f53dd0e0337a1 100644
--- a/third_party/WebKit/Source/devtools/front_end/network/RequestPreviewView.js
+++ b/third_party/WebKit/Source/devtools/front_end/network/RequestPreviewView.js
@@ -70,8 +70,9 @@ WebInspector.RequestPreviewView.prototype = {
         function handlePreviewView(view)
         {
             this._previewView = view;

[lushnikov, 2016/08/03 17:47:08]

it looks to me that this._previewView is equal to this.innerView (and almost
never used). Let's simplify this!

+            var isShowing = this._previewView.isShowing();

[lushnikov, 2016/08/03 17:38:32]

Why does this code run everytime you switch to the network panel? This doesn't
look right.

             this._previewView.show(this.element);
-            if (this._previewView instanceof WebInspector.View) {
+            if (!isShowing && this._previewView instanceof WebInspector.View) {
                 var toolbar = new WebInspector.Toolbar("network-item-preview-toolbar", this.element);
                 for (var item of /** @type {!WebInspector.View} */ (this._previewView).toolbarItems())
                     toolbar.appendToolbarItem(item);