mollyim/webrtc
1
0
Fork 0
mirror of https://github.com/mollyim/webrtc.git synced 2025-05-13 05:40:42 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions

Adds debug logs which warns about cursor flickering

Browse source 
Main issue is https://chromium-review.googlesource.com/c/chromium/src/+/4507078

Bug: chromium:1421656
Change-Id: I61c6df59176e10bc29356ea924a4e483270db75f
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/304284
Reviewed-by: Alexander Cooper <alcooper@chromium.org>
Commit-Queue: Henrik Andreassson <henrika@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#40038}
This commit is contained in:
henrika 2023-05-05 15:13:34 +02:00 committed by WebRTC LUCI CQ
parent cf75442ad3
commit 2d7424305d
1 changed files with 17 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

27
modules/desktop_capture/desktop_and_cursor_composer.cc
View file

@ -27,6 +27,17 @@ namespace webrtc {
namespace {
// Global reference counter which is increased when a DesktopFrameWithCursor is
// created and decreased when the same object is destructed. Only used for
// debugging purposes to ensure that we never end up in state where
// `g_ref_count` is larger than one since that could indicate a flickering
// cursor (cursor-less version of the frame is not restored properly and it can
// can lead to visible trails of old cursors).
// See https://crbug.com/1421656#c99 for more details.
int g_ref_count = 0;
uint64_t g_num_flicker_warnings = 0;
// Helper function that blends one image into another. Source image must be
// pre-multiplied with the alpha channel. Destination is assumed to be opaque.
void AlphaBlend(uint8_t* dest,
@ -96,6 +107,7 @@ DesktopFrameWithCursor::DesktopFrameWithCursor(
frame->data(),
frame->shared_memory()),
original_frame_(std::move(frame)) {
++g_ref_count;
MoveFrameInfoFrom(original_frame_.get());
DesktopVector image_pos = position.subtract(cursor.hotspot());
@ -105,11 +117,6 @@ DesktopFrameWithCursor::DesktopFrameWithCursor(
cursor_rect_.IntersectWith(DesktopRect::MakeSize(size()));
if (!previous_cursor_rect.equals(cursor_rect_)) {
RTC_LOG(LS_VERBOSE) << "[MOUSE] cursors moved => cursor_rect=("
<< cursor_rect_.top_left().x() << ","
<< cursor_rect_.top_left().y() << ") ("
<< cursor_rect_.size().width() << "x"
<< cursor_rect_.size().height() << ")";
mutable_updated_region()->AddRect(cursor_rect_);
// TODO(crbug:1323241) Update this code to properly handle the case where
// |previous_cursor_rect| is outside of the boundaries of |frame|.
@ -118,11 +125,6 @@ DesktopFrameWithCursor::DesktopFrameWithCursor(
// we're running on.
mutable_updated_region()->AddRect(previous_cursor_rect);
} else if (cursor_changed) {
RTC_LOG(LS_VERBOSE) << "[MOUSE] cursor changed => cursor_rect=("
<< cursor_rect_.top_left().x() << ","
<< cursor_rect_.top_left().y() << ") ("
<< cursor_rect_.size().width() << "x"
<< cursor_rect_.size().height() << ")";
mutable_updated_region()->AddRect(cursor_rect_);
}
@ -148,6 +150,11 @@ DesktopFrameWithCursor::DesktopFrameWithCursor(
}
DesktopFrameWithCursor::~DesktopFrameWithCursor() {
if (--g_ref_count > 0) {
++g_num_flicker_warnings;
RTC_LOG(LS_WARNING) << "Cursor might be flickering; number of warnings="
<< g_num_flicker_warnings;
}
// Restore original content of the frame.
if (restore_frame_) {
DesktopRect target_rect = DesktopRect::MakeSize(restore_frame_->size());