zqz/platform-external-webrtc
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Make deprecated OnMouseCursorPosition overload optional.

Browse Source 
The only callers or non-trivial implementations of this that I could
find are in remoting/ in Chromium, which I plan on fixing once this
gets rolled.

Bug: chromium:1043325
Change-Id: Id5a33fc09bb066f979876b2a7dcbc3dc5c2d3dd9
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/166560
Commit-Queue: Jamie Walch <jamiewalch@chromium.org>
Reviewed-by: Sergey Ulanov <sergeyu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#30310}
This commit is contained in:
Jamie Walch
2020-01-17 13:52:48 -08:00
committed by Commit Bot
parent 100fe639b2
commit 4b47dd39a7
4 changed files with 1 additions and 15 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
modules/desktop_capture/desktop_and_cursor_composer.cc
View File

@ -197,12 +197,6 @@ void DesktopAndCursorComposer::OnMouseCursor(MouseCursor* cursor) {
cursor_.reset(cursor);
}
void DesktopAndCursorComposer::OnMouseCursorPosition(
MouseCursorMonitor::CursorState state,
const DesktopVector& position) {
RTC_NOTREACHED();
}
void DesktopAndCursorComposer::OnMouseCursorPosition(
const DesktopVector& position) {
cursor_position_ = position;

2
modules/desktop_capture/desktop_and_cursor_composer.h
View File

@ -63,8 +63,6 @@ class RTC_EXPORT DesktopAndCursorComposer
// MouseCursorMonitor::Callback interface.
void OnMouseCursor(MouseCursor* cursor) override;
void OnMouseCursorPosition(MouseCursorMonitor::CursorState state,
const DesktopVector& position) override;
void OnMouseCursorPosition(const DesktopVector& position) override;
const std::unique_ptr<DesktopCapturer> desktop_capturer_;

3
modules/desktop_capture/mouse_cursor_monitor.h
View File

@ -54,7 +54,7 @@ class MouseCursorMonitor {
// relative to the |window| specified in the constructor.
// Deprecated: use the following overload instead.
virtual void OnMouseCursorPosition(CursorState state,
const DesktopVector& position) = 0;
const DesktopVector& position) {}
// Called in response to Capture(). |position| indicates cursor absolute
// position on the system in fullscreen coordinate, i.e. the top-left
@ -62,7 +62,6 @@ class MouseCursorMonitor {
// The coordinates of the position is controlled by OS, but it's always
// consistent with DesktopFrame.rect().top_left().
// TODO(zijiehe): Ensure all implementations return the absolute position.
// TODO(zijiehe): Make this function pure virtual after Chromium changes.
// TODO(zijiehe): Current this overload works correctly only when capturing
// mouse cursor against fullscreen.
virtual void OnMouseCursorPosition(const DesktopVector& position) {}

5
modules/desktop_capture/mouse_cursor_monitor_unittest.cc
View File

@ -34,11 +34,6 @@ class MouseCursorMonitorTest : public ::testing::Test,
cursor_image_.reset(cursor_image);
}
void OnMouseCursorPosition(MouseCursorMonitor::CursorState state,
const DesktopVector& position) override {
RTC_NOTREACHED();
}
void OnMouseCursorPosition(const DesktopVector& position) override {
position_ = position;
position_received_ = true;