From 9b652d3dd47bd00a9970bb18ebbd2f455db94d7c Mon Sep 17 00:00:00 2001
From: emersion <contact@emersion.fr>
Date: Sat, 23 Jun 2018 18:42:19 +0100
Subject: [PATCH] screencopy: remove invalid_output_region, format and stride
 are now enforced

---
 unstable/wlr-screencopy-unstable-v1.xml | 23 ++++++++---------------
 1 file changed, 8 insertions(+), 15 deletions(-)

diff --git a/unstable/wlr-screencopy-unstable-v1.xml b/unstable/wlr-screencopy-unstable-v1.xml
index 4327415..a7a2d17 100644
--- a/unstable/wlr-screencopy-unstable-v1.xml
+++ b/unstable/wlr-screencopy-unstable-v1.xml
@@ -43,11 +43,6 @@
       source.
     </description>
 
-    <enum name="error">
-      <entry name="invalid_output_region" value="0"
-        summary="tried to capture an invalid output region"/>
-    </enum>
-
     <request name="capture_output">
       <description summary="capture an output">
         Capture the next frame of an entire output.
@@ -63,8 +58,8 @@
         Capture the next frame of an output's region.
 
         The region is given in output logical coordinates, see
-        xdg_output.logical_size. Trying to capture a region spanning outside the
-        output extents is a protocol error.
+        xdg_output.logical_size. The region will be clipped to the output's
+        extents.
       </description>
       <arg name="frame" type="new_id" interface="zwlr_screencopy_frame_v1"/>
       <arg name="overlay_cursor" type="int"
@@ -104,14 +99,13 @@
         Provides information about the frame's buffer. This event is sent once
         as soon as the frame is created.
 
-        The client should then create a buffer with the provided width and
-        height, and send a "copy" request. It can optionally create a buffer
-        with the preferred format and stride.
+        The client should then create a buffer with the provided attributes, and
+        send a "copy" request.
       </description>
+      <arg name="format" type="uint" summary="buffer format"/>
       <arg name="width" type="uint" summary="buffer width"/>
       <arg name="height" type="uint" summary="buffer height"/>
-      <arg name="format" type="uint" summary="preferred DRM_FORMAT"/>
-      <arg name="stride" type="uint" summary="preferred stride"/>
+      <arg name="stride" type="uint" summary="buffer stride"/>
     </event>
 
     <request name="copy">
@@ -129,9 +123,8 @@
     <enum name="error">
       <entry name="already_used" value="0"
         summary="the object has already been used to copy a wl_buffer"/>
-      <entry name="invalid_format" value="1" summary="format not supported"/>
-      <entry name="invalid_dimensions" value="2"
-        summary="invalid width or height"/>
+      <entry name="invalid_buffer" value="1"
+        summary="buffer attributes are invalid"/>
     </enum>
 
     <enum name="flags" bitfield="true">