Closed Bug 1522621 Opened 5 years ago Closed 4 years ago

Intermittent /resource-timing/buffer-full-inspect-buffer-during-callback.html | Test that entries in the secondary buffer are not exposed during the callback and before they are copied to the primary buffer - assert_equals: All 4 entries should be stored

Categories

(Core :: Performance, defect, P1)

defect

Tracking

()

RESOLVED FIXED
81 Branch
Tracking Status
firefox81 --- fixed

People

(Reporter: intermittent-bug-filer, Assigned: sefeng)

References

Details

(Keywords: intermittent-failure, Whiteboard: [stockwell fixed:backout]{comment 7])

Attachments

(1 file)

Filed by: nbeleuzu [at] mozilla.com

https://treeherder.mozilla.org/logviewer.html#?job_id=223738094&repo=mozilla-central

https://queue.taskcluster.net/v1/task/fbSHAtzvSjGDWiFzi5NwGw/runs/0/artifacts/public/logs/live_backing.log

[task 2019-01-24T13:07:27.302Z] 13:07:27 INFO - PID 16658 | JavaScript error: http://web-platform.test:8000/resource-timing/resources/buffer-full-utilities.js, line 12: uncaught exception: Error: assert_equals: A sync request was not added to the primary buffer just yet, because it is full expected 1297 but got 1296
[task 2019-01-24T13:07:27.302Z] 13:07:27 INFO - PID 16658 | JavaScript error: http://web-platform.test:8000/resource-timing/resources/buffer-full-utilities.js, line 12: uncaught exception: Error: assert_equals: A sync request was not added to the primary buffer just yet, because it is full expected 1297 but got 1296
[task 2019-01-24T13:07:27.303Z] 13:07:27 INFO - PID 16658 | JavaScript error: http://web-platform.test:8000/resource-timing/resources/buffer-full-utilities.js, line 12: uncaught exception: Error: assert_equals: A sync request was not added to the primary buffer just yet, because it is full expected 1297 but got 1296
[task 2019-01-24T13:07:27.303Z] 13:07:27 INFO - PID 16658 | JavaScript error: , line 0: uncaught exception: Error: assert_equals: A sync request was not added to the primary buffer just yet, because it is full expected 1297 but got 1296
[task 2019-01-24T13:07:27.453Z] 13:07:27 INFO - Closing window 21474836481
[task 2019-01-24T13:07:27.503Z] 13:07:27 INFO -
[task 2019-01-24T13:07:27.504Z] 13:07:27 INFO - TEST-UNEXPECTED-FAIL | /resource-timing/buffer-full-inspect-buffer-during-callback.html | Test that entries in the secondary buffer are not exposed during the callback and before they are copied to the primary buffer - assert_equals: All 4 entries should be stored in resource timing buffer. expected 1298 but got 1297
[task 2019-01-24T13:07:27.504Z] 13:07:27 INFO - testThatBufferContainsTheRightResources@http://web-platform.test:8000/resource-timing/buffer-full-inspect-buffer-during-callback.html:40:5
[task 2019-01-24T13:07:27.504Z] 13:07:27 INFO - @http://web-platform.test:8000/resource-timing/buffer-full-inspect-buffer-during-callback.html:52:5
[task 2019-01-24T13:07:27.504Z] 13:07:27 INFO - asyncTest.prototype.step@http://web-platform.test:8000/resources/testharness.js:1566:20
[task 2019-01-24T13:07:27.505Z] 13:07:27 INFO - promise_test/tests.promise_tests</<@http://web-platform.test:8000/resources/testharness.js:591:31
[task 2019-01-24T13:07:27.505Z] 13:07:27 INFO - promise_test/tests.promise_tests<@http://web-platform.test:8000/resources/testharness.js:590:20
[task 2019-01-24T13:07:27.505Z] 13:07:27 INFO - promise callback
promise_test@http://web-platform.test:8000/resources/testharness.js:589:31
[task 2019-01-24T13:07:27.505Z] 13:07:27 INFO - @http://web-platform.test:8000/resource-timing/buffer-full-inspect-buffer-during-callback.html:48:1
[task 2019-01-24T13:07:27.506Z] 13:07:27 INFO - TEST-INFO | expected TIMEOUT
[task 2019-01-24T13:07:27.507Z] 13:07:27 INFO - TEST-ERROR | /resource-timing/buffer-full-inspect-buffer-during-callback.html | took 7843ms
[task 2019-01-24T13:07:27.528Z] 13:07:27 INFO - PID 16658 | 1548335247522 Marionette INFO Stopped listening on port 2828
[task 2019-01-24T13:07:27.652Z] 13:07:27 INFO - PID 16658 | [Parent 16658, Gecko_IOThread] WARNING: pipe error (86): Connection reset by peer: file /builds/worker/workspace/build/src/ipc/chromium/src/chrome/common/ipc_channel_posix.cc, line 357
[task 2019-01-24T13:07:27.652Z] 13:07:27 INFO - PID 16658 | [Parent 16658, Gecko_IOThread] WARNING: pipe error (87): Connection reset by peer: file /builds/worker/workspace/build/src/ipc/chromium/src/chrome/common/ipc_channel_posix.cc, line 357
[task 2019-01-24T13:07:28.121Z] 13:07:28 INFO - Browser exited with return code 0
[task 2019-01-24T13:07:28.122Z] 13:07:28 INFO - PROCESS LEAKS None
[task 2019-01-24T13:07:28.124Z] 13:07:28 INFO - Closing logging queue
[task 2019-01-24T13:07:28.124Z] 13:07:28 INFO - queue closed
[task 2019-01-24T13:07:28.145Z] 13:07:28 INFO - Setting up ssl
[task 2019-01-24T13:07:28.182Z] 13:07:28 INFO - certutil |
[task 2019-01-24T13:07:28.210Z] 13:07:28 INFO - certutil |
[task 2019-01-24T13:07:28.234Z] 13:07:28 INFO - certutil |
[task 2019-01-24T13:07:28.234Z] 13:07:28 INFO - Certificate Nickname Trust Attributes
[task 2019-01-24T13:07:28.234Z] 13:07:28 INFO - SSL,S/MIME,JAR/XPI
[task 2019-01-24T13:07:28.234Z] 13:07:28 INFO -
[task 2019-01-24T13:07:28.235Z] 13:07:28 INFO - web-platform-tests CT,,
[task 2019-01-24T13:07:28.235Z] 13:07:28 INFO -
[task 2019-01-24T13:07:28.271Z] 13:07:28 INFO - Application command: /builds/worker/workspace/build/application/firefox/firefox --marionette about:blank -profile /tmp/tmpV7k6yn.mozrunner
[task 2019-01-24T13:07:28.292Z] 13:07:28 INFO - Starting runner
[task 2019-01-24T13:07:28.810Z] 13:07:28 INFO - PID 18444 | 1548335248800 addons.webextension.screenshots@mozilla.org WARN Loading extension 'screenshots@mozilla.org': Reading manifest: Invalid extension permission: mozillaAddons
[task 2019-01-24T13:07:28.811Z] 13:07:28 INFO - PID 18444 | 1548335248800 addons.webextension.screenshots@mozilla.org WARN Loading extension 'screenshots@mozilla.org': Reading manifest: Invalid extension permission: resource://pdf.js/
[task 2019-01-24T13:07:28.811Z] 13:07:28 INFO - PID 18444 | 1548335248800 addons.webextension.screenshots@mozilla.org WARN Loading extension 'screenshots@mozilla.org': Reading manifest: Invalid extension permission: about:reader*
[task 2019-01-24T13:07:31.668Z] 13:07:31 INFO - PID 18444 | 1548335251663 Marionette INFO Listening on port 2828

