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

Unified Diff: Source/devtools/protocol.json

Issue 1167693010: Add a DevTools Security domain to protocol.json. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Created 5 years, 7 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 e11674e249baca518c6e2c93fe79765ed95feac1..da4ac635b64abcffc3f3e857e185aaf7c54e6e16 100644
--- a/Source/devtools/protocol.json
+++ b/Source/devtools/protocol.json
@@ -1057,6 +1057,44 @@
]
},
{
+ "domain": "Security",
+ "description": "Security",
pfeldman 2015/06/04 09:21:55 add "hidden": true
lgarron 2015/06/09 00:15:17 Done.
+ "types": [
+ {
+ "id": "SecurityState",
lgarron 2015/06/04 02:31:33 estark@: I used "SecurityState" here because "Secu
+ "type": "string",
+ "enum": ["SECURE", "WARNING", "INSECURE", "HTTP"],
pfeldman 2015/06/04 09:21:55 Please use lower case for the protocol enum litera
lgarron 2015/06/09 00:15:17 Done.
+ "description": "The security level of a page or resource."
+ }
+ ],
+ "events": [
+ {
+ "name": "securityStateChanged",
+ "description": "The security state of the page changed.",
+ "parameters": [
+ { "name": "securityState", "$ref": "SecurityState", "description": "Security state." }
+ ],
+ "handlers": ["browser"]
+ }
+ ],
+ "commands": [
pfeldman 2015/06/04 09:21:54 types, then commands, then events.
lgarron 2015/06/09 00:15:17 Done.
+ {
+ "name": "enable",
+ "parameters": [],
pfeldman 2015/06/04 09:21:55 no need to specify
lgarron 2015/06/09 00:15:17 Done.
+ "description": "Enables tracking security state changes.",
+ "handlers": ["browser"],
+ "hidden": true
pfeldman 2015/06/04 09:21:55 lets hide entire domain instead.
lgarron 2015/06/09 00:15:16 Done.
+ },
+ {
+ "name": "disable",
+ "parameters": [],
+ "description": "Disables tracking security state changes.",
+ "handlers": ["browser"],
+ "hidden": true
pfeldman 2015/06/04 09:21:55 ditto
lgarron 2015/06/09 00:15:17 Done (also with parameters).
+ }
+ ]
+ },
+ {
"domain": "Network",
"description": "Network domain allows tracking network activities of the page. It exposes information about http, file, data and other requests and responses, their headers, bodies, timing, etc.",
"types": [
« 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