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

Side by Side Diff: Source/core/frame/UseCounter.cpp

Issue 966843002: Mark DOMStringList.contains() as deprecated, add console warning. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Deprecate named getter 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 1
2 /* 2 /*
3 * Copyright (C) 2012 Google, Inc. All rights reserved. 3 * Copyright (C) 2012 Google, Inc. All rights reserved.
4 * 4 *
5 * Redistribution and use in source and binary forms, with or without 5 * Redistribution and use in source and binary forms, with or without
6 * modification, are permitted provided that the following conditions 6 * modification, are permitted provided that the following conditions
7 * are met: 7 * are met:
8 * 1. Redistributions of source code must retain the above copyright 8 * 1. Redistributions of source code must retain the above copyright
9 * notice, this list of conditions and the following disclaimer. 9 * notice, this list of conditions and the following disclaimer.
10 * 2. Redistributions in binary form must reproduce the above copyright 10 * 2. Redistributions in binary form must reproduce the above copyright
(...skipping 849 matching lines...) Expand 10 before | Expand all | Expand 10 after
860 860
861 case SVGSVGElementUnsuspendRedrawAll: 861 case SVGSVGElementUnsuspendRedrawAll:
862 return "'SVGSVGElement.unsuspendRedrawAll()' is deprecated, please do no t use it. It is a no-op, as per SVG2 (https://svgwg.org/svg2-draft/struct.html#_ _svg__SVGSVGElement__unsuspendRedrawAll)."; 862 return "'SVGSVGElement.unsuspendRedrawAll()' is deprecated, please do no t use it. It is a no-op, as per SVG2 (https://svgwg.org/svg2-draft/struct.html#_ _svg__SVGSVGElement__unsuspendRedrawAll).";
863 863
864 case ServiceWorkerClientPostMessage: 864 case ServiceWorkerClientPostMessage:
865 return "'Client.postMessage()' is an experimental API and may change. Se e https://github.com/slightlyoff/ServiceWorker/issues/609."; 865 return "'Client.postMessage()' is an experimental API and may change. Se e https://github.com/slightlyoff/ServiceWorker/issues/609.";
866 866
867 case ServiceWorkerClientsGetAll: 867 case ServiceWorkerClientsGetAll:
868 return "'Clients.getAll()' is deprecated and will be removed soon. Pleas e use 'Clients.matchAll()' instead."; 868 return "'Clients.getAll()' is deprecated and will be removed soon. Pleas e use 'Clients.matchAll()' instead.";
869 869
870 case DOMStringListContains:
871 return "'DOMStringList.contains()' is deprecated, and will be removed.";
872
873 case DOMStringListItem:
874 return "'DOMStringList.item(n)' is deprecated, and will be removed. Plea se use indexing instead, e.g.: list[n]";
875
870 // Features that aren't deprecated don't have a deprecation message. 876 // Features that aren't deprecated don't have a deprecation message.
871 default: 877 default:
872 return String(); 878 return String();
873 } 879 }
874 } 880 }
875 881
876 void UseCounter::count(CSSParserContext context, CSSPropertyID feature) 882 void UseCounter::count(CSSParserContext context, CSSPropertyID feature)
877 { 883 {
878 ASSERT(feature >= firstCSSProperty); 884 ASSERT(feature >= firstCSSProperty);
879 ASSERT(feature <= lastCSSProperty); 885 ASSERT(feature <= lastCSSProperty);
(...skipping 27 matching lines...) Expand all
907 UseCounter* UseCounter::getFrom(const StyleSheetContents* sheetContents) 913 UseCounter* UseCounter::getFrom(const StyleSheetContents* sheetContents)
908 { 914 {
909 // FIXME: We may want to handle stylesheets that have multiple owners 915 // FIXME: We may want to handle stylesheets that have multiple owners
910 // http://crbug.com/242125 916 // http://crbug.com/242125
911 if (sheetContents && sheetContents->hasSingleOwnerNode()) 917 if (sheetContents && sheetContents->hasSingleOwnerNode())
912 return getFrom(sheetContents->singleOwnerDocument()); 918 return getFrom(sheetContents->singleOwnerDocument());
913 return 0; 919 return 0;
914 } 920 }
915 921
916 } // namespace blink 922 } // namespace blink
OLDNEW
« Source/core/dom/DOMStringList.idl ('K') | « Source/core/frame/UseCounter.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698