DBC.
https://codereview.chromium.org/263833007/diff/80001/src/object-observe.js
File src/object-observe.js (right):
https://codereview.chromium.org/263833007/diff/80001/src/object-observe.js#newcode545
src/object-observe.js:545: function
NativeObjectNotifierPerformChange(objectInfo, changeType, changeFn) {
Please avoid name clashes of JS builtins and runtime functions.
https://codereview.chromium.org/263833007/diff/80001/src/runtime.cc
File src/runtime.cc (right):
https://codereview.chromium.org/263833007/diff/80001/src/runtime.cc#newcode15003
src/runtime.cc:15003: CONVERT_ARG_HANDLE_CHECKED(Object, callback, 1);
Please be as specific as you can regarding the object type here. I have
a feeling |callback| must be a JSFunction, so use
CONVERT_ARG_HANDLE_CHECKED(JSFunction, callback, 1);
https://codereview.chromium.org/263833007/diff/80001/src/runtime.cc#newcode15037
src/runtime.cc:15037:
RUNTIME_FUNCTION(Runtime_NativeObjectNotifierPerformChange) {
I don't see this function declared in runtime.h, is that intentional?
https://codereview.chromium.org/263833007/
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
