Closed
Bug 1770838
Opened 3 years ago
Closed 3 years ago
Fix stray reference to renamed function UpdateVsyncParentVsyncSource()
Categories
(Core :: Graphics, task)
Core
Graphics
Tracking
()
RESOLVED
FIXED
102 Branch
Tracking | Status | |
---|---|---|
firefox102 | --- | fixed |
People
(Reporter: dholbert, Assigned: dholbert)
References
Details
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
We renamed UpdateVsyncParentVsyncSource with s/Source/Dispatcher/ in bug 1765399 (specifically https://hg.mozilla.org/mozilla-central/rev/22b568631834e556ab8351ac8458d5b24aa638bd ) , but we have one straggling reference to the old function name:
https://searchfox.org/mozilla-central/search?q=UpdateVsyncParentVsyncSource&redirect=false
...which is:
https://searchfox.org/mozilla-central/rev/a688eb56e8f69acef88bbad1b224b5bd70dba842/widget/gtk/nsWindow.cpp#5906-5908
#ifdef MOZ_WAYLAND
// Initialize the window specific VsyncSource early in order to avoid races
// with BrowserParent::UpdateVsyncParentVsyncSource().
Let's fix that stray comment.
Assignee | ||
Comment 1•3 years ago
|
||
DONTBUILD
This patch doesn't change behavior; it just swaps out
s/UpdateVsyncParentVsyncSource/UpdateVsyncParentVsyncDispatcher/
in a code-comemnt.
(This is the only mention of UpdateVsyncParentVsyncSource in our codebase.)
Pushed by dholbert@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/2ffbc66c6556
Update a code-comment's reference to a vsync-related function that's been renamed. r=mstange
Comment 3•3 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 102 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•