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

Unified Diff: src/js/uri.js

Issue 1983593002: [builtins] Move EncodeURI from runtime to builtins. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Address review comments Created 4 years, 7 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.cc ('k') | src/runtime/runtime.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/js/uri.js
diff --git a/src/js/uri.js b/src/js/uri.js
index 37ba6b102fb738589e6b728cda2b9e46d77a5aed..19bfbd353de645f18fae01968a9ce041e577078b 100644
--- a/src/js/uri.js
+++ b/src/js/uri.js
@@ -219,18 +219,6 @@ function URIDecodeComponent(component) {
return Decode(component, reservedPredicate);
}
-// ECMA-262 - 15.1.3.3.
-function URIEncode(uri) {
- uri = TO_STRING(uri);
- return %URIEncode(uri, true);
-}
-
-// ECMA-262 - 15.1.3.4
-function URIEncodeComponent(component) {
- component = TO_STRING(component);
- return %URIEncode(component, false);
-}
-
// -------------------------------------------------------------------
// Install exported functions.
@@ -240,9 +228,7 @@ utils.InstallFunctions(global, DONT_ENUM, [
"escape", URIEscapeJS,
"unescape", URIUnescapeJS,
"decodeURI", URIDecode,
- "decodeURIComponent", URIDecodeComponent,
- "encodeURI", URIEncode,
- "encodeURIComponent", URIEncodeComponent
+ "decodeURIComponent", URIDecodeComponent
]);
})
« no previous file with comments | « src/builtins.cc ('k') | src/runtime/runtime.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698