[DevTools] No NetworkManager and NetworkLog for v8only mode

third_party/WebKit/Source/devtools/front_end/security/SecurityPanel.js

Index: third_party/WebKit/Source/devtools/front_end/security/SecurityPanel.js
diff --git a/third_party/WebKit/Source/devtools/front_end/security/SecurityPanel.js b/third_party/WebKit/Source/devtools/front_end/security/SecurityPanel.js
index aa0b8dfd62d58117b45f49f117cc3fce17e59c3b..083a9137f125703605ff18a0a180e34035dcc68c 100644
--- a/third_party/WebKit/Source/devtools/front_end/security/SecurityPanel.js
+++ b/third_party/WebKit/Source/devtools/front_end/security/SecurityPanel.js
@@ -169,10 +169,11 @@ WebInspector.SecurityPanel.prototype = {
             var originState = {};
             originState.securityState = securityState;
 
+            var networkManager = WebInspector.NetworkManager.fromTarget(request.target());
             var securityDetails = request.securityDetails();
-            if (securityDetails) {
+            if (securityDetails && networkManager) {
                 originState.securityDetails = securityDetails;
-                originState.certificateDetailsPromise = request.target().networkManager.certificateDetailsPromise(securityDetails.certificateId);
+                originState.certificateDetailsPromise = networkManager.certificateDetailsPromise(securityDetails.certificateId);
             }
 
             this._origins.set(origin, originState);
@@ -249,8 +250,12 @@ WebInspector.SecurityPanel.prototype = {
         this._target = target;
 
         target.resourceTreeModel.addEventListener(WebInspector.ResourceTreeModel.EventTypes.MainFrameNavigated, this._onMainFrameNavigated, this);
-        target.networkManager.addEventListener(WebInspector.NetworkManager.EventTypes.ResponseReceived, this._onResponseReceived, this);
-        target.networkManager.addEventListener(WebInspector.NetworkManager.EventTypes.RequestFinished, this._onRequestFinished, this);
+        var networkManager = WebInspector.NetworkManager.fromTarget(target);
+        if (networkManager)

[dgozman, 2016/06/29 18:37:58]

Page target must have a network manager. I wonder how we can make this better...

[eostroukhov-old, 2016/06/29 22:54:21]

For me it is not obvious that Elements tab always means you should have a
Network tab... I think it is ok to allow page targets without network.

+        {

[dgozman, 2016/06/29 18:37:58]

style: { on the same line

[eostroukhov-old, 2016/06/29 22:54:21]

Done.

+            networkManager.addEventListener(WebInspector.NetworkManager.EventTypes.ResponseReceived, this._onResponseReceived, this);
+            networkManager.addEventListener(WebInspector.NetworkManager.EventTypes.RequestFinished, this._onRequestFinished, this);
+        }
 
         var securityModel = WebInspector.SecurityModel.fromTarget(target);
         securityModel.addEventListener(WebInspector.SecurityModel.EventTypes.SecurityStateChanged, this._onSecurityStateChanged, this);