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

Unified Diff: src/builtins/builtins-string.cc

Issue 2315343002: [builtins] Move StringNormalize to a cpp builtin. (Closed)
Patch Set: take 2 Created 4 years, 3 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 | « src/builtins/builtins.h ('k') | src/js/string.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/builtins/builtins-string.cc
diff --git a/src/builtins/builtins-string.cc b/src/builtins/builtins-string.cc
index d38f6b069d50f5e751d7dfb3147f602a25b2d4f6..612b70a9c696977ee95f70ad0dd5d01fb2bded40 100644
--- a/src/builtins/builtins-string.cc
+++ b/src/builtins/builtins-string.cc
@@ -477,6 +477,37 @@ void Builtins::Generate_StringPrototypeCharCodeAt(
assembler->Return(result);
}
+// ECMA-262 v6, section 21.1.3.12 String.prototype.normalize ( [form] )
+//
+// For now we do nothing, as proper normalization requires big tables.
+// If Intl is enabled, then i18n.js will override it and provide the the
+// proper functionality.
+BUILTIN(StringPrototypeNormalize) {
+ HandleScope handle_scope(isolate);
+ TO_THIS_STRING(string, "String.prototype.trim");
Franzi 2016/09/09 12:51:04 trim?
+
+ Handle<String> form;
+
+ if (args.atOrUndefined(isolate, 1)->IsUndefined(isolate)) {
+ return *string;
+ }
+ MaybeHandle<String> maybe_form(Object::ToString(isolate, args.at<Object>(1)));
Benedikt Meurer 2016/09/08 04:41:30 This is not correct, as the ToString can throw and
petermarshall 2016/09/09 11:41:07 Done.
+ maybe_form.ToHandle(&form);
+
+ if (!(form->Equals(*isolate->factory()->NewStringFromStaticChars("NFC")) ||
+ form->Equals(*isolate->factory()->NewStringFromStaticChars("NFD")) ||
+ form->Equals(*isolate->factory()->NewStringFromStaticChars("NFKC")) ||
+ form->Equals(*isolate->factory()->NewStringFromStaticChars("NFKD")))) {
+ Handle<String> validForms =
+ isolate->factory()->NewStringFromStaticChars("NFC, NFD, NFKC, NFKD");
+ THROW_NEW_ERROR_RETURN_FAILURE(
+ isolate,
+ NewRangeError(MessageTemplate::kNormalizationForm, validForms));
+ }
+
+ return *string;
+}
+
// ES6 section 21.1.3.25 String.prototype.toString ()
void Builtins::Generate_StringPrototypeToString(CodeStubAssembler* assembler) {
typedef compiler::Node Node;
« no previous file with comments | « src/builtins/builtins.h ('k') | src/js/string.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698