Closed Bug 1782556 Opened 2 years ago Closed 2 years ago

Intermittent browser/components/firefoxview/tests/browser/browser_notification_dot.js | single tracking bug

Categories

(Firefox :: Firefox View, defect, P5)

defect
Points:
1

Tracking

()

RESOLVED FIXED
111 Branch
Tracking Status
firefox-esr91 --- unaffected
firefox-esr102 --- unaffected
firefox103 --- unaffected
firefox104 --- unaffected
firefox105 --- wontfix
firefox109 --- wontfix
firefox110 --- wontfix
firefox111 --- fixed

People

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

References

(Blocks 1 open bug, Regression)

Details

(Keywords: intermittent-failure, intermittent-testcase, regression, Whiteboard: [fidefe-firefox-view])

Attachments

(1 file)

Filed by: imoraru [at] mozilla.com
Parsed log: https://treeherder.mozilla.org/logviewer?job_id=386018933&repo=autoland
Full log: https://firefox-ci-tc.services.mozilla.com/api/queue/v1/task/DO8Kw96xTzixqSxBV9J4SA/runs/0/artifacts/public/logs/live_backing.log
Reftest URL: https://hg.mozilla.org/mozilla-central/raw-file/tip/layout/tools/reftest/reftest-analyzer.xhtml#logurl=https://firefox-ci-tc.services.mozilla.com/api/queue/v1/task/DO8Kw96xTzixqSxBV9J4SA/runs/0/artifacts/public/logs/live_backing.log&only_show_unexpected=1


[task 2022-08-01T16:06:57.972Z] 16:06:57     INFO - TEST-START | browser/components/firefoxview/tests/browser/browser_notification_dot.js
[task 2022-08-01T16:07:42.975Z] 16:07:42     INFO - TEST-INFO | started process screentopng
[task 2022-08-01T16:07:43.236Z] 16:07:43     INFO - TEST-INFO | screentopng: exit 0
[task 2022-08-01T16:07:43.236Z] 16:07:43     INFO - Buffered messages logged at 16:06:57
[task 2022-08-01T16:07:43.237Z] 16:07:43     INFO - Entering setup bound 
[task 2022-08-01T16:07:43.240Z] 16:07:43     INFO - true
[task 2022-08-01T16:07:43.241Z] 16:07:43     INFO - Leaving setup bound 
[task 2022-08-01T16:07:43.241Z] 16:07:43     INFO - Entering test bound testNotificationDot
[task 2022-08-01T16:07:43.241Z] 16:07:43     INFO - updating lastFetch:1659370018
[task 2022-08-01T16:07:43.242Z] 16:07:43     INFO - Buffered messages logged at 16:06:58
[task 2022-08-01T16:07:43.242Z] 16:07:43     INFO - Console message: 1659370017997	FirefoxView.TabsSetup	DEBUG	maybeUpdateUI, _uiUpdateNeeded:: true
[task 2022-08-01T16:07:43.244Z] 16:07:43     INFO - TEST-PASS | browser/components/firefoxview/tests/browser/browser_notification_dot.js | Got the Firefox View button - 
[task 2022-08-01T16:07:43.244Z] 16:07:43     INFO - TEST-PASS | browser/components/firefoxview/tests/browser/browser_notification_dot.js | The Firefox View button is showing - 
[task 2022-08-01T16:07:43.246Z] 16:07:43     INFO - Buffered messages finished
[task 2022-08-01T16:07:43.246Z] 16:07:43     INFO - TEST-UNEXPECTED-FAIL | browser/components/firefoxview/tests/browser/browser_notification_dot.js | Test timed out - 
[task 2022-08-01T16:07:43.246Z] 16:07:43     INFO - Not taking screenshot here: see the one that was previously logged
[task 2022-08-01T16:07:43.251Z] 16:07:43     INFO - TEST-UNEXPECTED-FAIL | browser/components/firefoxview/tests/browser/browser_notification_dot.js | Cleanup function threw an exception - at resource://testing-common/BrowserTestUtils.jsm:1797 - TypeError: can't access property "ownerGlobal", tab is null
[task 2022-08-01T16:07:43.252Z] 16:07:43     INFO - GECKO(10218) | MEMORY STAT | vsize 3321MB | residentFast 473MB | heapAllocated 218MB
[task 2022-08-01T16:07:43.252Z] 16:07:43     INFO - TEST-OK | browser/components/firefoxview/tests/browser/browser_notification_dot.js | took 45012ms
[task 2022-08-01T16:07:43.253Z] 16:07:43     INFO - checking window state
[task 2022-08-01T16:07:43.253Z] 16:07:43     INFO - GECKO(10218) | Completed ShutdownLeaks collections in process 10218
[task 2022-08-01T16:07:43.253Z] 16:07:43     INFO - TEST-START | Shutdown
[task 2022-08-01T16:07:43.253Z] 16:07:43     INFO - Browser Chrome Test Summary
[task 2022-08-01T16:07:43.254Z] 16:07:43     INFO - Passed:  25
[task 2022-08-01T16:07:43.254Z] 16:07:43     INFO - Failed:  2
[task 2022-08-01T16:07:43.254Z] 16:07:43     INFO - Todo:    0
[task 2022-08-01T16:07:43.255Z] 16:07:43     INFO - Mode:    e10s
[task 2022-08-01T16:07:43.255Z] 16:07:43     INFO - *** End BrowserChrome Test Results ***
[task 2022-08-01T16:07:43.257Z] 16:07:43     INFO - GECKO(10218) | Exiting due to channel error.
[task 2022-08-01T16:07:43.257Z] 16:07:43     INFO - GECKO(10218) | Exiting due to channel error.
[task 2022-08-01T16:07:43.258Z] 16:07:43     INFO - GECKO(10218) | Exiting due to channel error.
[task 2022-08-01T16:07:43.259Z] 16:07:43     INFO - GECKO(10218) | Exiting due to channel error.
[task 2022-08-01T16:07:43.260Z] 16:07:43     INFO - GECKO(10218) | Exiting due to channel error.
[task 2022-08-01T16:07:43.260Z] 16:07:43     INFO - GECKO(10218) | Exiting due to channel error.
[task 2022-08-01T16:07:43.261Z] 16:07:43     INFO - GECKO(10218) | Exiting due to channel error.
[task 2022-08-01T16:07:43.262Z] 16:07:43     INFO - TEST-INFO | Main app process: exit 0
[task 2022-08-01T16:07:43.263Z] 16:07:43     INFO - runtests.py | Application ran for: 0:00:59.055503

:niklas, since you are the author of the regressor, bug 1774397, could you take a look?
For more information, please visit auto_nag documentation.

Flags: needinfo?(nbaumgardner)

Set release status flags based on info from the regressing bug 1774397

Whiteboard: [fidefe-firefox-view]
Blocks: firefox-view
Summary: Intermittent TV browser/components/firefoxview/tests/browser/browser_notification_dot.js | single tracking bug → Intermittent browser/components/firefoxview/tests/browser/browser_notification_dot.js | single tracking bug
Blocks: 1797520
No longer blocks: firefox-view
Flags: needinfo?(nbaumgardner)

This feature was disabled so we should either remove or skip the tests.

Points: --- → 1
Assignee: nobody → sfoster

(In reply to Sarah Clements [:sclements] from comment #13)

This feature was disabled so we should either remove or skip the tests.

Unless we want to remove the feature entirely, I think we should keep the tests passing - otherwise we're implicitly supporting a pref which we're not actually sure is working from one release to another.

I've taken a look at this test, and at least some of the intermittency comes from the expectation of a pref-value-update from services.sync.lastTabFetch. We record the lastTabFetch in seconds - making it likely that from one test to another we'll update the pref to the same int value as it already had, and none of the cascading checks and updates that would result from a change happen - as there was no change.

I'll submit a patch to address that and we can see if that drops the intermittent frequency.

Status: NEW → ASSIGNED
Pushed by sfoster@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/3af6a65d70a4 Make sure lastTabFetch is actually changing when we update it in the notification dot tests. r=niklas
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 111 Branch
Regressions: 1813139
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: