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

Unified Diff: Source/modules/permissions/Permissions.idl

Issue 1057193003: Permissions: add UMA for navigator.permissions.query() usage. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@permissions_required
Patch Set: Created 5 years, 9 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 | « Source/core/frame/UseCounter.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/modules/permissions/Permissions.idl
diff --git a/Source/modules/permissions/Permissions.idl b/Source/modules/permissions/Permissions.idl
index 89f7dfd085feaf198c4d532d4c1d92001f12a072..bfe878f56e9b609fa28bdd3b207f67bf823a8ce2 100644
--- a/Source/modules/permissions/Permissions.idl
+++ b/Source/modules/permissions/Permissions.idl
@@ -10,5 +10,5 @@
GarbageCollected,
RuntimeEnabled=Permissions
] interface Permissions {
- [CallWith=ScriptState] Promise<PermissionStatus> query(object permission);
+ [CallWith=ScriptState, MeasureAs=PermissionsQuery] Promise<PermissionStatus> query(object permission);
philipj_slow 2015/04/03 16:49:39 As of https://codereview.chromium.org/877523004 it
mlamouri (slow - plz ping) 2015/04/03 17:49:48 Done.
};
« no previous file with comments | « Source/core/frame/UseCounter.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698