WeakHashTable::Put() and callers handlified.

WeakHashTable::Put() and callers handlified.

R=yangguo@chromium.org

Committed: https://code.google.com/p/v8/source/detail?r=20959

[Igor Sheludko, 2014-04-24 15:32:19]

PTAL #10.

[Yang, 2014-04-25 07:32:26]

lgtm with comments.

https://codereview.chromium.org/253393006/diff/1/src/objects.cc
File src/objects.cc (right):

https://codereview.chromium.org/253393006/diff/1/src/objects.cc#newcode16270
src/objects.cc:16270: Handle<Object> value) {
This method cannot allocate, right? Otherwise the this-pointer can be corrupted.
We should use a DisallowHeapAllocation scope.

https://codereview.chromium.org/253393006/diff/1/src/objects.h
File src/objects.h (right):

https://codereview.chromium.org/253393006/diff/1/src/objects.h#newcode4528
src/objects.h:4528: Handle<Object> key,
please align arguments

[Igor Sheludko, 2014-04-25 08:10:33]

https://codereview.chromium.org/253393006/diff/1/src/objects.cc
File src/objects.cc (right):

https://codereview.chromium.org/253393006/diff/1/src/objects.cc#newcode16270
src/objects.cc:16270: Handle<Object> value) {
On 2014/04/25 07:32:27, Yang wrote:
> This method cannot allocate, right? Otherwise the this-pointer can be
corrupted.
> We should use a DisallowHeapAllocation scope.

Done.

https://codereview.chromium.org/253393006/diff/1/src/objects.h
File src/objects.h (right):

https://codereview.chromium.org/253393006/diff/1/src/objects.h#newcode4528
src/objects.h:4528: Handle<Object> key,
On 2014/04/25 07:32:27, Yang wrote:
> please align arguments

Done.

[Igor Sheludko, 2014-04-25 08:10:52]

Committed patchset #2 manually as r20959 (presubmit successful).