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

Unified Diff: cc/surfaces/surface_id.h

Issue 2379653006: Replaced cc::SurfaceId::nonce_ with base::UnguessableToken (Closed)
Patch Set: Changed SurfaceManager::kRootSurfaceId to a private field to avoid static initialization 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_hittest_unittest.cc ('k') | cc/surfaces/surface_id_allocator.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/surfaces/surface_id.h
diff --git a/cc/surfaces/surface_id.h b/cc/surfaces/surface_id.h
index ae16b232bfbc7c5f1332daf1a8c88e29ac29befb..aa6c9c5d667fe420743e97585f796378b165076f 100644
--- a/cc/surfaces/surface_id.h
+++ b/cc/surfaces/surface_id.h
@@ -25,17 +25,13 @@ class SurfaceId {
constexpr SurfaceId(const SurfaceId& other) = default;
- // A SurfaceId consists of three components: FrameSinkId, local Id, and nonce.
+ // A SurfaceId consists of two components: FrameSinkId and LocalFrameId.
// A |frame_sink_id| consists of two components; one is allocated by the
// display compositor service and one is allocated by the client. The
// |frame_sink_id| uniquely identifies a FrameSink (and frame source).
- // A |local_id| is a sequentially allocated ID generated by the frame source
- // that uniquely identifies a sequential set of frames of the same size and
- // device scale factor.
- // A |nonce| is a cryptographically secure unguessable token that makes it
- // impossible for an unprivileged frame source to embed another frame source
- // without being explicitly given the surface ID of that frame source from a
- // privileged process.
+ // A |local_frame_id| is a sequentially allocated ID generated by the frame
+ // source that uniquely identifies a sequential set of frames of the same size
+ // and device scale factor.
constexpr SurfaceId(const FrameSinkId& frame_sink_id,
const LocalFrameId& local_frame_id)
: frame_sink_id_(frame_sink_id), local_frame_id_(local_frame_id) {}
« no previous file with comments | « cc/surfaces/surface_hittest_unittest.cc ('k') | cc/surfaces/surface_id_allocator.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698