Closed
Bug 700429
Opened 13 years ago
Closed 12 years ago
mochitest-1 test_webgl_conformance_test_suite unexpected-passes are good passes on lion
Categories
(Core :: Graphics: CanvasWebGL, defect)
Tracking
()
RESOLVED
FIXED
mozilla13
People
(Reporter: philor, Assigned: bjacob)
References
Details
(Whiteboard: [qa-])
Attachments
(2 files, 1 obsolete file)
3.55 KB,
patch
|
akeybl
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
3.66 KB,
patch
|
akeybl
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
I hid/am hiding all 10.7 tests, because I don't have time to deal with them today; the green stuff will need to be unhidden, perhaps on a less stressful day like tomorrow. So far, hidden on: mozilla-central/Firefox mozilla-inbound Fx-Team Services-Central mozilla-aurora
Reporter | ||
Comment 1•13 years ago
|
||
Try (where someone got to them before I remembered to)
Reporter | ||
Comment 2•13 years ago
|
||
Ionmonkey
Assignee | ||
Comment 3•13 years ago
|
||
The remaining mochitest-1 failures are two unexpected-passes in the WebGL mochitest: 46313 ERROR TEST-UNEXPECTED-PASS | /tests/content/canvas/test/webgl/test_webgl_conformance_test_suite.html | Test expected to fail, but passed: conformance/more/functions/uniformfBadArgs.html 46314 ERROR TEST-UNEXPECTED-PASS | /tests/content/canvas/test/webgl/test_webgl_conformance_test_suite.html | Test expected to fail, but passed: conformance/more/functions/uniformiBadArgs.html Patch coming.
Assignee | ||
Comment 4•13 years ago
|
||
on tryserver: https://tbpl.mozilla.org/?tree=Try&rev=cd828ff0c427 probably need to append ?noignore=1&jobname=lion to see the lion results.
Attachment #572827 -
Flags: review?(jgilbert)
Reporter | ||
Comment 5•13 years ago
|
||
Unhid the green stuff on mozilla-central. bjacob: since we're running them on Aurora (and I presume Beta as well), you and everyone else is going to need approval flags and status flags, so you'll want to file a separate bug for your patch. Bug numbers are cheap, and we need to hurry up and get to 800000 so we can have another contest ;)
Reporter | ||
Comment 6•13 years ago
|
||
Unhid on inbound, fx-team.
Reporter | ||
Comment 7•13 years ago
|
||
Unhid on Services-Central, Try, and Ionmonkey
Comment 8•13 years ago
|
||
Comment on attachment 572827 [details] [diff] [review] pass mochitest-1 on lion slaves Review of attachment 572827 [details] [diff] [review]: ----------------------------------------------------------------- Looks good. Is there a wiki page somewhere that covers the different places that we specify which tests are expected to fail? We have the main three files, but we should also make it clear that we insert sub-platform-dependent ones in test_webgl_conformance_test_suite.html. ::: content/canvas/test/webgl/test_webgl_conformance_test_suite.html @@ +327,5 @@ > // code borrowed from browser/components/wintaskbar/test/browser_taskbar_preview.js > var is106orHigher = false; > netscape.security.PrivilegeManager.enablePrivilege("UniversalXPConnect"); > + kDarwinVersion = parseFloat(Components.classes["@mozilla.org/system-info;1"] > + .getService(Components.interfaces.nsIPropertyBag2) Looks like the indenting might have gotten messed up. (Looks wrong in splinter)
Attachment #572827 -
Flags: review?(jgilbert) → review+
Comment 9•13 years ago
|
||
Yeah, please file new bugs for each set of tests.
Comment 10•13 years ago
|
||
Anything left to be done in this bug?
Comment 11•13 years ago
|
||
Or are we keeping it open until all 10.7 suites are green *and* unhidden?
Comment 12•13 years ago
|
||
The work to get all 10.7 tests green and unhidden is bug 700503. This work is specifically for the unhidding of temporarily hidden already green tests.
Comment 13•13 years ago
|
||
OK cool. FIXING for unhidding the already green tests.
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Comment 14•13 years ago
|
||
Actually, lets morph this bug to track that patch that has been uploaded here and gotten review to fix mochitest-1. Not sure which production or component to put this into.
Assignee: nobody → bjacob
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Summary: Unhide green 10.7 tests → mochitest-1 test_webgl_conformance_test_suite unexpected-passes are good passes on lion
Reporter | ||
Updated•13 years ago
|
Component: Release Engineering → Canvas: WebGL
Product: mozilla.org → Core
QA Contact: release → canvas.webgl
Version: other → Trunk
Blocks: 700503
Comment 15•12 years ago
|
||
Benoit, philor pointed out that this patch never was landed. Want to de-rot it and try again?
Reporter | ||
Comment 16•12 years ago
|
||
https://tbpl.mozilla.org/?tree=Try&rev=4ca95d3464c5&noignore=1 - I discovered that *after* I'd already rewritten it (rather more ham-fistedly), and pointed it out *while* I was unrotting it :) Once I see that I didn't break it, I'll probably throw in .push()ing the two tests that subsequent updates have added broken on 10.7 and push both.
Assignee | ||
Comment 17•12 years ago
|
||
Sorry for forgetting to land this, and thanks Phil for the work. So my understand is that you're handling this now.
Assignee: bjacob → philringnalda
Reporter | ||
Comment 18•12 years ago
|
||
Yup, though in your name ;) https://hg.mozilla.org/integration/mozilla-inbound/rev/e271afc540a9
Assignee: philringnalda → bjacob
status-firefox11:
--- → affected
status-firefox12:
--- → affected
Target Milestone: --- → mozilla13
Reporter | ||
Comment 19•12 years ago
|
||
[Approval Request Comment] User impact if declined: Another 11 weeks of totally wasted resources running mochitest-1 on 10.7 hidden, plus 11 more weeks of risking 10.7-only regressions in things tested in mochitest-1 because nobody looks at hidden tests. Risk to taking this patch (and alternatives if risky): zero, test only, either the tests remain 'failing' (in this case, they are passing but marked failing) and hidden, or, as will happen (with one more patch), become green and visible. String changes made by this patch: none.
Attachment #572827 -
Attachment is obsolete: true
Attachment #594617 -
Flags: approval-mozilla-aurora?
Reporter | ||
Comment 20•12 years ago
|
||
The only difference is context, but whenever I decide I don't need to attach a patch, I wind up losing it. [Approval Request Comment] User impact if declined: Another 5 weeks of totally wasted resources running mochitest-1 on 10.7 hidden, plus 5 more weeks of risking 10.7-only regressions in things tested in mochitest-1 because nobody looks at hidden tests. Risk to taking this patch (and alternatives if risky): none, test only String changes made by this patch: none
Attachment #594631 -
Flags: approval-mozilla-beta?
Comment 21•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/e271afc540a9
Status: REOPENED → RESOLVED
Closed: 13 years ago → 12 years ago
Resolution: --- → FIXED
Comment 22•12 years ago
|
||
Comment on attachment 594617 [details] [diff] [review] as pushed [Triage Comment] Test fix - approved for Aurora.
Attachment #594617 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Updated•12 years ago
|
Attachment #594631 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Reporter | ||
Comment 23•12 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/2ab48f3e27d3 https://hg.mozilla.org/releases/mozilla-beta/rev/eba29d1fba29
You need to log in
before you can comment on or make changes to this bug.
Description
•