CopyOutputResult must have a working release callback when received over mojo

cc/ipc/BUILD.gn

 # 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.
 
 import("//build/config/ui.gni")
 import("//cc/cc.gni")
 import("//mojo/public/tools/bindings/mojom.gni")
 
 cc_component("ipc") {
   output_name = "cc_ipc"
@@ skipping 19 matching lines @@
     "//ipc",
     "//ui/events/ipc",
     "//ui/gfx",
     "//ui/gfx/ipc",
     "//ui/gfx/ipc/color",
     "//ui/gfx/ipc/geometry",
     "//ui/gfx/ipc/skia",
   ]
 }
 
+mojom("texture_mailbox_releaser_interface") {
+  sources = [
+    "texture_mailbox_releaser.mojom",

[danakj, 2017/02/10 18:32:50]

why isn't this part of "interfaces" below?

[Saman Sami, 2017/02/10 19:10:07]

because struct traits depending on interfaces creates a cycle.

[danakj, 2017/02/10 22:51:03]

Can you leave a comment here explaining this is a helper interface used by
struct traits? And maybe put it lower in the file so people don't cargo-cult
into it as easily?

+  ]
+  public_deps = [
+    "//gpu/ipc/common:interfaces",
+  ]
+}
+
 mojom("interfaces") {
   sources = [
     "begin_frame_args.mojom",
     "compositor_frame.mojom",
     "compositor_frame_metadata.mojom",
     "copy_output_request.mojom",
     "copy_output_result.mojom",
     "filter_operation.mojom",
     "filter_operations.mojom",
     "frame_sink_id.mojom",
     "local_surface_id.mojom",
     "mojo_compositor_frame_sink.mojom",
     "quads.mojom",
     "render_pass.mojom",
     "returned_resource.mojom",
     "selection.mojom",
     "shared_quad_state.mojom",
     "surface_id.mojom",
     "surface_info.mojom",
     "surface_reference.mojom",
     "surface_sequence.mojom",
     "texture_mailbox.mojom",
     "transferable_resource.mojom",
   ]
 
   public_deps = [
+    "//cc/ipc:texture_mailbox_releaser_interface",
     "//gpu/ipc/common:interfaces",
     "//mojo/common:common_custom_types",
     "//skia/public/interfaces",
     "//ui/events/mojo:interfaces",
     "//ui/gfx/geometry/mojo",
     "//ui/gfx/mojo",
   ]
 }
 
 # Interfaces between two privileged services.
@@ skipping 47 matching lines @@
     "surface_reference_struct_traits.h",
     "surface_sequence_struct_traits.h",
     "texture_mailbox_struct_traits.h",
     "transferable_resource_struct_traits.cc",
     "transferable_resource_struct_traits.h",
   ]
   public_deps = [
     ":interfaces_shared_cpp_sources",
     "//base",
     "//cc",
+    "//cc/ipc:texture_mailbox_releaser_interface",
     "//cc/surfaces:surface_id",
     "//gpu/ipc/common:struct_traits",
     "//mojo/common:struct_traits",
     "//skia/public/interfaces",
     "//skia/public/interfaces:struct_traits",
     "//ui/events:events_base",
     "//ui/events/mojo:struct_traits",
     "//ui/gfx",
     "//ui/gfx:selection_bound",
     "//ui/gfx/geometry/mojo:struct_traits",
     "//ui/gfx/mojo:mojo",
     "//ui/gfx/mojo:struct_traits",
   ]
 }