Closed Bug 1880605 Opened 1 years ago Closed 1 years ago

Intermittent [tier 2] widget/tests/test_contextmenu_by_mouse_on_unix.html | single tracking bug

Categories

(Core :: Graphics, defect)

Product:
Core
Component:
Graphics
Type:
defect

Tracking

()

Status:
RESOLVED FIXED
Milestone:
125 Branch
Tracking Flags:
Tracking Status
firefox-esr115 --- unaffected
firefox123 --- unaffected
firefox124 --- fixed
firefox125 --- fixed

People

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

References

(Regression)

Details

(Keywords: intermittent-failure, intermittent-testcase, regression, Whiteboard: [stockwell unknown])

Attachments

(1 file)

Bug 1880605 - Wait for updating OOP frame position. r=masayuki!
48 bytes, text/x-phabricator-request
diannaS
: approval-mozilla-beta+
Details | Review

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

[task 2024-02-16T07:10:25.984Z] 07:10:25     INFO - TEST-OK | widget/tests/test_clipboard_asyncSetData.html | took 575ms
[task 2024-02-16T07:10:26.082Z] 07:10:26     INFO - TEST-START | widget/tests/test_contextmenu_by_mouse_on_unix.html
[task 2024-02-16T07:15:48.642Z] 07:15:48     INFO - TEST-INFO | started process screentopng
[task 2024-02-16T07:15:48.779Z] 07:15:48     INFO - TEST-INFO | screentopng: exit 0
[task 2024-02-16T07:15:48.779Z] 07:15:48     INFO - Buffered messages logged at 07:10:26
[task 2024-02-16T07:15:48.780Z] 07:15:48     INFO - add_task | Entering test_fire_contextmenu_by_mousedown
[task 2024-02-16T07:15:48.780Z] 07:15:48     INFO - Buffered messages finished
[task 2024-02-16T07:15:48.781Z] 07:15:48     INFO - TEST-UNEXPECTED-FAIL | widget/tests/test_contextmenu_by_mouse_on_unix.html | Test timed out. - 
[task 2024-02-16T07:15:49.647Z] 07:15:49     INFO - Not taking screenshot here: see the one that was previously logged
[task 2024-02-16T07:15:49.647Z] 07:15:49     INFO - TEST-UNEXPECTED-FAIL | widget/tests/test_contextmenu_by_mouse_on_unix.html | [SimpleTest.finish()] No checks actually run. (You need to call ok(), is(), or similar functions at least once.  Make sure you use SimpleTest.waitForExplicitFinish() if you need it.) 
[task 2024-02-16T07:15:49.648Z] 07:15:49     INFO -     SimpleTest.ok@SimpleTest/SimpleTest.js:426:16
[task 2024-02-16T07:15:49.648Z] 07:15:49     INFO -     afterCleanup@SimpleTest/SimpleTest.js:1477:18
[task 2024-02-16T07:15:49.648Z] 07:15:49     INFO -     executeCleanupFunction@SimpleTest/SimpleTest.js:1562:7
[task 2024-02-16T07:15:49.649Z] 07:15:49     INFO -     SimpleTest.finish@SimpleTest/SimpleTest.js:1582:3
[task 2024-02-16T07:15:49.649Z] 07:15:49     INFO -     @SimpleTest/SimpleTest.js:2265:16
[task 2024-02-16T07:15:49.649Z] 07:15:49     INFO -     async*@SimpleTest/SimpleTest.js:2262:17
[task 2024-02-16T07:15:49.651Z] 07:15:49     INFO - GECKO(23925) | MEMORY STAT | vsize 2532MB | residentFast 100MB | heapAllocated 6MB
[task 2024-02-16T07:15:49.713Z] 07:15:49     INFO - TEST-OK | widget/tests/test_contextmenu_by_mouse_on_unix.html | took 323633ms
Keywords: regression
Regressed by: 1880439

:m_kato, since you are the author of the regressor, bug 1880439, could you take a look?

For more information, please visit BugBot documentation.

Flags: needinfo?(m_kato)
Assignee: nobody → m_kato
Flags: needinfo?(m_kato)

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

status-firefox123: --- → unaffected
status-firefox124: --- → affected
status-firefox125: --- → affected
status-firefox-esr115: --- → unaffected

mozInnerScreenX and mozInnerScreenY into OOP frame aren't updated when using SendsynthesizeNativeMouseEvent.

synthesizeNativeMouseEvent uses screen coordinates, so we should wait for
updating OOP frame position before using it.

Pushed by m_kato@ga2.so-net.ne.jp: https://hg.mozilla.org/integration/autoland/rev/689cf8cf7e6b Wait for updating OOP frame position. r=masayuki

https://hg.mozilla.org/mozilla-central/rev/689cf8cf7e6b

Status: NEW → RESOLVED
Closed: 1 years ago
status-firefox125: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → 125 Branch

The patch landed in nightly and beta is affected.
:m_kato, is this bug important enough to require an uplift?

  • If yes, please nominate the patch for beta approval.
  • If no, please set status-firefox124 to wontfix.

For more information, please visit BugBot documentation.

Flags: needinfo?(m_kato)

Comment on attachment 9382429 [details]
Bug 1880605 - Wait for updating OOP frame position. r=masayuki!

Beta/Release Uplift Approval Request

  • User impact if declined: No. This issue is a frequency test failure
  • Is this code covered by automated tests?: Yes
  • Has the fix been verified in Nightly?: Yes
  • Needs manual test from QE?: No
  • If yes, steps to reproduce:
  • List of other uplifts needed: None
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): This is mochitest fix. Nothing risk.
  • String changes made/needed:
  • Is Android affected?: No
Flags: needinfo?(m_kato)
Attachment #9382429 - Flags: approval-mozilla-beta?

Comment on attachment 9382429 [details]
Bug 1880605 - Wait for updating OOP frame position. r=masayuki!

Approved for 124.0b5

Attachment #9382429 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: