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

Unified Diff: Source/devtools/protocol.json

Issue 1195533002: Update protocol.json to support explanations for securityStateChanged. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Use different SecurityStateExplanation property names. Created 5 years, 6 months 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/devtools/protocol.json
diff --git a/Source/devtools/protocol.json b/Source/devtools/protocol.json
index 090678dbf1739e49d87fc4126ed6f37af69bc981..ce5b60d726c5d21df04e366934111f8cc9a5af8b 100644
--- a/Source/devtools/protocol.json
+++ b/Source/devtools/protocol.json
@@ -5010,6 +5010,16 @@
"type": "string",
"enum": ["unknown", "http", "insecure", "warning", "secure"],
"description": "The security level of a page or resource."
+ },
+ {
+ "id": "SecurityStateExplanation",
pfeldman 2015/06/18 06:45:15 StateExplanation
+ "type": "object",
+ "properties": [
+ { "name": "securityState", "$ref": "SecurityState", "description": "Security state representing the severity of the factor being explained." },
+ { "name": "summary", "type": "string", "description": "Short phrase describing the type of factor." },
+ { "name": "description", "type": "string", "description": "Full text explanation of the factor." }
+ ],
+ "description": "An explanation of an factor contributing to the security state."
}
],
"commands": [
@@ -5029,7 +5039,8 @@
"name": "securityStateChanged",
"description": "The security state of the page changed.",
"parameters": [
- { "name": "securityState", "$ref": "SecurityState", "description": "Security state." }
+ { "name": "securityState", "$ref": "SecurityState", "description": "Security state." },
+ { "name": "explanations", "type": "array", "items": { "$ref": "SecurityStateExplanation" }, "description": "List of explanations for the security state. If the overall security state is `insecure` or `warning`, at least one corresponding explanation should be included.", "optional": true }
],
"handlers": ["browser"]
}
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698