Hi James, I wonder if there are changes to WPT harness recently. There are several wpt intermittent failures whose failure rate increased these days.

Flags: needinfo?(james)

Yeah, we changed from running the driver script (injected via marionette) in the test window instead of in the opener window. That's important for some kinds of tests since it allows the test window to be a true top level window with null opener, but does mean that we are getting different behaviour in edge conditions. I don't know if/how marionette-injected script will interfere with resource timing (it's basically running in a frame-script).

Flags: needinfo?(james)

(In reply to James Graham [:jgraham] from comment #4)

Yeah, we changed from running the driver script (injected via marionette) in the test window instead of in the opener window. That's important for some kinds of tests since it allows the test window to be a true top level window with null opener, but does mean that we are getting different behaviour in edge conditions. I don't know if/how marionette-injected script will interfere with resource timing (it's basically running in a frame-script).

Thanks James!

Thanks for the comment 7. My understanding is that this falls at Performance team's wheelhouse (as this is the test of performance API) so moving there to get proper attention. Feel free to reset if I got something wrong.

Component: DOM → Performance
Whiteboard: [stockwell disable-recommended]
Whiteboard: [stockwell disable-recommended]
Whiteboard: [stockwell disable-recommended] → [stockwell fixed:backout]{comment 7]
Depends on: 1525051
Assignee: nobody → whawkins
Priority: -- → P1

This should be solved thanks to the solution for 1539006 and 1525051.

If you can confirm, Mr. Graham, we can close this one!

Thanks,
Will

Depends on: 1539006
Flags: needinfo?(james)

OK, so I reenabled some tests and scheduled a try push: https://treeherder.mozilla.org/#/jobs?repo=try&revision=006f408b7ba5d3a31472865fbabaa22bbbfece6b

Hopefully that shows stability. If it does you should be able to land the patch. Note that in general I can't/shouldn't be on the critical path here; I simply don't have the bandwidth to do the test work every time something is fixed. Generally it's the reposibility of the engineers working on the feature to ensure that we have the appropriate test coverage and wpt tests enabled. Obviously I'm very happy to help with that process in any way I can, though.

Flags: needinfo?(james)
Assignee: whawkins → nobody

:jgraham Did that commit stick? I just checked the try push but the push has expired in th and I can't see the results. I'm just following up because I want to close this performance bug if it's not relevant anymore.

Flags: needinfo?(james)

I also have no memory of what happened in that try push, but the test is still disabled: https://searchfox.org/mozilla-central/source/testing/web-platform/meta/resource-timing/buffer-full-inspect-buffer-during-callback.html.ini I'd say we need to repeat the experiment of re-enabling it and see what happens.

Flags: needinfo?(james)

I've fixed some similar bugs for performance apis, and I'll take a look.

Flags: needinfo?(sefeng)

Thanks Sean I really appreciate it!

This test has been fixed in bug 1525051. We should enable it.

Assignee: nobody → sefeng
Status: NEW → ASSIGNED
Pushed by rmaries@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/d24060ec6ddf
Enable buffer-full-inspect-buffer-during-callback.html r=smaug
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 81 Branch

It's fixed, removing my NI.

Flags: needinfo?(sefeng)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: