Enable layout tests: worker-constructor.html and worker-script-error.html....

Enable layout tests: worker-constructor.html and worker-script-error.html.

BUG=none
TEST=none


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=21863

[levin, 2009-07-27 22:42:59]

http://codereview.chromium.org/159429/diff/1/4
File
webkit/data/layout_tests/platform/chromium-mac/LayoutTests/fast/workers/worker-constructor-expected.txt
(right):

http://codereview.chromium.org/159429/diff/1/4#newcode1
Line 1: Test Worker constructor functionality. Should print a series of PASS
messages, followed with DONE.
Why do we have separate results for mac?  Aren't the expected result the same?

http://codereview.chromium.org/159429/diff/1/2
File webkit/glue/webworkerclient_impl.cc (right):

http://codereview.chromium.org/159429/diff/1/2#newcode337
Line 337: else
Why is this an "else"?  (I though reportException should be called depending on
the result of dispatchScriptErrorEvent.)

[jianli, 2009-07-28 00:48:15]

http://codereview.chromium.org/159429/diff/1/4
File
webkit/data/layout_tests/platform/chromium-mac/LayoutTests/fast/workers/worker-constructor-expected.txt
(right):

http://codereview.chromium.org/159429/diff/1/4#newcode1
Line 1: Test Worker constructor functionality. Should print a series of PASS
messages, followed with DONE.
On 2009/07/27 22:42:59, levin wrote:
> Why do we have separate results for mac?  Aren't the expected result the same?

Added the logic to reuse the expected result and deleted the files in non-win32
platform.

http://codereview.chromium.org/159429/diff/1/2
File webkit/glue/webworkerclient_impl.cc (right):

http://codereview.chromium.org/159429/diff/1/2#newcode337
Line 337: else
On 2009/07/27 22:42:59, levin wrote:
> Why is this an "else"?  (I though reportException should be called depending
on
> the result of dispatchScriptErrorEvent.)
> 

Done.

[levin, 2009-07-28 00:59:05]

LGTM

http://codereview.chromium.org/159429/diff/12/1014
File chrome/worker/worker_uitest.cc (right):

http://codereview.chromium.org/159429/diff/12/1014#newcode264
Line 264: &expected_result_value))
indent off.