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

Unified Diff: third_party/WebKit/Source/modules/installedapp/NavigatorInstalledApp.idl

Issue 2785913003: Add an Origin Trial for Navigator.getInstalledRelatedApps. (Closed)
Patch Set: Fix tests and make non-Android-specific. Created 3 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
Index: third_party/WebKit/Source/modules/installedapp/NavigatorInstalledApp.idl
diff --git a/third_party/WebKit/Source/modules/installedapp/NavigatorInstalledApp.idl b/third_party/WebKit/Source/modules/installedapp/NavigatorInstalledApp.idl
index 1d2f914e3674ba38a402633e94e54c86c9c60162..dc2b26d586e43c8e4e3627f4fedca912b6efded1 100644
--- a/third_party/WebKit/Source/modules/installedapp/NavigatorInstalledApp.idl
+++ b/third_party/WebKit/Source/modules/installedapp/NavigatorInstalledApp.idl
@@ -6,7 +6,11 @@
// https://github.com/WICG/get-installed-related-apps/blob/master/EXPLAINER.md
[
- RuntimeEnabled=InstalledApp,
+ OriginTrialEnabled=InstalledApp,
] partial interface Navigator {
- [SecureContext, CallWith=ScriptState, Measure] Promise<RelatedApplication> getInstalledRelatedApps();
+ // TODO(mgiuca): Add [SecureContext] to this method. Currently prevented due
+ // to clash with OriginTrialEnabled (so it is manually checked in the method
+ // installation code instead). This can be resolved either when
+ // OriginTrialEnabled is removed, or https://crbug.com/695123 is fixed.
+ [CallWith=ScriptState, Measure] Promise<RelatedApplication> getInstalledRelatedApps();
};

Powered by Google App Engine
This is Rietveld 408576698