Cleanup of HEAP and FACTORY macro usage in accessors.cc.

Cleanup of HEAP and FACTORY macro usage in accessors.cc.


BUG=
TEST=

[Mads Ager (chromium), 2011-03-28 12:59:52]

http://codereview.chromium.org/6756003/diff/1/src/accessors.cc
File src/accessors.cc (right):

http://codereview.chromium.org/6756003/diff/1/src/accessors.cc#newcode122
src/accessors.cc:122: Isolate* isolate = Isolate::Current();
Please get the isolate from the object parameter instead of taking it through
thread-local storage: object->GetIsolate().

Also, please extract it before creating the handle scope and pass it in as an
argument there as well. Also pass it to the handle constructor.

http://codereview.chromium.org/6756003/diff/1/src/accessors.cc#newcode320
src/accessors.cc:320: Isolate* isolate = Isolate::Current();
Please extract the isolate from the object. The object is casted to a JSValue
which is a heap object from which you can get the isolate. Pass the isolate
pointer to the handle scope constructor and the handle constructor.

[Peter Varga, 2011-03-28 14:09:05]

The fixed patch is sent to the Patch Set 2.

The server didn't send notification about the update. I don't know what is the
proposed way to update a patch...

Thanks for the review.

[Vitaly Repeshko, 2011-03-29 06:55:02]

LGTM. I'll land.


Thanks,
Vitaly