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

Unified Diff: src/js/proxy.js

Issue 1419813010: [runtime] Remove the very dangerous %_CallFunction intrinsic. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Created 5 years, 1 month 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/js/promise.js ('k') | src/js/string.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/js/proxy.js
diff --git a/src/js/proxy.js b/src/js/proxy.js
index 56373861eec5676dcc817a57ac0586587ee6a917..fc38680a13a8daae96d7df99e3acf839ecb35232 100644
--- a/src/js/proxy.js
+++ b/src/js/proxy.js
@@ -81,7 +81,7 @@ function DerivedGetTrap(receiver, name) {
} else {
if (IS_UNDEFINED(desc.get)) { return desc.get }
// The proposal says: desc.get.call(receiver)
- return %_CallFunction(receiver, desc.get)
+ return %_Call(desc.get, receiver)
}
}
@@ -99,7 +99,7 @@ function DerivedSetTrap(receiver, name, val) {
} else { // accessor
if (desc.set) {
// The proposal says: desc.set.call(receiver, val)
- %_CallFunction(receiver, val, desc.set)
+ %_Call(desc.set, receiver, val)
return true
} else {
return false
@@ -117,7 +117,7 @@ function DerivedSetTrap(receiver, name, val) {
} else { // accessor
if (desc.set) {
// The proposal says: desc.set.call(receiver, val)
- %_CallFunction(receiver, val, desc.set)
+ %_Call(desc.set, receiver, val)
return true
} else {
return false
« no previous file with comments | « src/js/promise.js ('k') | src/js/string.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698