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

Unified Diff: cc/ipc/surface_id_struct_traits.h

Issue 2388753003: Introduce cc::LocalFrameId and use in SurfaceFactory (Closed)
Patch Set: Fix exo_unittests Created 4 years, 2 months 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/ipc/surface_id_struct_traits.h
diff --git a/cc/ipc/surface_id_struct_traits.h b/cc/ipc/surface_id_struct_traits.h
index d8be65f4e1c218245fbb12985dff15ac5a2faf60..6f66c9fd31a0ab29f8e0a033a8270e2fc5ab0556 100644
--- a/cc/ipc/surface_id_struct_traits.h
+++ b/cc/ipc/surface_id_struct_traits.h
@@ -6,6 +6,7 @@
#define CC_IPC_SURFACE_ID_STRUCT_TRAITS_H_
#include "cc/ipc/frame_sink_id_struct_traits.h"
+#include "cc/ipc/local_frame_id_struct_traits.h"
#include "cc/ipc/surface_id.mojom-shared.h"
#include "cc/surfaces/frame_sink_id.h"
#include "cc/surfaces/surface_id.h"
@@ -18,15 +19,20 @@ struct StructTraits<cc::mojom::SurfaceIdDataView, cc::SurfaceId> {
return id.frame_sink_id();
}
- static uint32_t local_id(const cc::SurfaceId& id) { return id.local_id(); }
-
- static uint64_t nonce(const cc::SurfaceId& id) { return id.nonce(); }
+ static const cc::LocalFrameId& local_frame_id(const cc::SurfaceId& id) {
+ return id.local_frame_id();
+ }
static bool Read(cc::mojom::SurfaceIdDataView data, cc::SurfaceId* out) {
cc::FrameSinkId frame_sink_id;
if (!data.ReadFrameSinkId(&frame_sink_id))
return false;
- *out = cc::SurfaceId(frame_sink_id, data.local_id(), data.nonce());
+
+ cc::LocalFrameId local_frame_id;
+ if (!data.ReadLocalFrameId(&local_frame_id))
+ return false;
+
+ *out = cc::SurfaceId(frame_sink_id, local_frame_id);
return true;
}
};
« no previous file with comments | « cc/ipc/surface_id.mojom ('k') | cc/ipc/typemaps.gni » ('j') | cc/surfaces/surface_factory.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698