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

Unified Diff: cc/surfaces/surface_manager_ref_unittest.cc

Issue 2494833002: Revert of Replaced cc::SurfaceId::nonce_ with base::UnguessableToken (Closed)
Patch Set: 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
« no previous file with comments | « cc/surfaces/surface_manager.cc ('k') | cc/surfaces/surface_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 42b5ccc049b2059aea1cd79d4619d6e4e05ddb96..7a7ab5e40fa5e28526292af67eb4188344bb4f5d 100644
--- a/cc/surfaces/surface_manager_ref_unittest.cc
+++ b/cc/surfaces/surface_manager_ref_unittest.cc
@@ -21,8 +21,8 @@
constexpr FrameSinkId kFrameSink1(1, 0);
constexpr FrameSinkId kFrameSink2(2, 0);
constexpr FrameSinkId kFrameSink3(3, 0);
-const LocalFrameId kLocalFrame1(1, base::UnguessableToken::Create());
-const LocalFrameId kLocalFrame2(2, base::UnguessableToken::Create());
+constexpr LocalFrameId kLocalFrame1(1, 0);
+constexpr LocalFrameId kLocalFrame2(2, 0);
// Tests for reference tracking in SurfaceManager.
class SurfaceManagerRefTest : public testing::Test {
@@ -247,8 +247,7 @@
TEST_F(SurfaceManagerRefTest, TryAddReferenceToBadSurface) {
// Not creating an accompanying Surface and SurfaceFactory.
- SurfaceId id(FrameSinkId(100u, 200u),
- LocalFrameId(1u, base::UnguessableToken::Create()));
+ SurfaceId id(FrameSinkId(100u, 200u), LocalFrameId(1u, 123123u));
// Adding reference from root to the Surface should do nothing because
// SurfaceManager doesn't know Surface for |id| exists.
« no previous file with comments | « cc/surfaces/surface_manager.cc ('k') | cc/surfaces/surface_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698