Closed
Bug 1303793
Opened 8 years ago
Closed 8 years ago
Beta mochitest(1) failing with dom/animation/test/mozilla/test_set-easing.html | Setting easing should not throw in sandbox
Categories
(Core :: DOM: Animation, defect)
Core
DOM: Animation
Tracking
()
RESOLVED
FIXED
mozilla52
People
(Reporter: intermittent-bug-filer, Assigned: hiro)
Details
(Keywords: intermittent-failure)
Attachments
(3 files)
Filed by: wkocher [at] mozilla.com https://treeherder.mozilla.org/logviewer.html#?job_id=1608017&repo=mozilla-beta https://archive.mozilla.org/pub/firefox/tinderbox-builds/mozilla-beta-linux-pgo/1474292431/mozilla-beta_ubuntu32_vm_test_pgo-mochitest-e10s-1-bm05-tests1-linux32-build1.txt.gz
status-firefox50:
--- → affected
status-firefox51:
--- → unaffected
Comment 1•8 years ago
|
||
Hello Hiro, this test got moved in https://hg.mozilla.org/mozilla-central/rev/e85703fad4a6 as part of bug 1277456. Can you take a look what's the issue, please? bbirtles is busy.
Flags: needinfo?(hiikezoe)
Assignee | ||
Comment 2•8 years ago
|
||
Sure! taking.
Assignee: nobody → hiikezoe
Status: NEW → ASSIGNED
Flags: needinfo?(hiikezoe)
Assignee | ||
Comment 3•8 years ago
|
||
It seems that document.getAnimations can not be seen in the test. Neither sandbox nor in the original document. I am not sure what the difference between nightly and beta, but we should be open an iframe window after setting dom.animations-api.core.enabled just like other tests do, and run the test in the iframe window.
Comment 4•8 years ago
|
||
dom.animations-api.core.enabled is set to false on beta and release, see http://hg.mozilla.org/releases/mozilla-beta/file/tip/modules/libpref/init/all.js#l2659 Does pushing the preference with the value |true| in the test like in https://dxr.mozilla.org/mozilla-aurora/source/dom/animation/test/mozilla/test_hide_and_show.html#10 fix it?
Assignee | ||
Comment 5•8 years ago
|
||
(In reply to Sebastian Hengst [:aryx][:archaeopteryx] from comment #4) > dom.animations-api.core.enabled is set to false on beta and release, see > http://hg.mozilla.org/releases/mozilla-beta/file/tip/modules/libpref/init/ > all.js#l2659 Thanks! I did not know that beta is 'RELEASE_BUILD'. > Does pushing the preference with the value |true| in the test like in > https://dxr.mozilla.org/mozilla-aurora/source/dom/animation/test/mozilla/ > test_hide_and_show.html#10 fix it? Yes, it is. I am just writing it.
Assignee | ||
Comment 6•8 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=d866ea8a27cf
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Assignee | ||
Comment 9•8 years ago
|
||
mozreview-review |
Comment on attachment 8792718 [details] Bug 1303793 - Part 1: Use testharness.js instead of SimpleTest.js to run the test in an iframe easily. https://reviewboard.mozilla.org/r/79628/#review78270 ::: dom/animation/test/mozilla/test_set-easing.html:21 (Diff revision 1) > div.animate({ opacity: [0, 1] }, 100000 ); > > const contentScript = function() { > try { > document.getAnimations()[0].effect.timing.easing = "linear"; > - ok(true, 'Setting easing should not throw in sandbox'); > + assert_true(true, 'Setting easing should not throw in sandbox'); I did drop the first argument 'true' locally.
Assignee | ||
Comment 10•8 years ago
|
||
Brian, can you please take time to review both of patches?
Flags: needinfo?(bbirtles)
Comment hidden (Intermittent Failures Robot) |
Updated•8 years ago
|
Severity: normal → blocker
Comment 12•8 years ago
|
||
mozreview-review |
Comment on attachment 8792718 [details] Bug 1303793 - Part 1: Use testharness.js instead of SimpleTest.js to run the test in an iframe easily. https://reviewboard.mozilla.org/r/79628/#review78382 Thanks for taking care of this.
Attachment #8792718 -
Flags: review+
Comment 13•8 years ago
|
||
Oh, (In reply to Brian Birtles (:birtles, busy 19-23 Sep) from comment #12) > Comment on attachment 8792718 [details] > Bug 1303793 - Part 1: Use testharnss.js instead of SimpleTest.js to run the > test in an iframe easily. r? Nit: s/testharnss.js/testharness.js/ in the commit message.
Flags: needinfo?(bbirtles)
Comment 14•8 years ago
|
||
mozreview-review |
Comment on attachment 8792719 [details] Bug 1303793 - Part 2: Run test which uses the Web Animations API in an iframe with the Web Animations API preference. https://reviewboard.mozilla.org/r/79630/#review78384
Attachment #8792719 -
Flags: review+
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Assignee | ||
Comment 17•8 years ago
|
||
(In reply to Hiroyuki Ikezoe (:hiro) from comment #9) > Comment on attachment 8792718 [details] > Bug 1303793 - Part 1: Use testharness.js instead of SimpleTest.js to run the > test in an iframe easily. > > https://reviewboard.mozilla.org/r/79628/#review78270 > > ::: dom/animation/test/mozilla/test_set-easing.html:21 > (Diff revision 1) > > div.animate({ opacity: [0, 1] }, 100000 ); > > > > const contentScript = function() { > > try { > > document.getAnimations()[0].effect.timing.easing = "linear"; > > - ok(true, 'Setting easing should not throw in sandbox'); > > + assert_true(true, 'Setting easing should not throw in sandbox'); > > I did drop the first argument 'true' locally. I was somewhat confused about ok().
Comment 18•8 years ago
|
||
Pushed by hiikezoe@mozilla-japan.org: https://hg.mozilla.org/integration/autoland/rev/088fee62d15e Part 1: Use testharness.js instead of SimpleTest.js to run the test in an iframe easily. r=birtles https://hg.mozilla.org/integration/autoland/rev/85624e2d619f Part 2: Run test which uses the Web Animations API in an iframe with the Web Animations API preference. r=birtles
Assignee | ||
Comment 19•8 years ago
|
||
Part 2 (Attachment 8792719 [details]) can not be applied to beta cleanly because of conflict of mochitest.ini. This is the modified of part 2 for beta branch.
Attachment #8792832 -
Flags: review+
Comment 20•8 years ago
|
||
(In reply to Hiroyuki Ikezoe (:hiro) from comment #19) > Created attachment 8792832 [details] [diff] [review] > [For beta] Part 2: Run test which uses the Web Animations API in an iframe > with the Web Animations API preference > > Part 2 (Attachment 8792719 [details]) can not be applied to beta cleanly > because of conflict of mochitest.ini. This is the modified of part 2 for > beta branch. thanks! landed both on beta remote: https://hg.mozilla.org/releases/mozilla-beta/rev/763ecbe1877e remote: https://hg.mozilla.org/releases/mozilla-beta/rev/50a24c86a83a
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla50
Comment 21•8 years ago
|
||
Unless I'm mistaken, this hasn't been merged to m-c yet. Also, it presumably needs to land on Aurora still as well.
Status: RESOLVED → REOPENED
status-firefox52:
--- → affected
Resolution: FIXED → ---
Target Milestone: mozilla50 → ---
Comment 22•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/088fee62d15e https://hg.mozilla.org/mozilla-central/rev/85624e2d619f
Status: REOPENED → RESOLVED
Closed: 8 years ago → 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla52
Comment 23•8 years ago
|
||
bugherder uplift |
Hiro landed this on Aurora. https://hg.mozilla.org/releases/mozilla-aurora/rev/271c91111de4 https://hg.mozilla.org/releases/mozilla-aurora/rev/f93ac29ad106
Comment hidden (Intermittent Failures Robot) |
You need to log in
before you can comment on or make changes to this bug.
Description
•