Commit 65e58340 authored by Robert Mader's avatar Robert Mader
Browse files

Bug 1726923 - Remove asserts on valid viewport values again, r=gfx-reviewers,lsalzman 

This reverts D123286 as we now have a generic way to trigger crash
reports on Wayland protocol errors.

Depends on D123894

Differential Revision: https://phabricator.services.mozilla.com/D123897
parent 51d265f2

gfx/layers/NativeLayerWayland.cpp

+0 −9
Original line number Diff line number Diff line
@@ -685,11 +685,6 @@ void NativeLayerWayland::SetViewportSourceRect(const Rect aSourceRect) { 
  }



  mViewportSourceRect = aSourceRect;

  MOZ_RELEASE_ASSERT(

      (mViewportSourceRect.x >= 0 && mViewportSourceRect.y >= 0 &&

       mViewportSourceRect.width > 0 && mViewportSourceRect.height > 0) ||

      (mViewportSourceRect.x == -1 && mViewportSourceRect.y == -1 &&

       mViewportSourceRect.width == -1 && mViewportSourceRect.height == -1));

  wp_viewport_set_source(mViewport, wl_fixed_from_double(mViewportSourceRect.x),

                         wl_fixed_from_double(mViewportSourceRect.y),

                         wl_fixed_from_double(mViewportSourceRect.width),
@@ -706,10 +701,6 @@ void NativeLayerWayland::SetViewportDestinationSize(int aWidth, int aHeight) { 


  mViewportDestinationSize.width = aWidth;

  mViewportDestinationSize.height = aHeight;

  MOZ_RELEASE_ASSERT((mViewportDestinationSize.width > 0 &&

                      mViewportDestinationSize.height > 0) ||

                     (mViewportDestinationSize.width == -1 &&

                      mViewportDestinationSize.height == -1));

  wp_viewport_set_destination(mViewport, mViewportDestinationSize.width,

                              mViewportDestinationSize.height);

}