Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(255)

Unified Diff: remoting/protocol/client_control_dispatcher.cc

Issue 833693002: Hide the cursor on the client when it is hidden on the host (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address Sergey's feedback Created 5 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « remoting/host/video_scheduler.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/protocol/client_control_dispatcher.cc
diff --git a/remoting/protocol/client_control_dispatcher.cc b/remoting/protocol/client_control_dispatcher.cc
index 599c779fbae4a3e4ea21f86e41c24a23d99221db..5a6bad59f49bf1019b53869ec19c11217eceaad2 100644
--- a/remoting/protocol/client_control_dispatcher.cc
+++ b/remoting/protocol/client_control_dispatcher.cc
@@ -38,8 +38,8 @@ bool CursorShapeIsValid(const CursorShapeInfo& cursor_shape) {
// Verify that |width| and |height| are within sane limits. Otherwise integer
// overflow can occur while calculating |cursor_total_bytes| below.
- if (width <= 0 || width > (SHRT_MAX / 2) ||
- height <= 0 || height > (SHRT_MAX / 2)) {
+ if (width < 0 || width > (SHRT_MAX / 2) ||
+ height < 0 || height > (SHRT_MAX / 2)) {
LOG(ERROR) << "Cursor dimensions are out of bounds for SetCursor: "
<< width << "x" << height;
return false;
« no previous file with comments | « remoting/host/video_scheduler.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698