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

Unified Diff: Source/core/frame/UseCounter.cpp

Issue 1115913002: Add deprecation warnings to old powerful features on insecure origins. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Rebase on ToT Created 5 years, 8 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: Source/core/frame/UseCounter.cpp
diff --git a/Source/core/frame/UseCounter.cpp b/Source/core/frame/UseCounter.cpp
index 7cce5a0ef4b188f6b95262f9673cede6dd469b05..4d445415e6ec675d4a21b920897434a0f0612220 100644
--- a/Source/core/frame/UseCounter.cpp
+++ b/Source/core/frame/UseCounter.cpp
@@ -899,6 +899,19 @@ String UseCounter::deprecationMessage(Feature feature)
case CSSKeyframesRuleInsertRule:
return "'CSSKeyframesRule.insertRule()' is deprecated and will be removed in M45, around August 2015. Please use 'CSSKeyframesRule.appendRule()' instead.";
+ // Powerful features on insecure origins (https://goo.gl/rStTGz)
+ case GeolocationInsecureOrigin:
+ return "Geolocation via getCurrentPosition() and watchPosition() will be deprecated over insecure origins in the future. You should consider switching your application to a secure origin, such as HTTPS. See http://goo.gl/rStTGz for more details.";
Michael van Ouwerkerk 2015/04/30 13:15:21 nit: might as well use https for all the goo.gl li
jww 2015/04/30 17:28:45 For sure. At least I got it right in the comment :
+
+ case GetUserMediaInsecureOrigin:
+ return "getUserMedia() will be deprecated over insecure origins in the future. You should consider switching your application to a secure origin, such as HTTPS. See http://goo.gl/rStTGz for more details.";
+
+ case DeviceMotionInsecureOrigin:
+ return "The 'devicemotion' event will be deprecated over insecure origins in the future. You should consider switching your application to a secure origin, such as HTTPS. See http://goo.gl/rStTGz for more details.";
+
+ case FullscreenInsecureOrigin:
+ return "requestFullscreen() will be deprecated over insecure origins in the future. You should consider switching your application to a secure origin, such as HTTPS. See http://goo.gl/rStTGz for more details.";
+
// Features that aren't deprecated don't have a deprecation message.
default:
return String();

Powered by Google App Engine
This is Rietveld 408576698