Bug 35006 - fast/dom/global-constructors.html is failing based on previous tests
Summary: fast/dom/global-constructors.html is failing based on previous tests
Status: RESOLVED WORKSFORME
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC OS X 10.5
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks: 34984
  Show dependency treegraph
 
Reported: 2010-02-16 15:58 PST by Eric Seidel (no email)
Modified: 2011-04-12 18:34 PDT (History)
3 users (show)

See Also:


Attachments
Patch (1.35 KB, patch)
2010-04-01 22:18 PDT, Eric Seidel (no email)
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Eric Seidel (no email) 2010-02-16 15:58:38 PST
fast/dom/global-constructors.html is failing based on previous tests

This is when run via run-chromium-webkit-tests (which runs tests in parallel, and thus out of order).

-FAIL: window.CSSValue.prototype.isPrototypeOf(cssPrimitiveValue) should be true but instead is false
-FAIL: window.CSSPrimitiveValue.prototype.isPrototypeOf(cssPrimitiveValue) should be true but instead is false
+PASS: window.CSSValue.prototype.isPrototypeOf(cssPrimitiveValue) should be true and is.
+PASS: window.CSSPrimitiveValue.prototype.isPrototypeOf(cssPrimitiveValue) should be true and is.
Comment 1 Eric Seidel (no email) 2010-02-25 16:54:28 PST
run-chromium-webkit-tests is now called new-run-webkit-tests.
Comment 2 Eric Seidel (no email) 2010-04-01 21:53:56 PDT
Bug 37007 is another test failing for similar reasons.
Comment 3 Eric Seidel (no email) 2010-04-01 21:57:14 PDT
This failure would be caused by there being some toJS() call which is wrapping a CSSPrimitiveValue which is not passing the right globalObject() (or any global object).  Finding out which is causing the failure could be difficult, however fixing all of such possible bugs should be straightforward by removing the two-argument toJS calls and making sure that ll of those callsites are replaced by 3-argument toJS calls passing a proper globalObject().
Comment 4 Eric Seidel (no email) 2010-04-01 22:18:59 PDT
Created attachment 52377 [details]
Patch
Comment 5 Eric Seidel (no email) 2010-04-01 22:22:34 PDT
Committed r56967: <http://trac.webkit.org/changeset/56967>
Comment 6 WebKit Review Bot 2010-04-01 22:57:05 PDT
http://trac.webkit.org/changeset/56967 might have broken Chromium Win Release
Comment 7 Eric Seidel (no email) 2010-04-01 22:59:12 PDT
No, that was just the Chroimum builder being unreliable and disconnecting. :(
Comment 8 Dirk Pranke 2011-04-06 19:42:05 PDT
If this is even still failing, I don't think it should block bug 34984. I plan on verifying that it works now, but if it doesn't, I plan to clear the blocking flag on Mon 4/11. If you disagree, now would be a good time to speak up :).
Comment 9 Dirk Pranke 2011-04-12 18:34:49 PDT
This test no longer appears to be failing. Closing as WORKSFORME.