Index: cc/surfaces/surface_aggregator_unittest.cc |
diff --git a/cc/surfaces/surface_aggregator_unittest.cc b/cc/surfaces/surface_aggregator_unittest.cc |
index 53b01e046b67c24d594651c6ab2a57b1ccbc5d17..d9302c3f658a84b307ab7bf0987ac53e48d06d2f 100644 |
--- a/cc/surfaces/surface_aggregator_unittest.cc |
+++ b/cc/surfaces/surface_aggregator_unittest.cc |
@@ -2,6 +2,8 @@ |
// Use of this source code is governed by a BSD-style license that can be |
// found in the LICENSE file. |
+#include <limits> |
+ |
#include "cc/output/compositor_frame.h" |
#include "cc/output/delegated_frame_data.h" |
#include "cc/quads/render_pass.h" |
@@ -30,9 +32,7 @@ namespace cc { |
namespace { |
SurfaceId InvalidSurfaceId() { |
- static SurfaceId invalid; |
- invalid.id = static_cast<uint64_t>(-1); |
- return invalid; |
+ return SurfaceId::FromUnsafeValue(std::numeric_limits<uint64_t>::max()); |
} |
gfx::Size SurfaceSize() { |
@@ -88,7 +88,7 @@ class SurfaceAggregatorTest : public testing::Test { |
}; |
TEST_F(SurfaceAggregatorTest, ValidSurfaceNoFrame) { |
- SurfaceId one_id(7); |
+ SurfaceId one_id = SurfaceId::FromUnsafeValue(7); |
factory_.Create(one_id); |
Surface* surface = manager_.GetSurfaceForId(one_id); |
@@ -1806,7 +1806,7 @@ void SubmitCompositorFrameWithResources(ResourceId* resource_ids, |
TEST_F(SurfaceAggregatorWithResourcesTest, TakeResourcesOneSurface) { |
ResourceTrackingSurfaceFactoryClient client; |
SurfaceFactory factory(&manager_, &client); |
- SurfaceId surface_id(7u); |
+ SurfaceId surface_id = SurfaceId::FromUnsafeValue(7u); |
factory.Create(surface_id); |
Surface* surface = manager_.GetSurfaceForId(surface_id); |
@@ -1845,7 +1845,7 @@ TEST_F(SurfaceAggregatorWithResourcesTest, TakeResourcesOneSurface) { |
TEST_F(SurfaceAggregatorWithResourcesTest, TakeInvalidResources) { |
ResourceTrackingSurfaceFactoryClient client; |
SurfaceFactory factory(&manager_, &client); |
- SurfaceId surface_id(7u); |
+ SurfaceId surface_id = SurfaceId::FromUnsafeValue(7u); |
factory.Create(surface_id); |
Surface* surface = manager_.GetSurfaceForId(surface_id); |
@@ -1885,11 +1885,11 @@ TEST_F(SurfaceAggregatorWithResourcesTest, TakeInvalidResources) { |
TEST_F(SurfaceAggregatorWithResourcesTest, TwoSurfaces) { |
ResourceTrackingSurfaceFactoryClient client; |
SurfaceFactory factory(&manager_, &client); |
- SurfaceId surface1_id(7u); |
+ SurfaceId surface1_id = SurfaceId::FromUnsafeValue(7u); |
factory.Create(surface1_id); |
Surface* surface1 = manager_.GetSurfaceForId(surface1_id); |
- SurfaceId surface2_id(8u); |
+ SurfaceId surface2_id = SurfaceId::FromUnsafeValue(8u); |
factory.Create(surface2_id); |
Surface* surface2 = manager_.GetSurfaceForId(surface2_id); |
@@ -1940,13 +1940,13 @@ TEST_F(SurfaceAggregatorWithResourcesTest, TwoSurfaces) { |
TEST_F(SurfaceAggregatorWithResourcesTest, InvalidChildSurface) { |
ResourceTrackingSurfaceFactoryClient client; |
SurfaceFactory factory(&manager_, &client); |
- SurfaceId root_surface_id(7u); |
+ SurfaceId root_surface_id = SurfaceId::FromUnsafeValue(7u); |
factory.Create(root_surface_id); |
Surface* root_surface = manager_.GetSurfaceForId(root_surface_id); |
- SurfaceId middle_surface_id(8u); |
+ SurfaceId middle_surface_id = SurfaceId::FromUnsafeValue(8u); |
factory.Create(middle_surface_id); |
Surface* middle_surface = manager_.GetSurfaceForId(middle_surface_id); |
- SurfaceId child_surface_id(9u); |
+ SurfaceId child_surface_id = SurfaceId::FromUnsafeValue(9u); |
factory.Create(child_surface_id); |
Surface* child_surface = manager_.GetSurfaceForId(child_surface_id); |