Closed Bug 1621655 Opened 4 years ago Closed 4 years ago

Intermittent devtools/client/styleeditor/test/browser_styleeditor_media_sidebar_links.js | media rule should now be matched after responsive mode is active -

Categories

(DevTools :: Style Editor, defect, P5)

defect

Tracking

(firefox81 fixed)

RESOLVED FIXED
81 Branch
Tracking Status
firefox81 --- fixed

People

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

References

(Regression)

Details

(Keywords: intermittent-failure, regression, Whiteboard: [retriggered][stockwell fixed:patch])

Attachments

(3 files)

Filed by: apavel [at] mozilla.com
Parsed log: https://treeherder.mozilla.org/logviewer.html#?job_id=292677034&repo=autoland
Full log: https://firefox-ci-tc.services.mozilla.com/api/queue/v1/task/W-WoSr_QRc6Nx8sWcPHdHw/runs/0/artifacts/public/logs/live_backing.log


[task 2020-03-11T13:53:51.063Z] 13:53:51 INFO - TEST-START | devtools/client/styleeditor/test/browser_styleeditor_media_sidebar_links.js
[task 2020-03-11T13:53:56.006Z] 13:53:56 INFO - GECKO(4525) | console.log: "[DISPATCH] action type:" "CHANGE_DISPLAY_PIXEL_RATIO"
[task 2020-03-11T13:53:56.007Z] 13:53:56 INFO - GECKO(4525) | console.log: "[DISPATCH] action type:" "ADD_VIEWPORT"
[task 2020-03-11T13:53:56.740Z] 13:53:56 INFO - GECKO(4525) | console.log: "[DISPATCH] action type:" "RESIZE_VIEWPORT"
[task 2020-03-11T13:53:56.837Z] 13:53:56 INFO - GECKO(4525) | console.log: "[DISPATCH] action type:" "LOAD_DEVICE_LIST_START"
[task 2020-03-11T13:53:56.954Z] 13:53:56 INFO - GECKO(4525) | console.log: "[DISPATCH] action type:" "RESIZE_VIEWPORT"
[task 2020-03-11T13:53:57.056Z] 13:53:57 INFO - GECKO(4525) | console.log: "[DISPATCH] action type:" "ADD_DEVICE_TYPE"
[task 2020-03-11T13:53:57.071Z] 13:53:57 INFO - GECKO(4525) | console.log: "[DISPATCH] action type:" "ADD_DEVICE"

[task 2020-03-11T13:53:58.114Z] 13:53:58 INFO - TEST-PASS | devtools/client/styleeditor/test/browser_styleeditor_media_sidebar_links.js | Responsive mode should be active. -
[task 2020-03-11T13:53:58.114Z] 13:53:58 INFO - Buffered messages finished
[task 2020-03-11T13:53:58.115Z] 13:53:58 INFO - TEST-UNEXPECTED-FAIL | devtools/client/styleeditor/test/browser_styleeditor_media_sidebar_links.js | media rule should now be matched after responsive mode is active -
[task 2020-03-11T13:53:58.116Z] 13:53:58 INFO - Stack trace:
[task 2020-03-11T13:53:58.117Z] 13:53:58 INFO - chrome://mochikit/content/browser-test.js:test_ok:1292
[task 2020-03-11T13:53:58.118Z] 13:53:58 INFO - chrome://mochitests/content/browser/devtools/client/styleeditor/test/browser_styleeditor_media_sidebar_links.js:testMediaLink:96
[task 2020-03-11T13:53:58.118Z] 13:53:58 INFO - chrome://mochitests/content/browser/devtools/client/styleeditor/test/browser_styleeditor_media_sidebar_links.js:null:38
[task 2020-03-11T13:53:58.118Z] 13:53:58 INFO - chrome://mochikit/content/browser-test.js:Tester_execTest/<:1062
[task 2020-03-11T13:53:58.118Z] 13:53:58 INFO - chrome://mochikit/content/browser-test.js:Tester_execTest:1097
[task 2020-03-11T13:53:58.118Z] 13:53:58 INFO - chrome://mochikit/content/browser-test.js:nextTest/<:925
[task 2020-03-11T13:53:58.118Z] 13:53:58 INFO - chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:SimpleTest.waitForFocus/waitForFocusInner/focusedOrLoaded/<:918
[task 2020-03-11T13:53:58.118Z] 13:53:58 INFO - TEST-PASS | devtools/client/styleeditor/test/browser_styleeditor_media_sidebar_links.js | height should be properly set. -
[task 2020-03-11T13:53:58.120Z] 13:53:58 INFO - Closing responsive mode
[task 2020-03-11T13:53:58.121Z] 13:53:58 INFO - Waiting for event: 'off' on [object Object].
[task 2020-03-11T13:53:58.162Z] 13:53:58 INFO - Got event: 'off' on [object Object].
[task 2020-03-11T13:53:58.163Z] 13:53:58 INFO - Wait for media-list-changed events to settle on StyleEditorUI
[task 2020-03-11T13:53:59.328Z] 13:53:59 INFO - TEST-PASS | devtools/client/styleeditor/test/browser_styleeditor_media_sidebar_links.js | Responsive mode should no longer be active. -
[task 2020-03-11T13:53:59.328Z] 13:53:59 INFO - TEST-PASS | devtools/client/styleeditor/test/browser_styleeditor_media_sidebar_links.js | The width condition should now be unmatched -
[task 2020-03-11T13:53:59.328Z] 13:53:59 INFO - TEST-PASS | devtools/client/styleeditor/test/browser_styleeditor_media_sidebar_links.js | The height condition should now be unmatched -
[task 2020-03-11T13:53:59.329Z] 13:53:59 INFO - Leaving test bound
[task 2020-03-11T13:54:00.115Z] 13:54:00 INFO - Removing tab.
[task 2020-03-11T13:54:00.115Z] 13:54:00 INFO - Waiting for event: 'TabClose' on [object XULElement].
[task 2020-03-11T13:54:00.171Z] 13:54:00 INFO - Got event: 'TabClose' on [object XULElement].
[task 2020-03-11T13:54:00.199Z] 13:54:00 INFO - Tab removed and finished closing
[task 2020-03-11T13:54:00.276Z] 13:54:00 INFO - TEST-PASS | devtools/client/styleeditor/test/browser_styleeditor_media_sidebar_links.js | The main process DevToolsServer has no pending connection when the test ends -
[task 2020-03-11T13:54:00.296Z] 13:54:00 INFO - GECKO(4525) | MEMORY STAT | vsize 20975633MB | residentFast 1449MB
[task 2020-03-11T13:54:00.297Z] 13:54:00 INFO - TEST-OK | devtools/client/styleeditor/test/browser_styleeditor_media_sidebar_links.js | took 9230ms

Regressed by: 1621306
Whiteboard: [retriggered]
Has Regression Range: --- → yes
Keywords: regression

Sure, will take a look. Thanks!

Assignee: nobody → mtigley
Status: NEW → ASSIGNED
Flags: needinfo?(mtigley)

Micah, any updates on this? It has reached the disable recommend bugs list with 193 total failures since it was filed on 11th of march: https://treeherder.mozilla.org/intermittent-failures.html#/bugdetails?startday=2020-02-19&endday=2020-03-20&tree=trunk&bug=1621655
The test will be disabled in the next couple of days if there is no patch for it. Thank you.

Flags: needinfo?(mtigley)
Whiteboard: [retriggered][stockwell disable-recommended] → [retriggered][stockwell needswork:owner]
Attachment #9134869 - Attachment description: Bug 1621655 - Disabled browser_styleeditor_media_sidebar_links.js on Windows, Linux and MacOSX for frequent failures. r?#intermittent-reviewers → Bug 1621655 - Disabled browser_styleeditor_media_sidebar_links.js on Linux and MacOSX for frequent failures. r?#intermittent-reviewers
Attachment #9134869 - Attachment description: Bug 1621655 - Disabled browser_styleeditor_media_sidebar_links.js on Linux and MacOSX for frequent failures. r?#intermittent-reviewers → Bug 1621655 - Disabled browser_styleeditor_media_sidebar_links.js on Linux and Windows for frequent failures. r?#intermittent-reviewers
Attachment #9134869 - Attachment description: Bug 1621655 - Disabled browser_styleeditor_media_sidebar_links.js on Linux and Windows for frequent failures. r?#intermittent-reviewers → Bug 1621655 - Disabled browser_styleeditor_media_sidebar_links.js on Linux and Windows for frequent failures. r=jmaher
Attachment #9134869 - Attachment description: Bug 1621655 - Disabled browser_styleeditor_media_sidebar_links.js on Linux and Windows for frequent failures. r=jmaher → Bug 1621655 - Disabled browser_styleeditor_media_sidebar_links.js on debug/asan for frequent failures. r=jmaher
Pushed by nbeleuzu@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/4376c20fd09e
Disabled browser_styleeditor_media_sidebar_links.js on debug/asan for frequent failures. r=jmaher
Whiteboard: [retriggered][stockwell disable-recommended] → [retriggered][stockwell disabled]
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 76
Status: RESOLVED → REOPENED
Keywords: leave-open
Resolution: FIXED → ---
Target Milestone: Firefox 76 → ---

Recently landed a fix affecting this test in Bug 1624889 that may have also resolved this. I've pushed to try-server with this test enabled at: https://treeherder.mozilla.org/#/jobs?repo=try&revision=9494560d619944d7515ce32825604326747d1552

Flags: needinfo?(mtigley)
Attachment #9136140 - Attachment description: Bug 1621655 - Re-enable browser_styleeditor_media_sidebar_links.js test → Bug 1621655 - Re-enable browser_styleeditor_media_sidebar_links.js on debug/asan builds
Pushed by mtigley@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/7bb4759349e9
Re-enable browser_styleeditor_media_sidebar_links.js on debug/asan builds  r=bradwerth

Unassigning myself from this for now since I'm not actively working on it.

Assignee: mtigley → nobody

I am seeing this failing consistently on try server when testing with a different resolution (1280x1024 vs 1600x1200). I see this fails shortly after a content-resize and then the media rules are not matching. I suspect this is due to a browser size issue.

:gl, can you help me find someone who can help fix this to work in 1280x1024 screen resolution?

Flags: needinfo?(gl)

I'll try again to find a fix for this. The first attempt clearly didn't work. Thanks for the detective work to determine that it has something to do with the resolution. That's incredibly helpful. Is there a way for me to run the test locally at different resolutions? I don't see a command line option with mochitest that appears to be relevant.

Assignee: nobody → bwerth
Flags: needinfo?(gl) → needinfo?(jmaher)

I only found this out with testing on different osx machines:
./mach try fuzzy --worker-suffix t-osx-1014=-staging ...

you are welcome to use that pool, there are 10 machines and they are used periodically

Flags: needinfo?(jmaher)
Pushed by bwerth@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/60c96f9da664
Make browser_styleeditor_media_sidebar_links.js test await RDM content reflow. r=rcaliman

This appears to be fixed. I'm not sure if I should close the bug. I'll let ccoroiu decide.

Flags: needinfo?(ccoroiu)

No failures since it was merged to central, it's fixed. Removing leave-open and adding appropriate flags.

Status: REOPENED → RESOLVED
Closed: 4 years ago4 years ago
Flags: needinfo?(ccoroiu)
Resolution: --- → FIXED
Target Milestone: --- → 81 Branch
Keywords: leave-open
Whiteboard: [retriggered][stockwell disabled] → [retriggered][stockwell fixed:patch]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: