[generators] Simplify %CreateJSGeneratorObject

[generators] Simplify %CreateJSGeneratorObject

Instead of looking at the top JavaScriptFrame, take the needed information as
arguments.  This is in preparation of the new generators implementation.

R=adamk@chromium.org, bmeurer@chromium.org
BUG=v8:4907
LOG=n

Committed: https://crrev.com/e298646be8f2de00e37dc43dad8968f269c9ce0f
Cr-Commit-Position: refs/heads/master@{#35430}

[Benedikt Meurer, 2016-04-12 12:57:03]

LGTM with nit.

https://codereview.chromium.org/1883443003/diff/1/src/runtime/runtime-generat...
File src/runtime/runtime-generator.cc (right):

https://codereview.chromium.org/1883443003/diff/1/src/runtime/runtime-generat...
src/runtime/runtime-generator.cc:17: DCHECK(args.length() == 2);
Nit: DCHECK_EQ(2, args.length())

[adamk, 2016-04-12 18:04:41]

lgtm

I would suggest filing a tracking bug to track work on the new generators
implementation, and attaching this CL to that bug. The tracking bug would be a
convenient place to gather all work for this feature, as well as to link other
info, such as a design doc.

[neis, 2016-04-13 07:51:46]

Description was changed from

==========
[generators] Simplify %CreateJSGeneratorObject

Instead of looking at the top JavaScriptFrame, take the needed information as
arguments.  This is in preparation of the new generators implementation.

R=adamk@chromium.org, bmeurer@chromium.org
BUG=
==========

to

==========
[generators] Simplify %CreateJSGeneratorObject

Instead of looking at the top JavaScriptFrame, take the needed information as
arguments.  This is in preparation of the new generators implementation.

R=adamk@chromium.org, bmeurer@chromium.org
BUG=v8:4907
LOG=n
==========

[neis, 2016-04-13 07:52:30]

The CQ bit was checked by neis@chromium.org

[neis, 2016-04-13 07:52:31]

The patchset sent to the CQ was uploaded after l-g-t-m from
bmeurer@chromium.org, adamk@chromium.org
Link to the patchset: https://codereview.chromium.org/1883443003/#ps20001
(title: "Add back initialization of continuation, just to be on the safe side.")

[commit-bot: I haz the power, 2016-04-13 07:52:41]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1883443003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1883443003/20001

[commit-bot: I haz the power, 2016-04-13 08:16:36]

Description was changed from

==========
[generators] Simplify %CreateJSGeneratorObject

Instead of looking at the top JavaScriptFrame, take the needed information as
arguments.  This is in preparation of the new generators implementation.

R=adamk@chromium.org, bmeurer@chromium.org
BUG=v8:4907
LOG=n
==========

to

==========
[generators] Simplify %CreateJSGeneratorObject

Instead of looking at the top JavaScriptFrame, take the needed information as
arguments.  This is in preparation of the new generators implementation.

R=adamk@chromium.org, bmeurer@chromium.org
BUG=v8:4907
LOG=n
==========

[commit-bot: I haz the power, 2016-04-13 08:16:37]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-04-13 08:18:25]

Description was changed from

==========
[generators] Simplify %CreateJSGeneratorObject

Instead of looking at the top JavaScriptFrame, take the needed information as
arguments.  This is in preparation of the new generators implementation.

R=adamk@chromium.org, bmeurer@chromium.org
BUG=v8:4907
LOG=n
==========

to

==========
[generators] Simplify %CreateJSGeneratorObject

Instead of looking at the top JavaScriptFrame, take the needed information as
arguments.  This is in preparation of the new generators implementation.

R=adamk@chromium.org, bmeurer@chromium.org
BUG=v8:4907
LOG=n

Committed: https://crrev.com/e298646be8f2de00e37dc43dad8968f269c9ce0f
Cr-Commit-Position: refs/heads/master@{#35430}
==========

[commit-bot: I haz the power, 2016-04-13 08:18:26]

Patchset 2 (id:??) landed as
https://crrev.com/e298646be8f2de00e37dc43dad8968f269c9ce0f
Cr-Commit-Position: refs/heads/master@{#35430}