Closed Bug 1847257 Opened 1 year ago Closed 1 year ago

revert change to ordering of ProcessPendingXEvents() call in window_capturer_x11.cc

Categories

(Core :: WebRTC, task, P2)

task

Tracking

()

RESOLVED FIXED
118 Branch
Tracking Status
firefox118 --- fixed

People

(Reporter: mjf, Assigned: mjf)

Details

Attachments

(1 file)

revert change to ordering of x_display_->ProcessPendingXEvents(); call in modules/desktop_capture/linux/x11/window_capturer_x11.cc

This will help reduce our libwebrtc patch-stack.

From Karl:
Added in https://hg.mozilla.org/mozilla-central/rev/ffb95db2dc7e#l6.165.
Seems to already be upstream in https://hg.mozilla.org/mozilla-central/rev/f5a4769477bf#l1338.112.
Moved in https://hg.mozilla.org/mozilla-central/rev/55c60f012e8f#l94.88 as "previously applied" change that did not exist previously.
Looks unnecessary.

Assignee: nobody → mfroman
Severity: -- → S3
Priority: -- → P2

This small change we've been carrying to when ProcessPendingXEvents is called
in WindowCapturerLinux::Capture should not be necessary.

Pushed by mfroman@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/9633cb1dde33 revert an unneccesary change in libwebrtc's window_capturer_x11.cc r=karlt,webrtc-reviewers
Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 118 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: