Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(335)

Side by Side Diff: third_party/WebKit/Source/devtools/front_end/security/SecurityModel.js

Issue 2542533004: Override DevTools security summary when a Safe Browsing warning shows. (Closed)
Patch Set: Address review feedback Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 /** 4 /**
5 * @unrestricted 5 * @unrestricted
6 */ 6 */
7 Security.SecurityModel = class extends SDK.SDKModel { 7 Security.SecurityModel = class extends SDK.SDKModel {
8 /** 8 /**
9 * @param {!SDK.Target} target 9 * @param {!SDK.Target} target
10 */ 10 */
(...skipping 55 matching lines...) Expand 10 before | Expand all | Expand 10 after
66 SecurityStateChanged: Symbol('SecurityStateChanged') 66 SecurityStateChanged: Symbol('SecurityStateChanged')
67 }; 67 };
68 68
69 69
70 /** 70 /**
71 * @unrestricted 71 * @unrestricted
72 */ 72 */
73 Security.PageSecurityState = class { 73 Security.PageSecurityState = class {
74 /** 74 /**
75 * @param {!Protocol.Security.SecurityState} securityState 75 * @param {!Protocol.Security.SecurityState} securityState
76 * @param {boolean} schemeIsCryptographic
76 * @param {!Array<!Protocol.Security.SecurityStateExplanation>} explanations 77 * @param {!Array<!Protocol.Security.SecurityStateExplanation>} explanations
77 * @param {?Protocol.Security.InsecureContentStatus} insecureContentStatus 78 * @param {?Protocol.Security.InsecureContentStatus} insecureContentStatus
78 * @param {boolean} schemeIsCryptographic 79 * @param {?string} summary
79 */ 80 */
80 constructor(securityState, explanations, insecureContentStatus, schemeIsCrypto graphic) { 81 constructor(securityState, schemeIsCryptographic, explanations, insecureConten tStatus, summary) {
81 this.securityState = securityState; 82 this.securityState = securityState;
83 this.schemeIsCryptographic = schemeIsCryptographic;
82 this.explanations = explanations; 84 this.explanations = explanations;
83 this.insecureContentStatus = insecureContentStatus; 85 this.insecureContentStatus = insecureContentStatus;
84 this.schemeIsCryptographic = schemeIsCryptographic; 86 this.summary = summary;
85 } 87 }
86 }; 88 };
87 89
88 /** 90 /**
89 * @implements {Protocol.SecurityDispatcher} 91 * @implements {Protocol.SecurityDispatcher}
90 * @unrestricted 92 * @unrestricted
91 */ 93 */
92 Security.SecurityDispatcher = class { 94 Security.SecurityDispatcher = class {
93 constructor(model) { 95 constructor(model) {
94 this._model = model; 96 this._model = model;
95 } 97 }
96 98
97 /** 99 /**
98 * @override 100 * @override
99 * @param {!Protocol.Security.SecurityState} securityState 101 * @param {!Protocol.Security.SecurityState} securityState
100 * @param {!Array<!Protocol.Security.SecurityStateExplanation>=} explanations 102 * @param {boolean} schemeIsCryptographic
101 * @param {!Protocol.Security.InsecureContentStatus=} insecureContentStatus 103 * @param {!Array<!Protocol.Security.SecurityStateExplanation>} explanations
102 * @param {boolean=} schemeIsCryptographic 104 * @param {!Protocol.Security.InsecureContentStatus} insecureContentStatus
105 * @param {?string=} summary
103 */ 106 */
104 securityStateChanged(securityState, explanations, insecureContentStatus, schem eIsCryptographic) { 107 securityStateChanged(securityState, schemeIsCryptographic, explanations, insec ureContentStatus, summary) {
105 var pageSecurityState = new Security.PageSecurityState( 108 var pageSecurityState = new Security.PageSecurityState(
106 securityState, explanations || [], insecureContentStatus || null, scheme IsCryptographic || false); 109 securityState, schemeIsCryptographic, explanations, insecureContentStatu s, summary || null);
107 this._model.dispatchEventToListeners(Security.SecurityModel.Events.SecurityS tateChanged, pageSecurityState); 110 this._model.dispatchEventToListeners(Security.SecurityModel.Events.SecurityS tateChanged, pageSecurityState);
108 } 111 }
109 }; 112 };
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698