Pass in the map while creating JSFunction, and set prototype in the client.

Pass in the map while creating JSFunction, and set prototype in the client.

BUG=
R=ishell@chromium.org

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

[Toon Verwaest, 2014-05-05 09:30:03]

PTAL

[Igor Sheludko, 2014-05-05 16:11:34]

lgtm with comments:

https://codereview.chromium.org/263083008/diff/1/src/factory.cc
File src/factory.cc (right):

https://codereview.chromium.org/263083008/diff/1/src/factory.cc#newcode1210
src/factory.cc:1210: Handle<Context>
context(isolate()->context()->native_context());
Does it make sense to add ASSERT(info->strict_mode() == SLOPPY); here to
guarantee that the info is in sync with the map?

https://codereview.chromium.org/263083008/diff/1/src/factory.cc#newcode1226
src/factory.cc:1226: Handle<Context>
context(isolate()->context()->native_context());
... and here?

https://codereview.chromium.org/263083008/diff/1/src/factory.cc#newcode1347
src/factory.cc:1347: Handle<Map> map = MapForNewFunction(isolate(), info);
Since this is the only caller of MapForNewFunction() and the function itself if
pretty small consider inlining it.

[Toon Verwaest, 2014-05-06 11:26:43]

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