Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2015)

Unified Diff: cc/surfaces/surface_manager_ref_unittest.cc

Issue 2379653006: Replaced cc::SurfaceId::nonce_ with base::UnguessableToken (Closed)
Patch Set: Nit change based on comment. Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: cc/surfaces/surface_manager_ref_unittest.cc
diff --git a/cc/surfaces/surface_manager_ref_unittest.cc b/cc/surfaces/surface_manager_ref_unittest.cc
index 7a7ab5e40fa5e28526292af67eb4188344bb4f5d..42b5ccc049b2059aea1cd79d4619d6e4e05ddb96 100644
--- a/cc/surfaces/surface_manager_ref_unittest.cc
+++ b/cc/surfaces/surface_manager_ref_unittest.cc
@@ -21,8 +21,8 @@ namespace {
constexpr FrameSinkId kFrameSink1(1, 0);
constexpr FrameSinkId kFrameSink2(2, 0);
constexpr FrameSinkId kFrameSink3(3, 0);
-constexpr LocalFrameId kLocalFrame1(1, 0);
-constexpr LocalFrameId kLocalFrame2(2, 0);
+const LocalFrameId kLocalFrame1(1, base::UnguessableToken::Create());
+const LocalFrameId kLocalFrame2(2, base::UnguessableToken::Create());
// Tests for reference tracking in SurfaceManager.
class SurfaceManagerRefTest : public testing::Test {
@@ -247,7 +247,8 @@ TEST_F(SurfaceManagerRefTest, CheckGCWithSequences) {
TEST_F(SurfaceManagerRefTest, TryAddReferenceToBadSurface) {
// Not creating an accompanying Surface and SurfaceFactory.
- SurfaceId id(FrameSinkId(100u, 200u), LocalFrameId(1u, 123123u));
+ SurfaceId id(FrameSinkId(100u, 200u),
+ LocalFrameId(1u, base::UnguessableToken::Create()));
// Adding reference from root to the Surface should do nothing because
// SurfaceManager doesn't know Surface for |id| exists.

Powered by Google App Engine
This is Rietveld 408576698