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

Unified Diff: third_party/WebKit/Source/core/frame/Deprecation.cpp

Issue 1736533004: Adjust deprecation messages to match Estimated Stable Dates (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 10 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 | « third_party/WebKit/LayoutTests/transitions/cross-fade-border-image-expected.txt ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/Source/core/frame/Deprecation.cpp
diff --git a/third_party/WebKit/Source/core/frame/Deprecation.cpp b/third_party/WebKit/Source/core/frame/Deprecation.cpp
index c8ed59688cbbdaaa12f3fab2d79dff5a4efdca2d..c849511e5b771bc9ee959153cb3db3df7f6ad2bb 100644
--- a/third_party/WebKit/Source/core/frame/Deprecation.cpp
+++ b/third_party/WebKit/Source/core/frame/Deprecation.cpp
@@ -105,13 +105,16 @@ void Deprecation::countDeprecationIfNotPrivateScript(v8::Isolate* isolate, Execu
static const char* milestoneString(int milestone)
{
+ // These are the Estimated Stable Dates:
+ // https://www.chromium.org/developers/calendar
+
switch (milestone) {
case 50:
return "M50, around April 2016";
case 51:
- return "M51, around June 2016";
+ return "M51, around May 2016";
case 52:
- return "M52, around August 2016";
+ return "M52, around July 2016";
case 53:
return "M53, around September 2016";
}
@@ -324,7 +327,7 @@ String Deprecation::deprecationMessage(UseCounter::Feature feature)
return willBeRemoved("'SVGZoomEvent'", 52, "5760883808534528");
case UseCounter::BorderImageWithBorderStyleNone:
- return "Elements using the 'border-image' CSS property with no 'border-style' set should have no border, but currently do. Setting 'border-style' will be required in M51, around June 2016. See https://www.chromestatus.com/features/5542503914668032 for more details.";
+ return "Elements using the 'border-image' CSS property with no 'border-style' set should have no border, but currently do. Setting 'border-style' will be required in M51, around May 2016. See https://www.chromestatus.com/features/5542503914668032 for more details.";
case UseCounter::WebAnimationHyphenatedProperty:
return replacedWillBeRemoved("Hyphenated naming in Web Animations keyframes", "camelCase", 51, "5650817352728576");
« no previous file with comments | « third_party/WebKit/LayoutTests/transitions/cross-fade-border-image-expected.txt ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698