Index: cc/quads/render_pass.cc |
diff --git a/cc/quads/render_pass.cc b/cc/quads/render_pass.cc |
index 567145a82d6d12565c6a835aef264d32ba94ab9c..5c4d98a33c96cbbf5f03f4b4f5740befe78057d9 100644 |
--- a/cc/quads/render_pass.cc |
+++ b/cc/quads/render_pass.cc |
@@ -6,6 +6,7 @@ |
#include <algorithm> |
+#include "base/numerics/safe_conversions.h" |
#include "base/trace_event/trace_event_argument.h" |
#include "base/values.h" |
#include "cc/base/math_util.h" |
@@ -52,8 +53,7 @@ scoped_ptr<RenderPass> RenderPass::Create(size_t shared_quad_state_list_size, |
} |
RenderPass::RenderPass() |
- : id(RenderPassId(-1, -1)), |
- has_transparent_background(true), |
+ : has_transparent_background(true), |
quad_list(kDefaultNumQuadsToReserve), |
shared_quad_state_list(sizeof(SharedQuadState), |
kDefaultNumSharedQuadStatesToReserve) { |
@@ -62,16 +62,14 @@ RenderPass::RenderPass() |
// Each layer usually produces one shared quad state, so the number of layers |
// is a good hint for what to reserve here. |
RenderPass::RenderPass(size_t num_layers) |
- : id(RenderPassId(-1, -1)), |
- has_transparent_background(true), |
+ : has_transparent_background(true), |
quad_list(kDefaultNumQuadsToReserve), |
shared_quad_state_list(sizeof(SharedQuadState), num_layers) { |
} |
RenderPass::RenderPass(size_t shared_quad_state_list_size, |
size_t quad_list_size) |
- : id(RenderPassId(-1, -1)), |
- has_transparent_background(true), |
+ : has_transparent_background(true), |
quad_list(quad_list_size), |
shared_quad_state_list(sizeof(SharedQuadState), |
shared_quad_state_list_size) { |
@@ -148,7 +146,6 @@ void RenderPass::SetNew(RenderPassId id, |
const gfx::Rect& damage_rect, |
const gfx::Transform& transform_to_root_target) { |
DCHECK_GT(id.layer_id, 0); |
- DCHECK_GE(id.index, 0); |
DCHECK(damage_rect.IsEmpty() || output_rect.Contains(damage_rect)) |
<< "damage_rect: " << damage_rect.ToString() |
<< " output_rect: " << output_rect.ToString(); |
@@ -168,7 +165,6 @@ void RenderPass::SetAll(RenderPassId id, |
const gfx::Transform& transform_to_root_target, |
bool has_transparent_background) { |
DCHECK_GT(id.layer_id, 0); |
- DCHECK_GE(id.index, 0); |
this->id = id; |
this->output_rect = output_rect; |
@@ -185,7 +181,8 @@ void RenderPass::AsValueInto(base::trace_event::TracedValue* value) const { |
MathUtil::AddToTracedValue("damage_rect", damage_rect, value); |
value->SetBoolean("has_transparent_background", has_transparent_background); |
- value->SetInteger("copy_requests", copy_requests.size()); |
+ value->SetInteger("copy_requests", |
+ base::saturated_cast<int>(copy_requests.size())); |
value->BeginArray("shared_quad_state_list"); |
for (const auto& shared_quad_state : shared_quad_state_list) { |