Add layout test to verify that global constructors in workers environment have right attributes

Add layout test to verify that global constructors in workers environment have right attributes
    
New test is added to webexposed/ folder.
The corresponding test for non-workers is moved the webexposed/ folder as well for consistency.

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=152107

[haraken, 2013-06-06 06:45:27]

LGTM

https://codereview.chromium.org/16528005/diff/1/LayoutTests/fast/js/script-te...
File LayoutTests/fast/js/script-tests/global-constructors-attributes-worker.js
(right):

https://codereview.chromium.org/16528005/diff/1/LayoutTests/fast/js/script-te...
LayoutTests/fast/js/script-tests/global-constructors-attributes-worker.js:27:
descriptorShouldBe("global", "'Float64Array'", {writable: true, enumerable:
false, configurable: true, value:"Float64Array"});

Aren't other typed arrays exposed on workers?

[do-not-use, 2013-06-06 06:53:26]

https://codereview.chromium.org/16528005/diff/1/LayoutTests/fast/js/script-te...
File LayoutTests/fast/js/script-tests/global-constructors-attributes-worker.js
(right):

https://codereview.chromium.org/16528005/diff/1/LayoutTests/fast/js/script-te...
LayoutTests/fast/js/script-tests/global-constructors-attributes-worker.js:27:
descriptorShouldBe("global", "'Float64Array'", {writable: true, enumerable:
false, configurable: true, value:"Float64Array"});
On 2013/06/06 06:45:27, haraken wrote:
> 
> Aren't other typed arrays exposed on workers?

They are. This test just tests a few of the exposed constructors (similarly to
the non-worker test: fast/js/script-tests/global-constructors-attributes.js).

We already have tests in webexposed/ that lists all the exposed constructors on
Window and WorkerContext. This one is merely for checking the attributes.

[haraken, 2013-06-06 06:54:25]

Makes sense!

[commit-bot: I haz the power, 2013-06-06 06:55:13]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ch.dumez@sisa.samsung.com/16528005/1

[commit-bot: I haz the power, 2013-06-06 07:26:02]

Retried try job too often on linux_layout_rel for step(s) webkit_unit_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_layo...

[eseidel, 2013-06-06 08:56:40]

If you'd liek this tested with teh stable configuration of Chromium (the flags
we actually ship) should should consider adding this to webexposed/ instead. 
I've not turned on that testing yet, but it's now possible with content_shell.
:)

[do-not-use, 2013-06-06 10:12:42]

On 2013/06/06 08:56:40, eseidel wrote:
> If you'd liek this tested with teh stable configuration of Chromium (the flags
> we actually ship) should should consider adding this to webexposed/ instead. 
> I've not turned on that testing yet, but it's now possible with content_shell.
> :)

Done. I have moved the corresponding test for non-workers as well, for
consistency.

[do-not-use, 2013-06-10 05:49:19]

On 2013/06/06 10:12:42, Christophe Dumez wrote:
> On 2013/06/06 08:56:40, eseidel wrote:
> > If you'd liek this tested with teh stable configuration of Chromium (the
flags
> > we actually ship) should should consider adding this to webexposed/ instead.

> > I've not turned on that testing yet, but it's now possible with
content_shell.
> > :)
> 
> Done. I have moved the corresponding test for non-workers as well, for
> consistency.

Is the change OK?

[haraken, 2013-06-10 05:49:53]

> > Done. I have moved the corresponding test for non-workers as well, for
> > consistency.
> 
> Is the change OK?

Sure. Still LGTM.

[commit-bot: I haz the power, 2013-06-10 05:50:51]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ch.dumez@sisa.samsung.com/16528005/12001

[commit-bot: I haz the power, 2013-06-10 06:02:22]

Retried try job too often on blink_presubmit for step(s) presubmit
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=blink_pres...

[commit-bot: I haz the power, 2013-06-10 07:25:08]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ch.dumez@sisa.samsung.com/16528005/23001

[commit-bot: I haz the power, 2013-06-10 08:03:41]

Change committed as 152107