Permaorange in test_webgl_debug_renderer_info.html when Gecko 53 merges to beta on 2017-03-06

RESOLVED FIXED in Firefox 53

Status

()

defect
P1
normal
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: philor, Assigned: jgilbert)

Tracking

53 Branch
mozilla54
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox-esr45 unaffected, firefox51 unaffected, firefox52 unaffected, firefox-esr52 unaffected, firefox53? fixed, firefox54 fixed)

Details

(Whiteboard: gfx-noted)

Attachments

(1 attachment)

Bug 1336645 removed preffing off webgl.enable-debug-renderer-info for RELEASE_OR_BETA, but https://hg.mozilla.org/mozilla-central/annotate/0eef1d5a39366059677c6d7944cfe8a97265a011/dom/canvas/test/chrome/test_webgl_debug_renderer_info.html#l27 hardcodes RELEASE_OR_BETA instead of either reading the pref, or just being removed now that the pref is on by default, so when aurora merges to beta in three weeks, it'll be permaorange.
Flags: needinfo?(jgilbert)
[Tracking Requested - why for this release]: permaorange on the merge to beta, closed trees, wailing and tearing of hair.
Forgot the link to a log, https://treeherder.mozilla.org/logviewer.html#?job_id=76638515&repo=try, from the simulation push, https://treeherder.mozilla.org/#/jobs?repo=try&revision=46c3eb85b4687c968b6b89873d5f5a2f9fc89435, though I like the way skipping it makes it look like I just figured it out from reading the patch.
Assignee: nobody → jgilbert
Flags: needinfo?(jgilbert)
Priority: -- → P1
Whiteboard: gfx-noted
Comment on attachment 8836926 [details]
Bug 1338862 - Update tests for WEBGL_debug_renderer_info. -

https://reviewboard.mozilla.org/r/112238/#review113714

::: dom/canvas/test/chrome/test_webgl_debug_renderer_info.html
(Diff revision 1)
> -  iframe.onload = function () {
> -
> -    // test that chrome can get WEBGL_debug_renderer_info on a canvas on the iframe...
> -    // this is useful to check in itself, and is also useful so the subsequent non-chrome test
> -    // will also test that doing so doesn't confuse our chrome-only check.
> -    checkChromeCase(iframe.contentDocument.createElement("canvas"));

I don't understand why you remove the test for iframe. Can you explain it?
Comment on attachment 8836926 [details]
Bug 1338862 - Update tests for WEBGL_debug_renderer_info. -

https://reviewboard.mozilla.org/r/112238/#review113714

> I don't understand why you remove the test for iframe. Can you explain it?

I removed the non-chrome test, since it's not a privileged extension anymore.
I think we should keep the chrome test to ensure that it's always available to chrome code though.
Comment on attachment 8836926 [details]
Bug 1338862 - Update tests for WEBGL_debug_renderer_info. -

https://reviewboard.mozilla.org/r/112238/#review114038

LGTM
Attachment #8836926 - Flags: review?(dmu) → review+
(In reply to Phil Ringnalda (:philor) from comment #4)
> https://treeherder.mozilla.org/#/
> jobs?repo=try&revision=9d297e2e0c77e071ad30eef7a66cface6f8d5456&filter-
> searchStr=gpu - doesn't fail, but it does consistently time out with that
> patch.

Updated again. Can you retest?
Flags: needinfo?(philringnalda)
Look at all them green gpus in https://treeherder.mozilla.org/#/jobs?repo=try&revision=d139e05854056ed9177d1b91995cc03b4f07120b&group_state=expanded, just look at them!
Flags: needinfo?(philringnalda)
Pushed by jgilbert@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/5387e70bd9c5
Update tests for WEBGL_debug_renderer_info. - r=daoshengmu
https://hg.mozilla.org/mozilla-central/rev/5387e70bd9c5
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla54
You need to log in before you can comment on or make changes to this bug.