Commit b44653f9 authored by Sunny Sachanandani's avatar Sunny Sachanandani Committed by Commit Bot

viz: DCHECK that clients don't send empty frames

The service rejects a CompostiorFrame if it has an empty render pass
list, or an empty output rect.  Add a DCHECK to ensure that clients
don't send these.

Change-Id: I2a1a25fab8b2e01f5355eaf0378c98d730490863
Reviewed-on: https://chromium-review.googlesource.com/c/1335268Reviewed-by: default avatarDaniel Cheng <dcheng@chromium.org>
Reviewed-by: default avatarFady Samuel <fsamuel@chromium.org>
Commit-Queue: Sunny Sachanandani <sunnyps@chromium.org>
Cr-Commit-Position: refs/heads/master@{#608087}
parent 9d9cd7ee
...@@ -28,6 +28,8 @@ struct StructTraits<viz::mojom::CompositorFrameDataView, viz::CompositorFrame> { ...@@ -28,6 +28,8 @@ struct StructTraits<viz::mojom::CompositorFrameDataView, viz::CompositorFrame> {
} }
static const viz::RenderPassList& passes(const viz::CompositorFrame& input) { static const viz::RenderPassList& passes(const viz::CompositorFrame& input) {
DCHECK(!input_render_pass_list.empty());
DCHECK(!input.render_pass_list.back()->output_rect.size().IsEmpty());
return input.render_pass_list; return input.render_pass_list;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment