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

Unified Diff: tools/metrics/actions/actions.xml

Issue 813873005: Add field trial and flag to mark HTTP as non-secure. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Only badge if SECURITY_STYLE_UNAUTHENTICATED && unauthenticated network transport. (E.g. not chrome… Created 6 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 side-by-side diff with in-line comments
Download patch
Index: tools/metrics/actions/actions.xml
diff --git a/tools/metrics/actions/actions.xml b/tools/metrics/actions/actions.xml
index ab2f7da5d308318039df41afddb2ea3b9d2b206f..322a845391306af4c9f35ebefadb5cb1341a4fde 100644
--- a/tools/metrics/actions/actions.xml
+++ b/tools/metrics/actions/actions.xml
@@ -182,6 +182,14 @@ should be able to be added at any place in this file.
<description>Please enter the description of this user action.</description>
</action>
+<action name="AboutFlags_mark-http-as-non-secure">
+ <owner>palmer@chromium.org, felt@chromium.org</owner>
Mark P 2015/01/02 19:14:28 owner is a repeated field, one e-mail address per
palmer 2015/01/07 00:59:00 Done.
+ <description>
+ This flag lets users opt into or out of marking HTTP as Dubious or as
Mark P 2015/01/02 19:14:28 Actions need to clearly state when they are emitte
palmer 2015/01/07 00:59:00 I'm not sure what you mean. I think I am following
Mark P 2015/01/07 22:43:58 Ah, I didn't notice this was using that action rec
+ Non-Secure.
+ </description>
+</action>
+
<action name="AboutFlags_match-preview">
<owner>Please list the metric's owners. Add more owner tags as needed.</owner>
<description>Please enter the description of this user action.</description>

Powered by Google App Engine
This is Rietveld 408576698