Irrational check in the constructor of DesktopFrame: stride_ may be negative

webrtc/modules/desktop_capture/desktop_frame.cc

 /*
  *  Copyright (c) 2013 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
@@ skipping 21 matching lines @@
 DesktopFrame::DesktopFrame(DesktopRect rect,
                            int stride,
                            uint8_t* data,
                            SharedMemory* shared_memory)
     : data_(data),
       shared_memory_(shared_memory),
       rect_(rect),
       stride_(stride),
       capture_time_ms_(0),
       capturer_id_(DesktopCapturerId::kUnknown) {
-  RTC_DCHECK(stride_ >= 0);
   RTC_DCHECK(rect.width() >= 0);
   RTC_DCHECK(rect.height() >= 0);
 }
 
 DesktopFrame::~DesktopFrame() = default;
 
 void DesktopFrame::CopyPixelsFrom(const uint8_t* src_buffer, int src_stride,
                                   const DesktopRect& dest_rect) {
   RTC_CHECK(DesktopRect::MakeSize(size()).ContainsRect(dest_rect));
 
@@ skipping 90 matching lines @@
     : DesktopFrame(rect,
                    stride,
                    reinterpret_cast<uint8_t*>(shared_memory->data()),
                    shared_memory) {}
 
 SharedMemoryDesktopFrame::~SharedMemoryDesktopFrame() {
   delete shared_memory_;
 }
 
 }  // namespace webrtc