Bug 86389 - [V8] Layout Test fast/dom/Window/window-property-shadowing-name.html is failing
Summary: [V8] Layout Test fast/dom/Window/window-property-shadowing-name.html is failing
Status: RESOLVED INVALID
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P1 Normal
Assignee: Erik Arvidsson
URL:
Keywords:
: 86593 (view as bug list)
Depends on:
Blocks:
 
Reported: 2012-05-14 12:08 PDT by Jian Li
Modified: 2012-05-16 12:43 PDT (History)
3 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Jian Li 2012-05-14 12:08:42 PDT 
The following layout test is failing on all platforms

fast/dom/Window/window-property-shadowing-name.html

Probable cause by V8 roll:

http://src.chromium.org/viewvc/chrome?view=rev&revision=136837
Comment 1 Alexey Proskuryakov 2012-05-14 13:44:57 PDT 
Is it failing on _all_ platforms, or only on those that use v8?
Comment 2 Jian Li 2012-05-14 13:47:26 PDT 
Failed on V8. Update title to reflect this.
Comment 3 Erik Arvidsson 2012-05-14 14:11:34 PDT 
This is part of a V8 fix to follow ES5.2.

I think we need to add this to test_expectations
Comment 4 Jian Li 2012-05-14 14:40:14 PDT 
(In reply to comment #3)
> This is part of a V8 fix to follow ES5.2.
> 
> I think we need to add this to test_expectations

Yes, it has been added to test_expectations.

The test should not timeout, as it is now.
Comment 5 Erik Arvidsson 2012-05-14 14:41:58 PDT 
(In reply to comment #4)
> The test should not timeout, as it is now.

I'll fix that too. Thanks for pointing that out.
Comment 6 Erik Arvidsson 2012-05-16 12:12:30 PDT 
Jian, I see that you removed the line from test_expectations again.

http://trac.webkit.org/changeset/117089/trunk/LayoutTests/platform/chromium/test_expectations.txt

I have a patch that makes the fail case faster.
Comment 7 Eric Seidel (no email) 2012-05-16 12:43:42 PDT 
*** Bug 86593 has been marked as a duplicate of this bug. ***