Refactoring on DOM/Promises

Refactoring on DOM/Promises

We are implementing DOM/promises[1].
This CL adds some layout tests and does some refactoring.
We change the status of "Promise" runtime enabled flag from test to experimental.

[1]: http://dom.spec.whatwg.org/#promises

R=abarth
BUG=243345
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=153798

[yhirano, 2013-07-09 05:06:35]

Hi, can you review this CL?
I changed the "Promise" flag status from test to experimental in this CL since
all APIs are implemented now.
If there is a problem to do that, please let me know.

[yhirano, 2013-07-09 05:38:55]

Sorry, the issue description was wrong. I fixed it.

[abarth-chromium, 2013-07-09 17:47:22]

On 2013/07/09 05:06:35, yhirano wrote:
> I changed the "Promise" flag status from test to experimental in this CL since
> all APIs are implemented now.

Great!

> If there is a problem to do that, please let me know.

That's exactly the right thing to do when all the parts of the feature are
implemented.

[abarth-chromium, 2013-07-09 17:49:27]

lgtm

https://codereview.chromium.org/18878002/diff/1/Source/bindings/v8/custom/V8P...
File Source/bindings/v8/custom/V8PromiseCustom.cpp (right):

https://codereview.chromium.org/18878002/diff/1/Source/bindings/v8/custom/V8P...
Source/bindings/v8/custom/V8PromiseCustom.cpp:54: v8::Local<v8::Function>
createClosure(v8::FunctionCallback function, v8::Handle<v8::Value> environment)
This seems like a really useful function.  Perhaps we should put it in
v8Bindings.h.

[commit-bot: I haz the power, 2013-07-09 17:49:42]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/yhirano@chromium.org/18878002/1

[commit-bot: I haz the power, 2013-07-09 18:54:09]

Change committed as 153798