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

Unified Diff: cc/ipc/compositor_frame_struct_traits.h

Issue 2092843002: Implement CompositorFrame Struct Traits (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Added unittest Created 4 years, 6 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/compositor_frame_struct_traits.h
diff --git a/cc/ipc/compositor_frame_struct_traits.h b/cc/ipc/compositor_frame_struct_traits.h
new file mode 100644
index 0000000000000000000000000000000000000000..ad24d53d51535b2be5b102106f7aea4498d517d7
--- /dev/null
+++ b/cc/ipc/compositor_frame_struct_traits.h
@@ -0,0 +1,42 @@
+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CC_IPC_COMPOSITOR_FRAME_STRUCT_TRAITS_H_
+#define CC_IPC_COMPOSITOR_FRAME_STRUCT_TRAITS_H_
+
+#include "cc/ipc/compositor_frame.mojom.h"
+#include "cc/output/compositor_frame.h"
+
+namespace mojo {
+
+template <>
+struct StructTraits<cc::mojom::CompositorFrame, cc::CompositorFrame> {
+ static const cc::CompositorFrameMetadata& metadata(
+ const cc::CompositorFrame& input) {
+ return input.metadata;
+ }
+
+ static const cc::TransferableResourceArray& resources(
+ const cc::CompositorFrame& input) {
+ return input.delegated_frame_data->resource_list;
+ }
+
+ static const cc::RenderPassList& passes(const cc::CompositorFrame& input) {
+ return input.delegated_frame_data->render_pass_list;
+ }
+
+ static bool Read(cc::mojom::CompositorFrameDataView data,
+ cc::CompositorFrame* out) {
+ if (!data.ReadMetadata(&out->metadata))
dcheng 2016/06/27 02:05:00 Let's move this to a .cc, since it's non-trivial.
Fady Samuel 2016/06/27 03:05:10 Done. I ended up also pulling out non-trivial Read
+ return false;
+
+ out->delegated_frame_data.reset(new cc::DelegatedFrameData());
+ return data.ReadResources(&out->delegated_frame_data->resource_list) &&
+ data.ReadPasses(&out->delegated_frame_data->render_pass_list);
+ }
+};
+
+} // namespace mojo
+
+#endif // CC_IPC_COMPOSITOR_FRAME_STRUCT_TRAITS_H_

Powered by Google App Engine
This is Rietveld 408576698