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

Unified Diff: src/builtins/builtins.h

Issue 2728763006: Migrate some case conversion functions from JS to CPP builtins (Closed)
Patch Set: flatten strings 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: src/builtins/builtins.h
diff --git a/src/builtins/builtins.h b/src/builtins/builtins.h
index df6ca6c0f10e1aef7b1abe6475bc0877a24894fa..e0d52bfa9d59e556bb3f90387339ff226e223607 100644
--- a/src/builtins/builtins.h
+++ b/src/builtins/builtins.h
@@ -785,14 +785,19 @@ class Isolate;
CPP(StringPrototypeStartsWith) \
/* ES6 section 21.1.3.25 String.prototype.toString () */ \
TFJ(StringPrototypeToString, 0) \
- /* ES #sec-string.prototype.tolocalelowercase */ \
+ /* (obsolete) Unibrow string functions */ \
CPP(StringPrototypeToLocaleLowerCase) \
- /* ES #sec-string.prototype.tolocaleuppercase */ \
CPP(StringPrototypeToLocaleUpperCase) \
- /* ES #sec-string.prototype.tolowercase */ \
CPP(StringPrototypeToLowerCase) \
- /* ES #sec-string.prototype.touppercase */ \
CPP(StringPrototypeToUpperCase) \
+ /* ES #sec-string.prototype.tolocalelowercase */ \
+ /* CPP(StringPrototypeToLocaleLowerCaseI18N) */ \
+ /* ES #sec-string.prototype.tolocaleuppercase */ \
+ /* CPP(StringPrototypeToLocaleUpperCaseI18N) */ \
+ /* ES #sec-string.prototype.tolowercase */ \
+ CPP(StringPrototypeToLowerCaseI18N) \
+ /* ES #sec-string.prototype.touppercase */ \
+ CPP(StringPrototypeToUpperCaseI18N) \
CPP(StringPrototypeTrim) \
CPP(StringPrototypeTrimLeft) \
CPP(StringPrototypeTrimRight) \

Powered by Google App Engine
This is Rietveld 408576698