New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 611802 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Jul 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug
mus

Blocking:
issue 621383



Sign in to add a comment

Get rid of cc type converters

Project Member Reported by fsam...@chromium.org, May 13 2016

Issue description

Currently, in Mus, we convert cc types back and forth including full CompositorFrames. This seems like a lot of unnecessary overhead. We should use ParamTraits or StructTraits instead.
 
It would be nice to remove the directory mojo/converters/surfaces.
Chatting with rockot@ offline, it sounds like StructTraits are the way forward, and we can place them alongside ParamTraits. Of course, cc ParamTraits live in content currently, so it seems like good housekeeping to move those traits to cc/ipc or something in the spirit of the content modularization effort.
Project Member

Comment 3 by bugdroid1@chromium.org, May 16 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/812b8489cadee4758ec6ffbe0930f96539374467

commit 812b8489cadee4758ec6ffbe0930f96539374467
Author: fsamuel <fsamuel@chromium.org>
Date: Mon May 16 18:52:18 2016

Move cc ParamTraits to cc/ipc

This CL moves the ParamTraits but doesn't yet remove the associated unit/perf
tests. I'll do that in a subsequent patch.

The end result is all cc serialization code should live in cc/ipc.

BUG= 611802 
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Review-Url: https://codereview.chromium.org/1978053002
Cr-Commit-Position: refs/heads/master@{#393884}

[add] https://crrev.com/812b8489cadee4758ec6ffbe0930f96539374467/cc/ipc/BUILD.gn
[add] https://crrev.com/812b8489cadee4758ec6ffbe0930f96539374467/cc/ipc/DEPS
[add] https://crrev.com/812b8489cadee4758ec6ffbe0930f96539374467/cc/ipc/cc_ipc.gyp
[add] https://crrev.com/812b8489cadee4758ec6ffbe0930f96539374467/cc/ipc/cc_ipc_export.h
[rename] https://crrev.com/812b8489cadee4758ec6ffbe0930f96539374467/cc/ipc/cc_param_traits.cc
[add] https://crrev.com/812b8489cadee4758ec6ffbe0930f96539374467/cc/ipc/cc_param_traits.h
[add] https://crrev.com/812b8489cadee4758ec6ffbe0930f96539374467/cc/ipc/cc_param_traits_macros.h
[modify] https://crrev.com/812b8489cadee4758ec6ffbe0930f96539374467/content/common/BUILD.gn
[delete] https://crrev.com/4711f4b1a02fbb8339c6f984049a60222e0adf3c/content/common/cc_messages.h
[modify] https://crrev.com/812b8489cadee4758ec6ffbe0930f96539374467/content/common/cc_messages_perftest.cc
[modify] https://crrev.com/812b8489cadee4758ec6ffbe0930f96539374467/content/common/cc_messages_unittest.cc
[modify] https://crrev.com/812b8489cadee4758ec6ffbe0930f96539374467/content/common/content_message_generator.h
[modify] https://crrev.com/812b8489cadee4758ec6ffbe0930f96539374467/content/common/view_messages.h
[modify] https://crrev.com/812b8489cadee4758ec6ffbe0930f96539374467/content/content_common.gypi
[modify] https://crrev.com/812b8489cadee4758ec6ffbe0930f96539374467/content/content_tests.gypi
[modify] https://crrev.com/812b8489cadee4758ec6ffbe0930f96539374467/content/renderer/BUILD.gn
[modify] https://crrev.com/812b8489cadee4758ec6ffbe0930f96539374467/content/renderer/android/synchronous_compositor_proxy.cc
[modify] https://crrev.com/812b8489cadee4758ec6ffbe0930f96539374467/content/test/BUILD.gn

Project Member

Comment 4 by bugdroid1@chromium.org, May 16 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/47cebcff36966c28ed428bd6e275635ded12030c

commit 47cebcff36966c28ed428bd6e275635ded12030c
Author: fsamuel <fsamuel@chromium.org>
Date: Mon May 16 20:50:23 2016

cc/ipc: Suppress size_t to int warning on winx64

BUG= 611802 
NOTRY=true
NOTREECHECKS=true
TBR=tsepez@chromium.org
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Review-Url: https://codereview.chromium.org/1982783002
Cr-Commit-Position: refs/heads/master@{#393908}

[modify] https://crrev.com/47cebcff36966c28ed428bd6e275635ded12030c/cc/ipc/cc_ipc.gyp

Comment 5 by jam@chromium.org, May 16 2016

I'm curious about the statement of StructTraits vs ParamTraits.

If the structures in question need to be sent from languages other than C++, then definitely we want to have the one definition in mojom files and we can use StructTraits to map to other existing C++ classes.

If the structures in question are in classes that also have useful methods but are only called by C++, that's a reason why we can't just move the definition to mojom. In that case, using ParamTraits is less work than ParamTraits (i.e. don't have to specify the structure twice).

Can you expand on which of the above cases (or a different one) applies here?
I want to get to a point where the one true versions are in mojoms. I want to use StructTraits to C++ once StructTraits are ready. The classes do indeed have useful methods however and they should move to elsewhere over time.

Comment 7 by jam@chromium.org, May 16 2016

Got it. If the end goal is to get to the one definition being in mojom, that's the best solution. StructTraits can be used as a temporary bridge until that happens.
Yea, I think the long term goal will likely be to move to mojom versions but that's probably going to take a while.
Labels: -mustash2 tadpole
Project Member

Comment 10 by bugdroid1@chromium.org, May 18 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/a6b0edf28c82f44492792801c3d4cb7afe112365

commit a6b0edf28c82f44492792801c3d4cb7afe112365
Author: fsamuel <fsamuel@chromium.org>
Date: Wed May 18 03:49:09 2016

Add security OWNERS to cc/ipc

BUG= 611802 
TBR=danakj@chromium.org
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Review-Url: https://codereview.chromium.org/1992633002
Cr-Commit-Position: refs/heads/master@{#394327}

[add] https://crrev.com/a6b0edf28c82f44492792801c3d4cb7afe112365/cc/ipc/OWNERS

Project Member

Comment 11 by bugdroid1@chromium.org, May 18 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/080ceaa9f26ab266bd28bf580af750afd973cad1

commit 080ceaa9f26ab266bd28bf580af750afd973cad1
Author: fsamuel <fsamuel@chromium.org>
Date: Wed May 18 12:50:29 2016

cc/ipc: Fix size_t to int implicit cast

BUG= 611802 
TBR=danakj@chromium.org
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Review-Url: https://codereview.chromium.org/1993563002
Cr-Commit-Position: refs/heads/master@{#394394}

[modify] https://crrev.com/080ceaa9f26ab266bd28bf580af750afd973cad1/cc/ipc/BUILD.gn
[modify] https://crrev.com/080ceaa9f26ab266bd28bf580af750afd973cad1/cc/ipc/cc_ipc.gyp
[modify] https://crrev.com/080ceaa9f26ab266bd28bf580af750afd973cad1/cc/ipc/cc_param_traits.cc

Owner: fsam...@chromium.org
Status: Assigned (was: Untriaged)
Project Member

Comment 14 by bugdroid1@chromium.org, May 27 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/d60bd68eb0201e4ac7b6d196f66c3bff7592715e

commit d60bd68eb0201e4ac7b6d196f66c3bff7592715e
Author: fsamuel <fsamuel@chromium.org>
Date: Fri May 27 04:38:42 2016

Implement cc::SurfaceId StructTraits

1. Move surface_id.mojom to cc/ipc, this matches gpu/ipc/common's placement for mojoms.

2. Write a StructTraits class for cc::SurfaceId.

3. Update usage.

BUG= 611802 
TBR=danakj@chromium.org
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Review-Url: https://codereview.chromium.org/2014233002
Cr-Commit-Position: refs/heads/master@{#396387}

[modify] https://crrev.com/d60bd68eb0201e4ac7b6d196f66c3bff7592715e/cc/ipc/BUILD.gn
[rename] https://crrev.com/d60bd68eb0201e4ac7b6d196f66c3bff7592715e/cc/ipc/surface_id.mojom
[add] https://crrev.com/d60bd68eb0201e4ac7b6d196f66c3bff7592715e/cc/ipc/surface_id.typemap
[add] https://crrev.com/d60bd68eb0201e4ac7b6d196f66c3bff7592715e/cc/ipc/surface_id_struct_traits.h
[add] https://crrev.com/d60bd68eb0201e4ac7b6d196f66c3bff7592715e/cc/ipc/typemaps.gni
[modify] https://crrev.com/d60bd68eb0201e4ac7b6d196f66c3bff7592715e/components/mus/public/cpp/surfaces/BUILD.gn
[modify] https://crrev.com/d60bd68eb0201e4ac7b6d196f66c3bff7592715e/components/mus/public/cpp/surfaces/DEPS
[modify] https://crrev.com/d60bd68eb0201e4ac7b6d196f66c3bff7592715e/components/mus/public/cpp/surfaces/surfaces_type_converters.cc
[modify] https://crrev.com/d60bd68eb0201e4ac7b6d196f66c3bff7592715e/components/mus/public/cpp/surfaces/surfaces_type_converters.h
[modify] https://crrev.com/d60bd68eb0201e4ac7b6d196f66c3bff7592715e/components/mus/public/cpp/surfaces/tests/surface_unittest.cc
[modify] https://crrev.com/d60bd68eb0201e4ac7b6d196f66c3bff7592715e/components/mus/public/cpp/window.h
[modify] https://crrev.com/d60bd68eb0201e4ac7b6d196f66c3bff7592715e/components/mus/public/interfaces/BUILD.gn
[modify] https://crrev.com/d60bd68eb0201e4ac7b6d196f66c3bff7592715e/components/mus/public/interfaces/quads.mojom
[modify] https://crrev.com/d60bd68eb0201e4ac7b6d196f66c3bff7592715e/components/mus/public/interfaces/window_tree.mojom
[modify] https://crrev.com/d60bd68eb0201e4ac7b6d196f66c3bff7592715e/components/mus/ws/BUILD.gn
[modify] https://crrev.com/d60bd68eb0201e4ac7b6d196f66c3bff7592715e/components/mus/ws/server_window_surface.cc
[modify] https://crrev.com/d60bd68eb0201e4ac7b6d196f66c3bff7592715e/components/mus/ws/window_tree.h
[modify] https://crrev.com/d60bd68eb0201e4ac7b6d196f66c3bff7592715e/mojo/public/tools/bindings/chromium_bindings_configuration.gni

Project Member

Comment 15 by bugdroid1@chromium.org, May 27 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/70eee944ac0e80daf3a0c95b46eb647bf95f6e07

commit 70eee944ac0e80daf3a0c95b46eb647bf95f6e07
Author: fsamuel <fsamuel@chromium.org>
Date: Fri May 27 20:13:45 2016

Implement RenderPassId StructTrait

BUG= 611802 
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Review-Url: https://codereview.chromium.org/2011263002
Cr-Commit-Position: refs/heads/master@{#396549}

[modify] https://crrev.com/70eee944ac0e80daf3a0c95b46eb647bf95f6e07/cc/ipc/BUILD.gn
[add] https://crrev.com/70eee944ac0e80daf3a0c95b46eb647bf95f6e07/cc/ipc/render_pass_id.mojom
[add] https://crrev.com/70eee944ac0e80daf3a0c95b46eb647bf95f6e07/cc/ipc/render_pass_id.typemap
[add] https://crrev.com/70eee944ac0e80daf3a0c95b46eb647bf95f6e07/cc/ipc/render_pass_id_struct_traits.h
[modify] https://crrev.com/70eee944ac0e80daf3a0c95b46eb647bf95f6e07/cc/ipc/typemaps.gni
[modify] https://crrev.com/70eee944ac0e80daf3a0c95b46eb647bf95f6e07/components/mus/public/cpp/surfaces/surfaces_type_converters.cc
[modify] https://crrev.com/70eee944ac0e80daf3a0c95b46eb647bf95f6e07/components/mus/public/cpp/surfaces/surfaces_type_converters.h
[modify] https://crrev.com/70eee944ac0e80daf3a0c95b46eb647bf95f6e07/components/mus/public/cpp/surfaces/surfaces_utils.cc
[modify] https://crrev.com/70eee944ac0e80daf3a0c95b46eb647bf95f6e07/components/mus/public/cpp/surfaces/tests/surface_unittest.cc
[modify] https://crrev.com/70eee944ac0e80daf3a0c95b46eb647bf95f6e07/components/mus/public/interfaces/quads.mojom

Cc: ben@chromium.org
Project Member

Comment 18 by bugdroid1@chromium.org, Jun 1 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/3de222455772a473df1585b47e3395a0c174fc38

commit 3de222455772a473df1585b47e3395a0c174fc38
Author: fsamuel <fsamuel@chromium.org>
Date: Wed Jun 01 05:37:04 2016

Implement StructTraits for various cc and gpu types

This CL implements StructTraits for the following types:

1. gpu::Mailbox
2. gpu::MailboxHolder
3. gpu::SyncToken
4. cc::ReturnedResource

This CL also implements unit tests in both gpu/ipc/common
and cc/ipc that test these traits.

BUG= 611802 
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Review-Url: https://codereview.chromium.org/2019833002
Cr-Commit-Position: refs/heads/master@{#397067}

[modify] https://crrev.com/3de222455772a473df1585b47e3395a0c174fc38/cc/ipc/BUILD.gn
[add] https://crrev.com/3de222455772a473df1585b47e3395a0c174fc38/cc/ipc/returned_resource.mojom
[add] https://crrev.com/3de222455772a473df1585b47e3395a0c174fc38/cc/ipc/returned_resource.typemap
[add] https://crrev.com/3de222455772a473df1585b47e3395a0c174fc38/cc/ipc/returned_resource_struct_traits.h
[modify] https://crrev.com/3de222455772a473df1585b47e3395a0c174fc38/cc/ipc/struct_traits_unittest.cc
[modify] https://crrev.com/3de222455772a473df1585b47e3395a0c174fc38/cc/ipc/surface_id.typemap
[modify] https://crrev.com/3de222455772a473df1585b47e3395a0c174fc38/cc/ipc/traits_test_service.mojom
[modify] https://crrev.com/3de222455772a473df1585b47e3395a0c174fc38/cc/ipc/typemaps.gni
[modify] https://crrev.com/3de222455772a473df1585b47e3395a0c174fc38/components/bitmap_uploader/bitmap_uploader.cc
[modify] https://crrev.com/3de222455772a473df1585b47e3395a0c174fc38/components/bitmap_uploader/bitmap_uploader.h
[modify] https://crrev.com/3de222455772a473df1585b47e3395a0c174fc38/components/mus/gpu/display_compositor/compositor_frame_sink_impl.cc
[modify] https://crrev.com/3de222455772a473df1585b47e3395a0c174fc38/components/mus/public/cpp/lib/output_surface.cc
[modify] https://crrev.com/3de222455772a473df1585b47e3395a0c174fc38/components/mus/public/cpp/lib/window_surface.cc
[modify] https://crrev.com/3de222455772a473df1585b47e3395a0c174fc38/components/mus/public/cpp/output_surface.h
[modify] https://crrev.com/3de222455772a473df1585b47e3395a0c174fc38/components/mus/public/cpp/surfaces/surfaces_type_converters.cc
[modify] https://crrev.com/3de222455772a473df1585b47e3395a0c174fc38/components/mus/public/cpp/surfaces/surfaces_type_converters.h
[modify] https://crrev.com/3de222455772a473df1585b47e3395a0c174fc38/components/mus/public/cpp/surfaces/tests/surface_unittest.cc
[modify] https://crrev.com/3de222455772a473df1585b47e3395a0c174fc38/components/mus/public/cpp/window_surface.h
[modify] https://crrev.com/3de222455772a473df1585b47e3395a0c174fc38/components/mus/public/cpp/window_surface_client.h
[modify] https://crrev.com/3de222455772a473df1585b47e3395a0c174fc38/components/mus/public/interfaces/compositor_frame.mojom
[modify] https://crrev.com/3de222455772a473df1585b47e3395a0c174fc38/components/mus/public/interfaces/gpu/display_compositor.mojom
[modify] https://crrev.com/3de222455772a473df1585b47e3395a0c174fc38/components/mus/ws/server_window_surface.cc
[modify] https://crrev.com/3de222455772a473df1585b47e3395a0c174fc38/gpu/BUILD.gn
[add] https://crrev.com/3de222455772a473df1585b47e3395a0c174fc38/gpu/command_buffer/common/DEPS
[modify] https://crrev.com/3de222455772a473df1585b47e3395a0c174fc38/gpu/command_buffer/common/unittest_main.cc
[modify] https://crrev.com/3de222455772a473df1585b47e3395a0c174fc38/gpu/gpu.gyp
[modify] https://crrev.com/3de222455772a473df1585b47e3395a0c174fc38/gpu/ipc/common/BUILD.gn
[modify] https://crrev.com/3de222455772a473df1585b47e3395a0c174fc38/gpu/ipc/common/DEPS
[modify] https://crrev.com/3de222455772a473df1585b47e3395a0c174fc38/gpu/ipc/common/mailbox.mojom
[modify] https://crrev.com/3de222455772a473df1585b47e3395a0c174fc38/gpu/ipc/common/mailbox.typemap
[modify] https://crrev.com/3de222455772a473df1585b47e3395a0c174fc38/gpu/ipc/common/mailbox_holder.mojom
[modify] https://crrev.com/3de222455772a473df1585b47e3395a0c174fc38/gpu/ipc/common/mailbox_holder.typemap
[add] https://crrev.com/3de222455772a473df1585b47e3395a0c174fc38/gpu/ipc/common/mailbox_holder_struct_traits.h
[add] https://crrev.com/3de222455772a473df1585b47e3395a0c174fc38/gpu/ipc/common/mailbox_struct_traits.cc
[add] https://crrev.com/3de222455772a473df1585b47e3395a0c174fc38/gpu/ipc/common/mailbox_struct_traits.h
[add] https://crrev.com/3de222455772a473df1585b47e3395a0c174fc38/gpu/ipc/common/struct_traits_unittest.cc
[modify] https://crrev.com/3de222455772a473df1585b47e3395a0c174fc38/gpu/ipc/common/sync_token.mojom
[modify] https://crrev.com/3de222455772a473df1585b47e3395a0c174fc38/gpu/ipc/common/sync_token.typemap
[add] https://crrev.com/3de222455772a473df1585b47e3395a0c174fc38/gpu/ipc/common/sync_token_struct_traits.h
[add] https://crrev.com/3de222455772a473df1585b47e3395a0c174fc38/gpu/ipc/common/traits_test_service.mojom

Project Member

Comment 19 by bugdroid1@chromium.org, Jun 1 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b4185b229021653f1000f8224244bb65b36f31bc

commit b4185b229021653f1000f8224244bb65b36f31bc
Author: fsamuel <fsamuel@chromium.org>
Date: Wed Jun 01 12:13:26 2016

Implement gfx::Transform StructTraits

This has test coverage through surfaces unittests, but I will
add a standalone test in a separate patch.

BUG= 611802 

Review-Url: https://codereview.chromium.org/2028573003
Cr-Commit-Position: refs/heads/master@{#397110}

[modify] https://crrev.com/b4185b229021653f1000f8224244bb65b36f31bc/components/mus/public/cpp/surfaces/BUILD.gn
[modify] https://crrev.com/b4185b229021653f1000f8224244bb65b36f31bc/components/mus/public/cpp/surfaces/surfaces_type_converters.cc
[modify] https://crrev.com/b4185b229021653f1000f8224244bb65b36f31bc/components/mus/public/cpp/surfaces/surfaces_utils.cc
[modify] https://crrev.com/b4185b229021653f1000f8224244bb65b36f31bc/components/mus/public/cpp/surfaces/tests/BUILD.gn
[modify] https://crrev.com/b4185b229021653f1000f8224244bb65b36f31bc/components/mus/public/cpp/surfaces/tests/surface_unittest.cc
[modify] https://crrev.com/b4185b229021653f1000f8224244bb65b36f31bc/components/mus/public/interfaces/BUILD.gn
[modify] https://crrev.com/b4185b229021653f1000f8224244bb65b36f31bc/components/mus/public/interfaces/quads.mojom
[modify] https://crrev.com/b4185b229021653f1000f8224244bb65b36f31bc/components/mus/ws/BUILD.gn
[modify] https://crrev.com/b4185b229021653f1000f8224244bb65b36f31bc/components/mus/ws/platform_display.cc
[modify] https://crrev.com/b4185b229021653f1000f8224244bb65b36f31bc/mash/app_driver/BUILD.gn
[modify] https://crrev.com/b4185b229021653f1000f8224244bb65b36f31bc/mash/wm/BUILD.gn
[modify] https://crrev.com/b4185b229021653f1000f8224244bb65b36f31bc/mojo/public/tools/bindings/mojom.gni
[modify] https://crrev.com/b4185b229021653f1000f8224244bb65b36f31bc/ui/gfx/geometry/mojo/geometry.mojom
[modify] https://crrev.com/b4185b229021653f1000f8224244bb65b36f31bc/ui/gfx/mojo/BUILD.gn
[delete] https://crrev.com/79b66d38a29dda10dc6801a5e9d5950d09abb39a/ui/gfx/mojo/DEPS
[delete] https://crrev.com/79b66d38a29dda10dc6801a5e9d5950d09abb39a/ui/gfx/mojo/mojo_transform_export.h
[add] https://crrev.com/b4185b229021653f1000f8224244bb65b36f31bc/ui/gfx/mojo/transform.mojom
[add] https://crrev.com/b4185b229021653f1000f8224244bb65b36f31bc/ui/gfx/mojo/transform.typemap
[add] https://crrev.com/b4185b229021653f1000f8224244bb65b36f31bc/ui/gfx/mojo/transform_struct_traits.h
[delete] https://crrev.com/79b66d38a29dda10dc6801a5e9d5950d09abb39a/ui/gfx/mojo/transform_type_converters.cc
[delete] https://crrev.com/79b66d38a29dda10dc6801a5e9d5950d09abb39a/ui/gfx/mojo/transform_type_converters.h
[add] https://crrev.com/b4185b229021653f1000f8224244bb65b36f31bc/ui/gfx/mojo/typemaps.gni
[modify] https://crrev.com/b4185b229021653f1000f8224244bb65b36f31bc/ui/gfx/typemaps.gni

Project Member

Comment 21 by bugdroid1@chromium.org, Jun 1 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/dfa82fb0c75da3b6ad5b13a30a87ceb51ab74d6e

commit dfa82fb0c75da3b6ad5b13a30a87ceb51ab74d6e
Author: fsamuel <fsamuel@chromium.org>
Date: Wed Jun 01 18:47:45 2016

Rename mojo.* ui/gfx/geometry/mojo types to gfx.mojom.*

BUG= 611802 
TBR=ben@chromium.org
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Review-Url: https://codereview.chromium.org/2027203002
Cr-Commit-Position: refs/heads/master@{#397193}

[modify] https://crrev.com/dfa82fb0c75da3b6ad5b13a30a87ceb51ab74d6e/cc/ipc/transferable_resource.mojom
[modify] https://crrev.com/dfa82fb0c75da3b6ad5b13a30a87ceb51ab74d6e/components/mus/public/interfaces/command_buffer.mojom
[modify] https://crrev.com/dfa82fb0c75da3b6ad5b13a30a87ceb51ab74d6e/components/mus/public/interfaces/event_matcher.mojom
[modify] https://crrev.com/dfa82fb0c75da3b6ad5b13a30a87ceb51ab74d6e/components/mus/public/interfaces/gpu_service.mojom
[modify] https://crrev.com/dfa82fb0c75da3b6ad5b13a30a87ceb51ab74d6e/components/mus/public/interfaces/quads.mojom
[modify] https://crrev.com/dfa82fb0c75da3b6ad5b13a30a87ceb51ab74d6e/components/mus/public/interfaces/window_manager.mojom
[modify] https://crrev.com/dfa82fb0c75da3b6ad5b13a30a87ceb51ab74d6e/components/mus/public/interfaces/window_manager_constants.mojom
[modify] https://crrev.com/dfa82fb0c75da3b6ad5b13a30a87ceb51ab74d6e/components/mus/public/interfaces/window_tree.mojom
[modify] https://crrev.com/dfa82fb0c75da3b6ad5b13a30a87ceb51ab74d6e/components/mus/public/interfaces/window_tree_host.mojom
[modify] https://crrev.com/dfa82fb0c75da3b6ad5b13a30a87ceb51ab74d6e/content/common/image_downloader/image_downloader.mojom
[modify] https://crrev.com/dfa82fb0c75da3b6ad5b13a30a87ceb51ab74d6e/media/mojo/interfaces/media_types.mojom
[modify] https://crrev.com/dfa82fb0c75da3b6ad5b13a30a87ceb51ab74d6e/media/mojo/interfaces/renderer.mojom
[modify] https://crrev.com/dfa82fb0c75da3b6ad5b13a30a87ceb51ab74d6e/services/navigation/public/interfaces/view.mojom
[modify] https://crrev.com/dfa82fb0c75da3b6ad5b13a30a87ceb51ab74d6e/ui/gfx/geometry/mojo/geometry.mojom
[modify] https://crrev.com/dfa82fb0c75da3b6ad5b13a30a87ceb51ab74d6e/ui/gfx/geometry/mojo/geometry.typemap
[modify] https://crrev.com/dfa82fb0c75da3b6ad5b13a30a87ceb51ab74d6e/ui/gfx/geometry/mojo/geometry_struct_traits.h
[modify] https://crrev.com/dfa82fb0c75da3b6ad5b13a30a87ceb51ab74d6e/ui/keyboard/keyboard.mojom

Project Member

Comment 22 by bugdroid1@chromium.org, Jun 1 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/633487455d1731b853e6ae52019198a5e3d1963a

commit 633487455d1731b853e6ae52019198a5e3d1963a
Author: fsamuel <fsamuel@chromium.org>
Date: Wed Jun 01 18:51:09 2016

gpu::mojom::Mailbox: Remove unnecessary ArrayTraits IsNull/SetToNull methods

BUG= 611802 
TBR=tsepez@chromium.org, piman@chromium.org
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel

Review-Url: https://codereview.chromium.org/2027223002
Cr-Commit-Position: refs/heads/master@{#397195}

[modify] https://crrev.com/633487455d1731b853e6ae52019198a5e3d1963a/gpu/ipc/common/mailbox_struct_traits.cc
[modify] https://crrev.com/633487455d1731b853e6ae52019198a5e3d1963a/gpu/ipc/common/mailbox_struct_traits.h

Project Member

Comment 24 by bugdroid1@chromium.org, Jun 2 2016

Project Member

Comment 26 by bugdroid1@chromium.org, Jun 3 2016

Project Member

Comment 27 by bugdroid1@chromium.org, Jun 3 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/181cbc09384b9175d6de86f60b97b5a9aadd9711

commit 181cbc09384b9175d6de86f60b97b5a9aadd9711
Author: fsamuel <fsamuel@chromium.org>
Date: Fri Jun 03 02:31:14 2016

Move SelectionBound from ui/base/touch to ui/gfx to be used by cc

We need to be able to serialize CompositorFrames over mojo which depend
(indirectly) on cc::ViewportSelectionBound. To avoid writing two serializers for
two very similar structs, I've addressed jdduke@'s TODO to unify
the two SelectionBounds. As a first step, this CL moves SelectionBound to a
directory that can be used by cc.

BUG= 611802 
TBR=sadrul@chromium.org, ben@chromium.org

Review-Url: https://codereview.chromium.org/2034863002
Cr-Commit-Position: refs/heads/master@{#397585}

[modify] https://crrev.com/181cbc09384b9175d6de86f60b97b5a9aadd9711/content/browser/renderer_host/input/ui_touch_selection_helper.cc
[modify] https://crrev.com/181cbc09384b9175d6de86f60b97b5a9aadd9711/content/browser/renderer_host/input/ui_touch_selection_helper.h
[modify] https://crrev.com/181cbc09384b9175d6de86f60b97b5a9aadd9711/content/browser/renderer_host/render_widget_host_view_android.cc
[modify] https://crrev.com/181cbc09384b9175d6de86f60b97b5a9aadd9711/content/browser/renderer_host/render_widget_host_view_aura.cc
[modify] https://crrev.com/181cbc09384b9175d6de86f60b97b5a9aadd9711/content/browser/renderer_host/render_widget_host_view_aura.h
[modify] https://crrev.com/181cbc09384b9175d6de86f60b97b5a9aadd9711/ui/base/BUILD.gn
[modify] https://crrev.com/181cbc09384b9175d6de86f60b97b5a9aadd9711/ui/base/touch/touch_editing_controller.h
[modify] https://crrev.com/181cbc09384b9175d6de86f60b97b5a9aadd9711/ui/base/ui_base.gyp
[modify] https://crrev.com/181cbc09384b9175d6de86f60b97b5a9aadd9711/ui/base/ui_base_tests.gyp
[modify] https://crrev.com/181cbc09384b9175d6de86f60b97b5a9aadd9711/ui/gfx/BUILD.gn
[modify] https://crrev.com/181cbc09384b9175d6de86f60b97b5a9aadd9711/ui/gfx/gfx.gyp
[modify] https://crrev.com/181cbc09384b9175d6de86f60b97b5a9aadd9711/ui/gfx/gfx_tests.gyp
[rename] https://crrev.com/181cbc09384b9175d6de86f60b97b5a9aadd9711/ui/gfx/selection_bound.cc
[rename] https://crrev.com/181cbc09384b9175d6de86f60b97b5a9aadd9711/ui/gfx/selection_bound.h
[rename] https://crrev.com/181cbc09384b9175d6de86f60b97b5a9aadd9711/ui/gfx/selection_bound_unittest.cc
[modify] https://crrev.com/181cbc09384b9175d6de86f60b97b5a9aadd9711/ui/touch_selection/BUILD.gn
[modify] https://crrev.com/181cbc09384b9175d6de86f60b97b5a9aadd9711/ui/touch_selection/touch_selection_controller.cc
[modify] https://crrev.com/181cbc09384b9175d6de86f60b97b5a9aadd9711/ui/touch_selection/touch_selection_controller.h
[modify] https://crrev.com/181cbc09384b9175d6de86f60b97b5a9aadd9711/ui/touch_selection/touch_selection_controller_unittest.cc
[modify] https://crrev.com/181cbc09384b9175d6de86f60b97b5a9aadd9711/ui/touch_selection/ui_touch_selection.gyp
[modify] https://crrev.com/181cbc09384b9175d6de86f60b97b5a9aadd9711/ui/views/controls/textfield/textfield.cc
[modify] https://crrev.com/181cbc09384b9175d6de86f60b97b5a9aadd9711/ui/views/controls/textfield/textfield.h
[modify] https://crrev.com/181cbc09384b9175d6de86f60b97b5a9aadd9711/ui/views/touchui/touch_selection_controller_impl.cc
[modify] https://crrev.com/181cbc09384b9175d6de86f60b97b5a9aadd9711/ui/views/touchui/touch_selection_controller_impl.h
[modify] https://crrev.com/181cbc09384b9175d6de86f60b97b5a9aadd9711/ui/views/touchui/touch_selection_controller_impl_unittest.cc

Project Member

Comment 30 by bugdroid1@chromium.org, Jun 3 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/4690c5bb2208c108dfd76a210cd2bec61c67c884

commit 4690c5bb2208c108dfd76a210cd2bec61c67c884
Author: fsamuel <fsamuel@chromium.org>
Date: Fri Jun 03 21:35:30 2016

Implement cc::SharedQuadState StructTraits

BUG= 611802 
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Review-Url: https://codereview.chromium.org/2032643002
Cr-Commit-Position: refs/heads/master@{#397812}

[modify] https://crrev.com/4690c5bb2208c108dfd76a210cd2bec61c67c884/cc/ipc/BUILD.gn
[add] https://crrev.com/4690c5bb2208c108dfd76a210cd2bec61c67c884/cc/ipc/shared_quad_state.mojom
[add] https://crrev.com/4690c5bb2208c108dfd76a210cd2bec61c67c884/cc/ipc/shared_quad_state.typemap
[add] https://crrev.com/4690c5bb2208c108dfd76a210cd2bec61c67c884/cc/ipc/shared_quad_state_struct_traits.h
[modify] https://crrev.com/4690c5bb2208c108dfd76a210cd2bec61c67c884/cc/ipc/struct_traits_unittest.cc
[modify] https://crrev.com/4690c5bb2208c108dfd76a210cd2bec61c67c884/cc/ipc/traits_test_service.mojom
[modify] https://crrev.com/4690c5bb2208c108dfd76a210cd2bec61c67c884/cc/ipc/typemaps.gni
[modify] https://crrev.com/4690c5bb2208c108dfd76a210cd2bec61c67c884/components/bitmap_uploader/bitmap_uploader.cc
[modify] https://crrev.com/4690c5bb2208c108dfd76a210cd2bec61c67c884/components/mus/public/cpp/surfaces/surfaces_type_converters.cc
[modify] https://crrev.com/4690c5bb2208c108dfd76a210cd2bec61c67c884/components/mus/public/cpp/surfaces/surfaces_type_converters.h
[modify] https://crrev.com/4690c5bb2208c108dfd76a210cd2bec61c67c884/components/mus/public/cpp/surfaces/surfaces_utils.cc
[modify] https://crrev.com/4690c5bb2208c108dfd76a210cd2bec61c67c884/components/mus/public/cpp/surfaces/surfaces_utils.h
[modify] https://crrev.com/4690c5bb2208c108dfd76a210cd2bec61c67c884/components/mus/public/cpp/surfaces/tests/surface_unittest.cc
[modify] https://crrev.com/4690c5bb2208c108dfd76a210cd2bec61c67c884/components/mus/public/interfaces/quads.mojom

Project Member

Comment 31 by bugdroid1@chromium.org, Jun 4 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/5ca57dfa32cb7a1d497890a166e82df86077a11d

commit 5ca57dfa32cb7a1d497890a166e82df86077a11d
Author: fsamuel <fsamuel@chromium.org>
Date: Sat Jun 04 00:34:55 2016

Replace cc::ViewportSelectionBound with ui::SelectionBound

We'd like to be able to fully serialize CompositorFrames over
mojo. CompositorFrameMetadata has a
cc::ViewportSelection(Bound), which closely resembles
gfx::SelectionBound. This CL works towards unifying these
structs to reduce code duplication.

BUG= 611802 
TBR=piman@ for content changes.
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Review-Url: https://codereview.chromium.org/2030033003
Cr-Commit-Position: refs/heads/master@{#397855}

[modify] https://crrev.com/5ca57dfa32cb7a1d497890a166e82df86077a11d/cc/BUILD.gn
[modify] https://crrev.com/5ca57dfa32cb7a1d497890a166e82df86077a11d/cc/cc.gyp
[modify] https://crrev.com/5ca57dfa32cb7a1d497890a166e82df86077a11d/cc/input/layer_selection_bound.cc
[modify] https://crrev.com/5ca57dfa32cb7a1d497890a166e82df86077a11d/cc/input/layer_selection_bound.h
[modify] https://crrev.com/5ca57dfa32cb7a1d497890a166e82df86077a11d/cc/input/layer_selection_bound_unittest.cc
[delete] https://crrev.com/e1f7c121717a5b0272e0011415eb8698a6fde1d2/cc/input/selection_bound_type.h
[modify] https://crrev.com/5ca57dfa32cb7a1d497890a166e82df86077a11d/cc/ipc/cc_param_traits.h
[modify] https://crrev.com/5ca57dfa32cb7a1d497890a166e82df86077a11d/cc/ipc/cc_param_traits_macros.h
[modify] https://crrev.com/5ca57dfa32cb7a1d497890a166e82df86077a11d/cc/output/compositor_frame_metadata.h
[delete] https://crrev.com/e1f7c121717a5b0272e0011415eb8698a6fde1d2/cc/output/viewport_selection_bound.cc
[delete] https://crrev.com/e1f7c121717a5b0272e0011415eb8698a6fde1d2/cc/output/viewport_selection_bound.h
[modify] https://crrev.com/5ca57dfa32cb7a1d497890a166e82df86077a11d/cc/trees/layer_tree_host_impl_unittest.cc
[modify] https://crrev.com/5ca57dfa32cb7a1d497890a166e82df86077a11d/cc/trees/layer_tree_impl.cc
[modify] https://crrev.com/5ca57dfa32cb7a1d497890a166e82df86077a11d/cc/trees/layer_tree_impl.h
[modify] https://crrev.com/5ca57dfa32cb7a1d497890a166e82df86077a11d/cc/trees/layer_tree_impl_unittest.cc
[modify] https://crrev.com/5ca57dfa32cb7a1d497890a166e82df86077a11d/content/browser/android/content_view_core_impl.cc
[modify] https://crrev.com/5ca57dfa32cb7a1d497890a166e82df86077a11d/content/browser/android/content_view_core_impl.h
[delete] https://crrev.com/e1f7c121717a5b0272e0011415eb8698a6fde1d2/content/browser/renderer_host/input/ui_touch_selection_helper.cc
[delete] https://crrev.com/e1f7c121717a5b0272e0011415eb8698a6fde1d2/content/browser/renderer_host/input/ui_touch_selection_helper.h
[modify] https://crrev.com/5ca57dfa32cb7a1d497890a166e82df86077a11d/content/browser/renderer_host/render_widget_host_view_android.cc
[modify] https://crrev.com/5ca57dfa32cb7a1d497890a166e82df86077a11d/content/browser/renderer_host/render_widget_host_view_android.h
[modify] https://crrev.com/5ca57dfa32cb7a1d497890a166e82df86077a11d/content/browser/renderer_host/render_widget_host_view_aura.cc
[modify] https://crrev.com/5ca57dfa32cb7a1d497890a166e82df86077a11d/content/content_browser.gypi
[modify] https://crrev.com/5ca57dfa32cb7a1d497890a166e82df86077a11d/content/renderer/gpu/render_widget_compositor.cc
[modify] https://crrev.com/5ca57dfa32cb7a1d497890a166e82df86077a11d/ui/gfx/BUILD.gn
[modify] https://crrev.com/5ca57dfa32cb7a1d497890a166e82df86077a11d/ui/gfx/ipc/BUILD.gn
[modify] https://crrev.com/5ca57dfa32cb7a1d497890a166e82df86077a11d/ui/gfx/ipc/gfx_ipc.gyp
[modify] https://crrev.com/5ca57dfa32cb7a1d497890a166e82df86077a11d/ui/gfx/ipc/gfx_param_traits.cc
[modify] https://crrev.com/5ca57dfa32cb7a1d497890a166e82df86077a11d/ui/gfx/ipc/gfx_param_traits.h
[modify] https://crrev.com/5ca57dfa32cb7a1d497890a166e82df86077a11d/ui/gfx/ipc/gfx_param_traits_macros.h

Project Member

Comment 33 by bugdroid1@chromium.org, Jun 7 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/6b3ec8379d9e19ec9a379cba96f3e40569328961

commit 6b3ec8379d9e19ec9a379cba96f3e40569328961
Author: fsamuel <fsamuel@chromium.org>
Date: Tue Jun 07 02:55:48 2016

Implement LatencyInfo StructTraits

This CL implements LatencyInfo mojom types, and associated
StructTraits for serialization and deserialization. This will be used
by cc::CompositorFrameMetadata and ultimately cc::CompositorFrame
serialization/deserialization over mojo.

BUG= 611802 

Review-Url: https://codereview.chromium.org/2040733002
Cr-Commit-Position: refs/heads/master@{#398211}

[modify] https://crrev.com/6b3ec8379d9e19ec9a379cba96f3e40569328961/mojo/common/common_custom_types.typemap
[modify] https://crrev.com/6b3ec8379d9e19ec9a379cba96f3e40569328961/ui/events/BUILD.gn
[modify] https://crrev.com/6b3ec8379d9e19ec9a379cba96f3e40569328961/ui/events/DEPS
[modify] https://crrev.com/6b3ec8379d9e19ec9a379cba96f3e40569328961/ui/events/latency_info.h
[modify] https://crrev.com/6b3ec8379d9e19ec9a379cba96f3e40569328961/ui/events/mojo/BUILD.gn
[modify] https://crrev.com/6b3ec8379d9e19ec9a379cba96f3e40569328961/ui/events/mojo/DEPS
[add] https://crrev.com/6b3ec8379d9e19ec9a379cba96f3e40569328961/ui/events/mojo/latency_info.mojom
[add] https://crrev.com/6b3ec8379d9e19ec9a379cba96f3e40569328961/ui/events/mojo/latency_info.typemap
[add] https://crrev.com/6b3ec8379d9e19ec9a379cba96f3e40569328961/ui/events/mojo/latency_info_struct_traits.cc
[add] https://crrev.com/6b3ec8379d9e19ec9a379cba96f3e40569328961/ui/events/mojo/latency_info_struct_traits.h
[add] https://crrev.com/6b3ec8379d9e19ec9a379cba96f3e40569328961/ui/events/mojo/struct_traits_unittest.cc
[add] https://crrev.com/6b3ec8379d9e19ec9a379cba96f3e40569328961/ui/events/mojo/traits_test_service.mojom
[modify] https://crrev.com/6b3ec8379d9e19ec9a379cba96f3e40569328961/ui/events/mojo/typemaps.gni

Project Member

Comment 34 by bugdroid1@chromium.org, Jun 7 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/10bec68301c42372e7b1d31a53150462baaae1d9

commit 10bec68301c42372e7b1d31a53150462baaae1d9
Author: fsamuel <fsamuel@chromium.org>
Date: Tue Jun 07 04:25:20 2016

Implement cc::CompositorFrameMetadata StructTraits

This CL replaces the simple mus CompositorFrameMetadata struct with a
full fidelity struct along with StructTraits. w00t! Soon full fidelity
CompositorFrames in Mus without extra copies!

BUG= 611802 
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Review-Url: https://codereview.chromium.org/2042643002
Cr-Commit-Position: refs/heads/master@{#398222}

[modify] https://crrev.com/10bec68301c42372e7b1d31a53150462baaae1d9/cc/ipc/BUILD.gn
[add] https://crrev.com/10bec68301c42372e7b1d31a53150462baaae1d9/cc/ipc/compositor_frame_metadata.mojom
[add] https://crrev.com/10bec68301c42372e7b1d31a53150462baaae1d9/cc/ipc/compositor_frame_metadata.typemap
[add] https://crrev.com/10bec68301c42372e7b1d31a53150462baaae1d9/cc/ipc/compositor_frame_metadata_struct_traits.h
[modify] https://crrev.com/10bec68301c42372e7b1d31a53150462baaae1d9/cc/ipc/struct_traits_unittest.cc
[modify] https://crrev.com/10bec68301c42372e7b1d31a53150462baaae1d9/cc/ipc/traits_test_service.mojom
[modify] https://crrev.com/10bec68301c42372e7b1d31a53150462baaae1d9/cc/ipc/typemaps.gni
[modify] https://crrev.com/10bec68301c42372e7b1d31a53150462baaae1d9/components/bitmap_uploader/bitmap_uploader.cc
[modify] https://crrev.com/10bec68301c42372e7b1d31a53150462baaae1d9/components/mus/public/cpp/surfaces/custom_surface_converter.h
[modify] https://crrev.com/10bec68301c42372e7b1d31a53150462baaae1d9/components/mus/public/cpp/surfaces/surfaces_type_converters.cc
[modify] https://crrev.com/10bec68301c42372e7b1d31a53150462baaae1d9/components/mus/public/cpp/surfaces/surfaces_type_converters.h
[modify] https://crrev.com/10bec68301c42372e7b1d31a53150462baaae1d9/components/mus/public/cpp/surfaces/tests/surface_unittest.cc
[modify] https://crrev.com/10bec68301c42372e7b1d31a53150462baaae1d9/components/mus/public/interfaces/compositor_frame.mojom
[modify] https://crrev.com/10bec68301c42372e7b1d31a53150462baaae1d9/components/mus/ws/server_window_surface.cc
[modify] https://crrev.com/10bec68301c42372e7b1d31a53150462baaae1d9/components/mus/ws/server_window_surface.h

Project Member

Comment 35 by bugdroid1@chromium.org, Jun 7 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/7b3867c7fe43b232dee8611280bb3f0da5a1111a

commit 7b3867c7fe43b232dee8611280bb3f0da5a1111a
Author: fsamuel <fsamuel@chromium.org>
Date: Tue Jun 07 08:13:04 2016

mojo ArrayTraits<T>::Resize returns bool indicating success

ArrayTraits<T>::Resize should return false and thus fail deserialization
if the array cannot be resized.

BUG= 611802 
TBR=ben@chromium.org
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel

Review-Url: https://codereview.chromium.org/2045723002
Cr-Commit-Position: refs/heads/master@{#398251}

[modify] https://crrev.com/7b3867c7fe43b232dee8611280bb3f0da5a1111a/gpu/ipc/common/mailbox_struct_traits.cc
[modify] https://crrev.com/7b3867c7fe43b232dee8611280bb3f0da5a1111a/gpu/ipc/common/mailbox_struct_traits.h
[modify] https://crrev.com/7b3867c7fe43b232dee8611280bb3f0da5a1111a/mojo/public/cpp/bindings/array_traits.h
[modify] https://crrev.com/7b3867c7fe43b232dee8611280bb3f0da5a1111a/mojo/public/cpp/bindings/array_traits_standard.h
[modify] https://crrev.com/7b3867c7fe43b232dee8611280bb3f0da5a1111a/mojo/public/cpp/bindings/array_traits_stl.h
[modify] https://crrev.com/7b3867c7fe43b232dee8611280bb3f0da5a1111a/mojo/public/cpp/bindings/array_traits_wtf.h
[modify] https://crrev.com/7b3867c7fe43b232dee8611280bb3f0da5a1111a/mojo/public/cpp/bindings/array_traits_wtf_vector.h
[modify] https://crrev.com/7b3867c7fe43b232dee8611280bb3f0da5a1111a/mojo/public/cpp/bindings/lib/array_serialization.h
[modify] https://crrev.com/7b3867c7fe43b232dee8611280bb3f0da5a1111a/skia/public/interfaces/bitmap_skbitmap_struct_traits.cc
[modify] https://crrev.com/7b3867c7fe43b232dee8611280bb3f0da5a1111a/skia/public/interfaces/bitmap_skbitmap_struct_traits.h
[modify] https://crrev.com/7b3867c7fe43b232dee8611280bb3f0da5a1111a/ui/events/mojo/latency_info_struct_traits.cc
[modify] https://crrev.com/7b3867c7fe43b232dee8611280bb3f0da5a1111a/ui/events/mojo/latency_info_struct_traits.h

Project Member

Comment 36 by bugdroid1@chromium.org, Jun 7 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/cc016ede169b3f1a9d7153a27eadd6d8ead52e1a

commit cc016ede169b3f1a9d7153a27eadd6d8ead52e1a
Author: fsamuel <fsamuel@chromium.org>
Date: Tue Jun 07 12:39:49 2016

Replace ui::LatencyInfo::InputCoordinate with gfx::PointF

This reduces code duplication and makes the world a happier place.

BUG= 611802 
TBR=ben@chromium.org

Review-Url: https://codereview.chromium.org/2039853004
Cr-Commit-Position: refs/heads/master@{#398278}

[modify] https://crrev.com/cc016ede169b3f1a9d7153a27eadd6d8ead52e1a/content/browser/renderer_host/input/render_widget_host_latency_tracker.cc
[modify] https://crrev.com/cc016ede169b3f1a9d7153a27eadd6d8ead52e1a/content/browser/renderer_host/input/render_widget_host_latency_tracker_unittest.cc
[modify] https://crrev.com/cc016ede169b3f1a9d7153a27eadd6d8ead52e1a/tools/ipc_fuzzer/fuzzer/fuzzer.cc
[modify] https://crrev.com/cc016ede169b3f1a9d7153a27eadd6d8ead52e1a/ui/events/BUILD.gn
[modify] https://crrev.com/cc016ede169b3f1a9d7153a27eadd6d8ead52e1a/ui/events/events.gyp
[modify] https://crrev.com/cc016ede169b3f1a9d7153a27eadd6d8ead52e1a/ui/events/events_unittests.gyp
[modify] https://crrev.com/cc016ede169b3f1a9d7153a27eadd6d8ead52e1a/ui/events/ipc/BUILD.gn
[modify] https://crrev.com/cc016ede169b3f1a9d7153a27eadd6d8ead52e1a/ui/events/ipc/latency_info_param_traits.cc
[modify] https://crrev.com/cc016ede169b3f1a9d7153a27eadd6d8ead52e1a/ui/events/ipc/latency_info_param_traits_macros.h
[modify] https://crrev.com/cc016ede169b3f1a9d7153a27eadd6d8ead52e1a/ui/events/ipc/latency_info_param_traits_unittest.cc
[modify] https://crrev.com/cc016ede169b3f1a9d7153a27eadd6d8ead52e1a/ui/events/latency_info.cc
[modify] https://crrev.com/cc016ede169b3f1a9d7153a27eadd6d8ead52e1a/ui/events/latency_info.h
[modify] https://crrev.com/cc016ede169b3f1a9d7153a27eadd6d8ead52e1a/ui/events/mojo/BUILD.gn
[modify] https://crrev.com/cc016ede169b3f1a9d7153a27eadd6d8ead52e1a/ui/events/mojo/latency_info.mojom
[modify] https://crrev.com/cc016ede169b3f1a9d7153a27eadd6d8ead52e1a/ui/events/mojo/latency_info.typemap
[modify] https://crrev.com/cc016ede169b3f1a9d7153a27eadd6d8ead52e1a/ui/events/mojo/latency_info_struct_traits.cc
[modify] https://crrev.com/cc016ede169b3f1a9d7153a27eadd6d8ead52e1a/ui/events/mojo/latency_info_struct_traits.h
[modify] https://crrev.com/cc016ede169b3f1a9d7153a27eadd6d8ead52e1a/ui/events/mojo/struct_traits_unittest.cc
[modify] https://crrev.com/cc016ede169b3f1a9d7153a27eadd6d8ead52e1a/ui/events/mojo/traits_test_service.mojom

Project Member

Comment 37 by bugdroid1@chromium.org, Jun 7 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/4a11f31975f19c063f4d42da0b5feae678188d9d

commit 4a11f31975f19c063f4d42da0b5feae678188d9d
Author: geofflang <geofflang@chromium.org>
Date: Tue Jun 07 15:43:28 2016

Revert of Replace ui::LatencyInfo::InputCoordinate with gfx::PointF (patchset #6 id:90001 of https://codereview.chromium.org/2039853004/ )

Reason for revert:
This appears to have broken the Windows GPU debug builder.  First broken build: https://build.chromium.org/p/chromium.gpu.fyi/builders/GPU%20Win%20Builder%20%28dbg%29/builds/40788

Original issue's description:
> Replace ui::LatencyInfo::InputCoordinate with gfx::PointF
>
> This reduces code duplication and makes the world a happier place.
>
> BUG= 611802 
> TBR=ben@chromium.org
>
> Committed: https://crrev.com/cc016ede169b3f1a9d7153a27eadd6d8ead52e1a
> Cr-Commit-Position: refs/heads/master@{#398278}

TBR=sadrul@chromium.org,tsepez@chromium.org,ben@chromium.org,fsamuel@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG= 611802 

Review-Url: https://codereview.chromium.org/2048673002
Cr-Commit-Position: refs/heads/master@{#398308}

[modify] https://crrev.com/4a11f31975f19c063f4d42da0b5feae678188d9d/content/browser/renderer_host/input/render_widget_host_latency_tracker.cc
[modify] https://crrev.com/4a11f31975f19c063f4d42da0b5feae678188d9d/content/browser/renderer_host/input/render_widget_host_latency_tracker_unittest.cc
[modify] https://crrev.com/4a11f31975f19c063f4d42da0b5feae678188d9d/tools/ipc_fuzzer/fuzzer/fuzzer.cc
[modify] https://crrev.com/4a11f31975f19c063f4d42da0b5feae678188d9d/ui/events/BUILD.gn
[modify] https://crrev.com/4a11f31975f19c063f4d42da0b5feae678188d9d/ui/events/events.gyp
[modify] https://crrev.com/4a11f31975f19c063f4d42da0b5feae678188d9d/ui/events/events_unittests.gyp
[modify] https://crrev.com/4a11f31975f19c063f4d42da0b5feae678188d9d/ui/events/ipc/BUILD.gn
[modify] https://crrev.com/4a11f31975f19c063f4d42da0b5feae678188d9d/ui/events/ipc/latency_info_param_traits.cc
[modify] https://crrev.com/4a11f31975f19c063f4d42da0b5feae678188d9d/ui/events/ipc/latency_info_param_traits_macros.h
[modify] https://crrev.com/4a11f31975f19c063f4d42da0b5feae678188d9d/ui/events/ipc/latency_info_param_traits_unittest.cc
[modify] https://crrev.com/4a11f31975f19c063f4d42da0b5feae678188d9d/ui/events/latency_info.cc
[modify] https://crrev.com/4a11f31975f19c063f4d42da0b5feae678188d9d/ui/events/latency_info.h
[modify] https://crrev.com/4a11f31975f19c063f4d42da0b5feae678188d9d/ui/events/mojo/BUILD.gn
[modify] https://crrev.com/4a11f31975f19c063f4d42da0b5feae678188d9d/ui/events/mojo/latency_info.mojom
[modify] https://crrev.com/4a11f31975f19c063f4d42da0b5feae678188d9d/ui/events/mojo/latency_info.typemap
[modify] https://crrev.com/4a11f31975f19c063f4d42da0b5feae678188d9d/ui/events/mojo/latency_info_struct_traits.cc
[modify] https://crrev.com/4a11f31975f19c063f4d42da0b5feae678188d9d/ui/events/mojo/latency_info_struct_traits.h
[modify] https://crrev.com/4a11f31975f19c063f4d42da0b5feae678188d9d/ui/events/mojo/struct_traits_unittest.cc
[modify] https://crrev.com/4a11f31975f19c063f4d42da0b5feae678188d9d/ui/events/mojo/traits_test_service.mojom

Project Member

Comment 38 by bugdroid1@chromium.org, Jun 7 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/ac64128a4ca6529e861813159ef5852fcc304614

commit ac64128a4ca6529e861813159ef5852fcc304614
Author: fsamuel <fsamuel@chromium.org>
Date: Tue Jun 07 17:49:27 2016

Replace ui::LatencyInfo::InputCoordinate with gfx::PointF

This reduces code duplication and makes the world a happier place.

BUG= 611802 
TBR=ben@chromium.org

Committed: https://crrev.com/cc016ede169b3f1a9d7153a27eadd6d8ead52e1a
Review-Url: https://codereview.chromium.org/2039853004
Cr-Original-Commit-Position: refs/heads/master@{#398278}
Cr-Commit-Position: refs/heads/master@{#398328}

[modify] https://crrev.com/ac64128a4ca6529e861813159ef5852fcc304614/content/browser/renderer_host/input/render_widget_host_latency_tracker.cc
[modify] https://crrev.com/ac64128a4ca6529e861813159ef5852fcc304614/content/browser/renderer_host/input/render_widget_host_latency_tracker_unittest.cc
[modify] https://crrev.com/ac64128a4ca6529e861813159ef5852fcc304614/tools/ipc_fuzzer/fuzzer/fuzzer.cc
[modify] https://crrev.com/ac64128a4ca6529e861813159ef5852fcc304614/ui/events/BUILD.gn
[modify] https://crrev.com/ac64128a4ca6529e861813159ef5852fcc304614/ui/events/events.gyp
[modify] https://crrev.com/ac64128a4ca6529e861813159ef5852fcc304614/ui/events/events_unittests.gyp
[modify] https://crrev.com/ac64128a4ca6529e861813159ef5852fcc304614/ui/events/ipc/BUILD.gn
[modify] https://crrev.com/ac64128a4ca6529e861813159ef5852fcc304614/ui/events/ipc/latency_info_param_traits.cc
[modify] https://crrev.com/ac64128a4ca6529e861813159ef5852fcc304614/ui/events/ipc/latency_info_param_traits_macros.h
[modify] https://crrev.com/ac64128a4ca6529e861813159ef5852fcc304614/ui/events/ipc/latency_info_param_traits_unittest.cc
[modify] https://crrev.com/ac64128a4ca6529e861813159ef5852fcc304614/ui/events/latency_info.cc
[modify] https://crrev.com/ac64128a4ca6529e861813159ef5852fcc304614/ui/events/latency_info.h
[modify] https://crrev.com/ac64128a4ca6529e861813159ef5852fcc304614/ui/events/mojo/BUILD.gn
[modify] https://crrev.com/ac64128a4ca6529e861813159ef5852fcc304614/ui/events/mojo/latency_info.mojom
[modify] https://crrev.com/ac64128a4ca6529e861813159ef5852fcc304614/ui/events/mojo/latency_info.typemap
[modify] https://crrev.com/ac64128a4ca6529e861813159ef5852fcc304614/ui/events/mojo/latency_info_struct_traits.cc
[modify] https://crrev.com/ac64128a4ca6529e861813159ef5852fcc304614/ui/events/mojo/latency_info_struct_traits.h
[modify] https://crrev.com/ac64128a4ca6529e861813159ef5852fcc304614/ui/events/mojo/struct_traits_unittest.cc
[modify] https://crrev.com/ac64128a4ca6529e861813159ef5852fcc304614/ui/events/mojo/traits_test_service.mojom

Project Member

Comment 39 by bugdroid1@chromium.org, Jun 7 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/567de3bea75fb0006ab1ca74be7f394473444ce5

commit 567de3bea75fb0006ab1ca74be7f394473444ce5
Author: fsamuel <fsamuel@chromium.org>
Date: Tue Jun 07 21:34:55 2016

Implement sk_sp<SkImageFilter> StructTraits

RenderPassDrawQuads need to be able to ship cc::FilterOperations which use
sk_sp<SkImageFilter>. This CL makes it possible to serialize and deserialize
them over mojo.

BUG= 611802 

Review-Url: https://codereview.chromium.org/2040233002
Cr-Commit-Position: refs/heads/master@{#398379}

[modify] https://crrev.com/567de3bea75fb0006ab1ca74be7f394473444ce5/skia/BUILD.gn
[modify] https://crrev.com/567de3bea75fb0006ab1ca74be7f394473444ce5/skia/public/interfaces/BUILD.gn
[add] https://crrev.com/567de3bea75fb0006ab1ca74be7f394473444ce5/skia/public/interfaces/image_filter.mojom
[add] https://crrev.com/567de3bea75fb0006ab1ca74be7f394473444ce5/skia/public/interfaces/image_filter.typemap
[add] https://crrev.com/567de3bea75fb0006ab1ca74be7f394473444ce5/skia/public/interfaces/image_filter_struct_traits.cc
[add] https://crrev.com/567de3bea75fb0006ab1ca74be7f394473444ce5/skia/public/interfaces/image_filter_struct_traits.h
[add] https://crrev.com/567de3bea75fb0006ab1ca74be7f394473444ce5/skia/public/interfaces/test/struct_traits_unittest.cc
[add] https://crrev.com/567de3bea75fb0006ab1ca74be7f394473444ce5/skia/public/interfaces/test/traits_test_service.mojom
[modify] https://crrev.com/567de3bea75fb0006ab1ca74be7f394473444ce5/skia/public/interfaces/typemaps.gni

Project Member

Comment 40 by bugdroid1@chromium.org, Jun 8 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/ea3c906a94710cbc68483e9c5d5cee88f5b1569a

commit ea3c906a94710cbc68483e9c5d5cee88f5b1569a
Author: fsamuel <fsamuel@chromium.org>
Date: Wed Jun 08 19:15:54 2016

Add ArrayTraits<CArray<T>> utility template

This CL adds a utility ArrayTraits class to read and write from and into a c array from StructTraits.

This CL also deletes duplicate code across various traits.

BUG= 611802 
TBR=ben@chromium.org, tsepez@chromium.org
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel

Review-Url: https://codereview.chromium.org/2046563005
Cr-Commit-Position: refs/heads/master@{#398634}

[modify] https://crrev.com/ea3c906a94710cbc68483e9c5d5cee88f5b1569a/gpu/ipc/common/mailbox_struct_traits.cc
[modify] https://crrev.com/ea3c906a94710cbc68483e9c5d5cee88f5b1569a/gpu/ipc/common/mailbox_struct_traits.h
[modify] https://crrev.com/ea3c906a94710cbc68483e9c5d5cee88f5b1569a/mojo/mojo_public.gyp
[modify] https://crrev.com/ea3c906a94710cbc68483e9c5d5cee88f5b1569a/mojo/public/cpp/bindings/BUILD.gn
[add] https://crrev.com/ea3c906a94710cbc68483e9c5d5cee88f5b1569a/mojo/public/cpp/bindings/array_traits_carray.h
[modify] https://crrev.com/ea3c906a94710cbc68483e9c5d5cee88f5b1569a/mojo/public/cpp/bindings/lib/serialization.h
[modify] https://crrev.com/ea3c906a94710cbc68483e9c5d5cee88f5b1569a/skia/public/interfaces/bitmap_skbitmap_struct_traits.cc
[modify] https://crrev.com/ea3c906a94710cbc68483e9c5d5cee88f5b1569a/skia/public/interfaces/bitmap_skbitmap_struct_traits.h
[modify] https://crrev.com/ea3c906a94710cbc68483e9c5d5cee88f5b1569a/ui/events/mojo/latency_info_struct_traits.cc
[modify] https://crrev.com/ea3c906a94710cbc68483e9c5d5cee88f5b1569a/ui/events/mojo/latency_info_struct_traits.h

Project Member

Comment 42 by bugdroid1@chromium.org, Jun 8 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/a9ace74447c40544805e5c6a4f785dd1ef8d49a5

commit a9ace74447c40544805e5c6a4f785dd1ef8d49a5
Author: fsamuel <fsamuel@chromium.org>
Date: Wed Jun 08 21:53:44 2016

Implement cc::FilterOperation StructTraits

FilterOperation is used by RenderPassDrawQuad whcih is used by CompostiorFrame.
This CL allows FilterOperation to be serialized over mojo.

BUG= 611802 
TBR=enne@chromium.org
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Review-Url: https://codereview.chromium.org/2042413003
Cr-Commit-Position: refs/heads/master@{#398692}

[modify] https://crrev.com/a9ace74447c40544805e5c6a4f785dd1ef8d49a5/cc/ipc/BUILD.gn
[add] https://crrev.com/a9ace74447c40544805e5c6a4f785dd1ef8d49a5/cc/ipc/filter_operation.mojom
[add] https://crrev.com/a9ace74447c40544805e5c6a4f785dd1ef8d49a5/cc/ipc/filter_operation.typemap
[add] https://crrev.com/a9ace74447c40544805e5c6a4f785dd1ef8d49a5/cc/ipc/filter_operation_struct_traits.h
[modify] https://crrev.com/a9ace74447c40544805e5c6a4f785dd1ef8d49a5/cc/ipc/struct_traits_unittest.cc
[modify] https://crrev.com/a9ace74447c40544805e5c6a4f785dd1ef8d49a5/cc/ipc/traits_test_service.mojom
[modify] https://crrev.com/a9ace74447c40544805e5c6a4f785dd1ef8d49a5/cc/ipc/typemaps.gni
[modify] https://crrev.com/a9ace74447c40544805e5c6a4f785dd1ef8d49a5/cc/output/filter_operation.cc
[modify] https://crrev.com/a9ace74447c40544805e5c6a4f785dd1ef8d49a5/cc/output/filter_operation.h
[modify] https://crrev.com/a9ace74447c40544805e5c6a4f785dd1ef8d49a5/skia/public/interfaces/image_filter_struct_traits.h
[modify] https://crrev.com/a9ace74447c40544805e5c6a4f785dd1ef8d49a5/skia/public/interfaces/test/struct_traits_unittest.cc

Project Member

Comment 43 by bugdroid1@chromium.org, Jun 9 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/4af5eca9361ef3e4253d78e1655f895e5e290214

commit 4af5eca9361ef3e4253d78e1655f895e5e290214
Author: fsamuel <fsamuel@chromium.org>
Date: Thu Jun 09 01:32:45 2016

Implement cc::FilterOperations StructTraits

BUG= 611802 
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Review-Url: https://codereview.chromium.org/2047573005
Cr-Commit-Position: refs/heads/master@{#398759}

[modify] https://crrev.com/4af5eca9361ef3e4253d78e1655f895e5e290214/cc/ipc/BUILD.gn
[add] https://crrev.com/4af5eca9361ef3e4253d78e1655f895e5e290214/cc/ipc/filter_operations.mojom
[add] https://crrev.com/4af5eca9361ef3e4253d78e1655f895e5e290214/cc/ipc/filter_operations.typemap
[add] https://crrev.com/4af5eca9361ef3e4253d78e1655f895e5e290214/cc/ipc/filter_operations_struct_traits.h
[modify] https://crrev.com/4af5eca9361ef3e4253d78e1655f895e5e290214/cc/ipc/struct_traits_unittest.cc
[modify] https://crrev.com/4af5eca9361ef3e4253d78e1655f895e5e290214/cc/ipc/traits_test_service.mojom
[modify] https://crrev.com/4af5eca9361ef3e4253d78e1655f895e5e290214/cc/ipc/typemaps.gni
[modify] https://crrev.com/4af5eca9361ef3e4253d78e1655f895e5e290214/cc/output/filter_operations.cc
[modify] https://crrev.com/4af5eca9361ef3e4253d78e1655f895e5e290214/cc/output/filter_operations.h
[modify] https://crrev.com/4af5eca9361ef3e4253d78e1655f895e5e290214/components/mus/public/cpp/surfaces/surfaces_type_converters.cc
[modify] https://crrev.com/4af5eca9361ef3e4253d78e1655f895e5e290214/components/mus/public/interfaces/quads.mojom

Project Member

Comment 44 by bugdroid1@chromium.org, Jun 9 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/10bd63df0dbbc74da797c7d919d8c5461e947ff5

commit 10bd63df0dbbc74da797c7d919d8c5461e947ff5
Author: fsamuel <fsamuel@chromium.org>
Date: Thu Jun 09 04:56:18 2016

Move quads.mojom from Mus to cc

This CL moves quads.mojom closer to the types they will map to along with
other cc mojoms. This CL also updates the module name to cc.mojom, and updates
Pass => RenderPass, Quad => DrawQuad to better match cc naming.

BUG= 611802 
TBR=ben@chromium.org
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Review-Url: https://codereview.chromium.org/2056483002
Cr-Commit-Position: refs/heads/master@{#398788}

[modify] https://crrev.com/10bd63df0dbbc74da797c7d919d8c5461e947ff5/cc/ipc/BUILD.gn
[rename] https://crrev.com/10bd63df0dbbc74da797c7d919d8c5461e947ff5/cc/ipc/quads.mojom
[modify] https://crrev.com/10bd63df0dbbc74da797c7d919d8c5461e947ff5/components/bitmap_uploader/bitmap_uploader.cc
[modify] https://crrev.com/10bd63df0dbbc74da797c7d919d8c5461e947ff5/components/mus/public/cpp/surfaces/custom_surface_converter.h
[modify] https://crrev.com/10bd63df0dbbc74da797c7d919d8c5461e947ff5/components/mus/public/cpp/surfaces/surfaces_type_converters.cc
[modify] https://crrev.com/10bd63df0dbbc74da797c7d919d8c5461e947ff5/components/mus/public/cpp/surfaces/surfaces_type_converters.h
[modify] https://crrev.com/10bd63df0dbbc74da797c7d919d8c5461e947ff5/components/mus/public/cpp/surfaces/surfaces_utils.cc
[modify] https://crrev.com/10bd63df0dbbc74da797c7d919d8c5461e947ff5/components/mus/public/cpp/surfaces/surfaces_utils.h
[modify] https://crrev.com/10bd63df0dbbc74da797c7d919d8c5461e947ff5/components/mus/public/cpp/surfaces/tests/surface_unittest.cc
[modify] https://crrev.com/10bd63df0dbbc74da797c7d919d8c5461e947ff5/components/mus/public/interfaces/BUILD.gn
[modify] https://crrev.com/10bd63df0dbbc74da797c7d919d8c5461e947ff5/components/mus/public/interfaces/compositor_frame.mojom
[modify] https://crrev.com/10bd63df0dbbc74da797c7d919d8c5461e947ff5/components/mus/ws/platform_display.cc
[modify] https://crrev.com/10bd63df0dbbc74da797c7d919d8c5461e947ff5/components/mus/ws/server_window_surface.cc
[modify] https://crrev.com/10bd63df0dbbc74da797c7d919d8c5461e947ff5/components/mus/ws/server_window_surface.h

Project Member

Comment 45 by bugdroid1@chromium.org, Jun 9 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/a28a58be2d6a0d5aa8e1cc19b97d9a9502ecc462

commit a28a58be2d6a0d5aa8e1cc19b97d9a9502ecc462
Author: fsamuel <fsamuel@chromium.org>
Date: Thu Jun 09 13:46:34 2016

Remove dead code: mojom CheckerboardQuadState

It doesn't look like CheckerboardQuadState is used anymore so this CL
removes it.

BUG= 611802 
TBR=tsepez@chromium.org
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Review-Url: https://codereview.chromium.org/2056703002
Cr-Commit-Position: refs/heads/master@{#398864}

[modify] https://crrev.com/a28a58be2d6a0d5aa8e1cc19b97d9a9502ecc462/cc/ipc/quads.mojom

Project Member

Comment 46 by bugdroid1@chromium.org, Jun 13 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/09ab6968c2bb80dc2ea64a56fbd4a60b632af4ca

commit 09ab6968c2bb80dc2ea64a56fbd4a60b632af4ca
Author: fsamuel <fsamuel@chromium.org>
Date: Mon Jun 13 17:56:53 2016

mojo::ArrayTraits: Add Support for Iterators

This CL adds support for iterating over a data structure to serialize over
mojo as an array wire format.

This CL also modifies changes to LatencyInfo StructTraits that avoids
extra copies of LatencyMap on serialization.

Note that this solution doesn't fully address deserialization though.
You cannot directly assign an std::pair to a map iterator, for example.
The solution is to introduce an ArrayTraits::Assign. I'll do that in a
separate CL as this CL has already gotten very complex.

BUG= 611802 
TBR=ben@chromium.org

Review-Url: https://codereview.chromium.org/2058633002
Cr-Commit-Position: refs/heads/master@{#399479}

[modify] https://crrev.com/09ab6968c2bb80dc2ea64a56fbd4a60b632af4ca/mojo/public/cpp/bindings/array_traits.h
[modify] https://crrev.com/09ab6968c2bb80dc2ea64a56fbd4a60b632af4ca/mojo/public/cpp/bindings/lib/array_serialization.h
[modify] https://crrev.com/09ab6968c2bb80dc2ea64a56fbd4a60b632af4ca/mojo/public/cpp/bindings/lib/serialization_util.h
[modify] https://crrev.com/09ab6968c2bb80dc2ea64a56fbd4a60b632af4ca/ui/events/mojo/latency_info_struct_traits.cc
[modify] https://crrev.com/09ab6968c2bb80dc2ea64a56fbd4a60b632af4ca/ui/events/mojo/latency_info_struct_traits.h

Project Member

Comment 47 by bugdroid1@chromium.org, Jun 13 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/4767e6f7f9e2e54033215d8952cc5931acea2a20

commit 4767e6f7f9e2e54033215d8952cc5931acea2a20
Author: fsamuel <fsamuel@chromium.org>
Date: Mon Jun 13 20:52:36 2016

Mus: Delete unused Mandoline Surfaces code

In Mandoline, we converted SurfaceIds in SurfaceDrawQuads from submitted
CompositorFrames to global Window IDs to embed other mus::Window.
This was useful for out of process iframes so that the embedder could
securely synchronously control position, transform and clipping of the frame.
This has been obviated by unguessable SurfaceIds. In an effort to remove
surfaces type converters, I'm deleting this dead code.

BUG= 611802 
TBR=ben@chromium.org

Review-Url: https://codereview.chromium.org/2065573003
Cr-Commit-Position: refs/heads/master@{#399530}

[modify] https://crrev.com/4767e6f7f9e2e54033215d8952cc5931acea2a20/components/mus/gpu/display_compositor/compositor_frame_sink_impl.cc
[modify] https://crrev.com/4767e6f7f9e2e54033215d8952cc5931acea2a20/components/mus/public/cpp/surfaces/BUILD.gn
[delete] https://crrev.com/6ac615dfd11e86b1ce7766e1de263ce3d2e3b727/components/mus/public/cpp/surfaces/custom_surface_converter.h
[modify] https://crrev.com/4767e6f7f9e2e54033215d8952cc5931acea2a20/components/mus/public/cpp/surfaces/surfaces_type_converters.cc
[modify] https://crrev.com/4767e6f7f9e2e54033215d8952cc5931acea2a20/components/mus/public/cpp/surfaces/surfaces_type_converters.h
[modify] https://crrev.com/4767e6f7f9e2e54033215d8952cc5931acea2a20/components/mus/ws/platform_display.cc
[modify] https://crrev.com/4767e6f7f9e2e54033215d8952cc5931acea2a20/components/mus/ws/server_window_delegate.h
[modify] https://crrev.com/4767e6f7f9e2e54033215d8952cc5931acea2a20/components/mus/ws/server_window_surface.cc
[modify] https://crrev.com/4767e6f7f9e2e54033215d8952cc5931acea2a20/components/mus/ws/server_window_surface.h
[modify] https://crrev.com/4767e6f7f9e2e54033215d8952cc5931acea2a20/components/mus/ws/test_server_window_delegate.cc
[modify] https://crrev.com/4767e6f7f9e2e54033215d8952cc5931acea2a20/components/mus/ws/test_server_window_delegate.h
[modify] https://crrev.com/4767e6f7f9e2e54033215d8952cc5931acea2a20/components/mus/ws/window_server.cc
[modify] https://crrev.com/4767e6f7f9e2e54033215d8952cc5931acea2a20/components/mus/ws/window_server.h

Project Member

Comment 48 by bugdroid1@chromium.org, Jun 15 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/a28a58be2d6a0d5aa8e1cc19b97d9a9502ecc462

commit a28a58be2d6a0d5aa8e1cc19b97d9a9502ecc462
Author: fsamuel <fsamuel@chromium.org>
Date: Thu Jun 09 13:46:34 2016

Remove dead code: mojom CheckerboardQuadState

It doesn't look like CheckerboardQuadState is used anymore so this CL
removes it.

BUG= 611802 
TBR=tsepez@chromium.org
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Review-Url: https://codereview.chromium.org/2056703002
Cr-Commit-Position: refs/heads/master@{#398864}

[modify] https://crrev.com/a28a58be2d6a0d5aa8e1cc19b97d9a9502ecc462/cc/ipc/quads.mojom

Project Member

Comment 49 by bugdroid1@chromium.org, Jun 15 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/09ab6968c2bb80dc2ea64a56fbd4a60b632af4ca

commit 09ab6968c2bb80dc2ea64a56fbd4a60b632af4ca
Author: fsamuel <fsamuel@chromium.org>
Date: Mon Jun 13 17:56:53 2016

mojo::ArrayTraits: Add Support for Iterators

This CL adds support for iterating over a data structure to serialize over
mojo as an array wire format.

This CL also modifies changes to LatencyInfo StructTraits that avoids
extra copies of LatencyMap on serialization.

Note that this solution doesn't fully address deserialization though.
You cannot directly assign an std::pair to a map iterator, for example.
The solution is to introduce an ArrayTraits::Assign. I'll do that in a
separate CL as this CL has already gotten very complex.

BUG= 611802 
TBR=ben@chromium.org

Review-Url: https://codereview.chromium.org/2058633002
Cr-Commit-Position: refs/heads/master@{#399479}

[modify] https://crrev.com/09ab6968c2bb80dc2ea64a56fbd4a60b632af4ca/mojo/public/cpp/bindings/array_traits.h
[modify] https://crrev.com/09ab6968c2bb80dc2ea64a56fbd4a60b632af4ca/mojo/public/cpp/bindings/lib/array_serialization.h
[modify] https://crrev.com/09ab6968c2bb80dc2ea64a56fbd4a60b632af4ca/mojo/public/cpp/bindings/lib/serialization_util.h
[modify] https://crrev.com/09ab6968c2bb80dc2ea64a56fbd4a60b632af4ca/ui/events/mojo/latency_info_struct_traits.cc
[modify] https://crrev.com/09ab6968c2bb80dc2ea64a56fbd4a60b632af4ca/ui/events/mojo/latency_info_struct_traits.h

Project Member

Comment 50 by bugdroid1@chromium.org, Jun 15 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/4767e6f7f9e2e54033215d8952cc5931acea2a20

commit 4767e6f7f9e2e54033215d8952cc5931acea2a20
Author: fsamuel <fsamuel@chromium.org>
Date: Mon Jun 13 20:52:36 2016

Mus: Delete unused Mandoline Surfaces code

In Mandoline, we converted SurfaceIds in SurfaceDrawQuads from submitted
CompositorFrames to global Window IDs to embed other mus::Window.
This was useful for out of process iframes so that the embedder could
securely synchronously control position, transform and clipping of the frame.
This has been obviated by unguessable SurfaceIds. In an effort to remove
surfaces type converters, I'm deleting this dead code.

BUG= 611802 
TBR=ben@chromium.org

Review-Url: https://codereview.chromium.org/2065573003
Cr-Commit-Position: refs/heads/master@{#399530}

[modify] https://crrev.com/4767e6f7f9e2e54033215d8952cc5931acea2a20/components/mus/gpu/display_compositor/compositor_frame_sink_impl.cc
[modify] https://crrev.com/4767e6f7f9e2e54033215d8952cc5931acea2a20/components/mus/public/cpp/surfaces/BUILD.gn
[delete] https://crrev.com/6ac615dfd11e86b1ce7766e1de263ce3d2e3b727/components/mus/public/cpp/surfaces/custom_surface_converter.h
[modify] https://crrev.com/4767e6f7f9e2e54033215d8952cc5931acea2a20/components/mus/public/cpp/surfaces/surfaces_type_converters.cc
[modify] https://crrev.com/4767e6f7f9e2e54033215d8952cc5931acea2a20/components/mus/public/cpp/surfaces/surfaces_type_converters.h
[modify] https://crrev.com/4767e6f7f9e2e54033215d8952cc5931acea2a20/components/mus/ws/platform_display.cc
[modify] https://crrev.com/4767e6f7f9e2e54033215d8952cc5931acea2a20/components/mus/ws/server_window_delegate.h
[modify] https://crrev.com/4767e6f7f9e2e54033215d8952cc5931acea2a20/components/mus/ws/server_window_surface.cc
[modify] https://crrev.com/4767e6f7f9e2e54033215d8952cc5931acea2a20/components/mus/ws/server_window_surface.h
[modify] https://crrev.com/4767e6f7f9e2e54033215d8952cc5931acea2a20/components/mus/ws/test_server_window_delegate.cc
[modify] https://crrev.com/4767e6f7f9e2e54033215d8952cc5931acea2a20/components/mus/ws/test_server_window_delegate.h
[modify] https://crrev.com/4767e6f7f9e2e54033215d8952cc5931acea2a20/components/mus/ws/window_server.cc
[modify] https://crrev.com/4767e6f7f9e2e54033215d8952cc5931acea2a20/components/mus/ws/window_server.h

Project Member

Comment 51 by bugdroid1@chromium.org, Jun 17 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/6ea4b2c26e6f2d2a021b10c965c3d06adb493898

commit 6ea4b2c26e6f2d2a021b10c965c3d06adb493898
Author: fsamuel <fsamuel@chromium.org>
Date: Fri Jun 17 03:39:41 2016

Implement DrawQuad StructTraits

This CL partially implements DrawQuad and QuadList StructTraits.

The code is a bit clumsy currently for a number of reasons:

1. QuadList does not provide O(1) random access and so we use an
iterator.
2. DrawQuads are created inline within a QuadList, and so we must
create them prior to invoking the ArrayTraits.

BUG= 611802 
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Review-Url: https://codereview.chromium.org/2064833002
Cr-Commit-Position: refs/heads/master@{#400345}

[modify] https://crrev.com/6ea4b2c26e6f2d2a021b10c965c3d06adb493898/cc/base/list_container.h
[modify] https://crrev.com/6ea4b2c26e6f2d2a021b10c965c3d06adb493898/cc/ipc/DEPS
[modify] https://crrev.com/6ea4b2c26e6f2d2a021b10c965c3d06adb493898/cc/ipc/filter_operation_struct_traits.h
[modify] https://crrev.com/6ea4b2c26e6f2d2a021b10c965c3d06adb493898/cc/ipc/quads.mojom
[add] https://crrev.com/6ea4b2c26e6f2d2a021b10c965c3d06adb493898/cc/ipc/quads.typemap
[add] https://crrev.com/6ea4b2c26e6f2d2a021b10c965c3d06adb493898/cc/ipc/quads_struct_traits.cc
[add] https://crrev.com/6ea4b2c26e6f2d2a021b10c965c3d06adb493898/cc/ipc/quads_struct_traits.h
[modify] https://crrev.com/6ea4b2c26e6f2d2a021b10c965c3d06adb493898/cc/ipc/struct_traits_unittest.cc
[modify] https://crrev.com/6ea4b2c26e6f2d2a021b10c965c3d06adb493898/cc/ipc/traits_test_service.mojom
[modify] https://crrev.com/6ea4b2c26e6f2d2a021b10c965c3d06adb493898/cc/ipc/typemaps.gni
[modify] https://crrev.com/6ea4b2c26e6f2d2a021b10c965c3d06adb493898/cc/quads/render_pass.cc
[modify] https://crrev.com/6ea4b2c26e6f2d2a021b10c965c3d06adb493898/cc/quads/render_pass.h
[modify] https://crrev.com/6ea4b2c26e6f2d2a021b10c965c3d06adb493898/cc/quads/render_pass_draw_quad.h
[modify] https://crrev.com/6ea4b2c26e6f2d2a021b10c965c3d06adb493898/cc/quads/texture_draw_quad.h
[modify] https://crrev.com/6ea4b2c26e6f2d2a021b10c965c3d06adb493898/cc/quads/tile_draw_quad.h
[modify] https://crrev.com/6ea4b2c26e6f2d2a021b10c965c3d06adb493898/components/bitmap_uploader/bitmap_uploader.cc
[modify] https://crrev.com/6ea4b2c26e6f2d2a021b10c965c3d06adb493898/components/mus/public/cpp/surfaces/surfaces_type_converters.cc
[modify] https://crrev.com/6ea4b2c26e6f2d2a021b10c965c3d06adb493898/components/mus/public/cpp/surfaces/surfaces_type_converters.h
[modify] https://crrev.com/6ea4b2c26e6f2d2a021b10c965c3d06adb493898/components/mus/public/cpp/surfaces/tests/surface_unittest.cc

Comment 53 by sa...@chromium.org, Jun 20 2016

Blocking: 621383
Project Member

Comment 54 by bugdroid1@chromium.org, Jun 20 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/abbce962323ac5660a9622674ef862d8f0530fe4

commit abbce962323ac5660a9622674ef862d8f0530fe4
Author: fsamuel <fsamuel@chromium.org>
Date: Mon Jun 20 16:16:28 2016

Implement SharedQuadStateList StructTraits

This serializes and deserializes SharedQuadStateList via an iterator as
ListContainer does not support O(1) random access.

BUG= 611802 
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Review-Url: https://codereview.chromium.org/2075853004
Cr-Commit-Position: refs/heads/master@{#400695}

[modify] https://crrev.com/abbce962323ac5660a9622674ef862d8f0530fe4/cc/ipc/quads.mojom
[modify] https://crrev.com/abbce962323ac5660a9622674ef862d8f0530fe4/cc/ipc/quads.typemap
[modify] https://crrev.com/abbce962323ac5660a9622674ef862d8f0530fe4/cc/ipc/shared_quad_state.mojom
[modify] https://crrev.com/abbce962323ac5660a9622674ef862d8f0530fe4/cc/ipc/shared_quad_state_struct_traits.h
[modify] https://crrev.com/abbce962323ac5660a9622674ef862d8f0530fe4/components/bitmap_uploader/bitmap_uploader.cc
[modify] https://crrev.com/abbce962323ac5660a9622674ef862d8f0530fe4/components/mus/public/cpp/surfaces/surfaces_type_converters.cc
[modify] https://crrev.com/abbce962323ac5660a9622674ef862d8f0530fe4/components/mus/public/cpp/surfaces/surfaces_utils.cc
[modify] https://crrev.com/abbce962323ac5660a9622674ef862d8f0530fe4/components/mus/public/cpp/surfaces/surfaces_utils.h
[modify] https://crrev.com/abbce962323ac5660a9622674ef862d8f0530fe4/components/mus/public/cpp/surfaces/tests/surface_unittest.cc

Project Member

Comment 55 by bugdroid1@chromium.org, Jun 21 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/caa652befe567c179c2a501aed75ba86202a5343

commit caa652befe567c179c2a501aed75ba86202a5343
Author: fsamuel <fsamuel@chromium.org>
Date: Tue Jun 21 11:47:41 2016

cc/ipc: Move CompositorFrame mojoms

This CL moves some mojom structs around for readability in anticipation of being
able to serialize CompositorFrames over mojo.

1. RenderPass moves out of quads.mojom as quads_struct_traits was growing very
   rapidly. RenderPass StructTraits will come in a subsequent CL.

2. compositor_frame.mojom moved to cc/ipc with the rest of the cc mojoms

3. Remaining components/mus/public/interfaces/compositor_frame.mojom moved to
   surface.mojom to clarify functionality.

BUG= 611802 
TBR=ben@chromium.org
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Review-Url: https://codereview.chromium.org/2080383002
Cr-Commit-Position: refs/heads/master@{#400956}

[modify] https://crrev.com/caa652befe567c179c2a501aed75ba86202a5343/cc/ipc/BUILD.gn
[add] https://crrev.com/caa652befe567c179c2a501aed75ba86202a5343/cc/ipc/compositor_frame.mojom
[modify] https://crrev.com/caa652befe567c179c2a501aed75ba86202a5343/cc/ipc/quads.mojom
[add] https://crrev.com/caa652befe567c179c2a501aed75ba86202a5343/cc/ipc/render_pass.mojom
[modify] https://crrev.com/caa652befe567c179c2a501aed75ba86202a5343/components/bitmap_uploader/DEPS
[modify] https://crrev.com/caa652befe567c179c2a501aed75ba86202a5343/components/bitmap_uploader/bitmap_uploader.cc
[modify] https://crrev.com/caa652befe567c179c2a501aed75ba86202a5343/components/bitmap_uploader/bitmap_uploader.h
[modify] https://crrev.com/caa652befe567c179c2a501aed75ba86202a5343/components/mus/gpu/display_compositor/compositor_frame_sink_impl.cc
[modify] https://crrev.com/caa652befe567c179c2a501aed75ba86202a5343/components/mus/gpu/display_compositor/compositor_frame_sink_impl.h
[modify] https://crrev.com/caa652befe567c179c2a501aed75ba86202a5343/components/mus/public/cpp/lib/output_surface.cc
[modify] https://crrev.com/caa652befe567c179c2a501aed75ba86202a5343/components/mus/public/cpp/lib/window_surface.cc
[modify] https://crrev.com/caa652befe567c179c2a501aed75ba86202a5343/components/mus/public/cpp/surfaces/OWNERS
[modify] https://crrev.com/caa652befe567c179c2a501aed75ba86202a5343/components/mus/public/cpp/surfaces/surfaces_type_converters.cc
[modify] https://crrev.com/caa652befe567c179c2a501aed75ba86202a5343/components/mus/public/cpp/surfaces/surfaces_type_converters.h
[modify] https://crrev.com/caa652befe567c179c2a501aed75ba86202a5343/components/mus/public/cpp/surfaces/surfaces_utils.h
[modify] https://crrev.com/caa652befe567c179c2a501aed75ba86202a5343/components/mus/public/cpp/surfaces/tests/surface_unittest.cc
[modify] https://crrev.com/caa652befe567c179c2a501aed75ba86202a5343/components/mus/public/cpp/window_surface.h
[modify] https://crrev.com/caa652befe567c179c2a501aed75ba86202a5343/components/mus/public/interfaces/BUILD.gn
[delete] https://crrev.com/8e535c0f176e9202da7bcc3dcf7ea8ce2203898e/components/mus/public/interfaces/compositor_frame.mojom
[add] https://crrev.com/caa652befe567c179c2a501aed75ba86202a5343/components/mus/public/interfaces/gpu/OWNERS
[modify] https://crrev.com/caa652befe567c179c2a501aed75ba86202a5343/components/mus/public/interfaces/gpu/display_compositor.mojom
[add] https://crrev.com/caa652befe567c179c2a501aed75ba86202a5343/components/mus/public/interfaces/surface.mojom
[modify] https://crrev.com/caa652befe567c179c2a501aed75ba86202a5343/components/mus/public/interfaces/window_tree.mojom
[modify] https://crrev.com/caa652befe567c179c2a501aed75ba86202a5343/components/mus/ws/server_window.h
[modify] https://crrev.com/caa652befe567c179c2a501aed75ba86202a5343/components/mus/ws/server_window_surface.cc
[modify] https://crrev.com/caa652befe567c179c2a501aed75ba86202a5343/components/mus/ws/server_window_surface.h
[modify] https://crrev.com/caa652befe567c179c2a501aed75ba86202a5343/components/mus/ws/server_window_surface_manager.h
[modify] https://crrev.com/caa652befe567c179c2a501aed75ba86202a5343/content/renderer/mus/render_widget_mus_connection.cc

Project Member

Comment 56 by bugdroid1@chromium.org, Jun 24 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/ff18d775f07c5c2f8002dfe563f081427045a187

commit ff18d775f07c5c2f8002dfe563f081427045a187
Author: fsamuel <fsamuel@chromium.org>
Date: Fri Jun 24 14:44:29 2016

Implement RenderPass StructTraits

This CL implements RenderPass StructTraits. Most of
surfaces_type_converters goes away now. There's still extra
copies due to lack of context propagation and array traits
data views. There's also an extra copy in
surfaces_type_converters that will go away in a subsequent
CL. bitmap_uploader has been updated to speak to cc
directly.

This CL also deletes the test executable mojo_surfaces_lib_unittests.

BUG= 611802 
TBR=ben@chromium.org
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Review-Url: https://codereview.chromium.org/2088603002
Cr-Commit-Position: refs/heads/master@{#401861}

[modify] https://crrev.com/ff18d775f07c5c2f8002dfe563f081427045a187/cc/ipc/quads.mojom
[modify] https://crrev.com/ff18d775f07c5c2f8002dfe563f081427045a187/cc/ipc/quads_struct_traits.cc
[modify] https://crrev.com/ff18d775f07c5c2f8002dfe563f081427045a187/cc/ipc/quads_struct_traits.h
[modify] https://crrev.com/ff18d775f07c5c2f8002dfe563f081427045a187/cc/ipc/render_pass.mojom
[add] https://crrev.com/ff18d775f07c5c2f8002dfe563f081427045a187/cc/ipc/render_pass.typemap
[add] https://crrev.com/ff18d775f07c5c2f8002dfe563f081427045a187/cc/ipc/render_pass_struct_traits.cc
[add] https://crrev.com/ff18d775f07c5c2f8002dfe563f081427045a187/cc/ipc/render_pass_struct_traits.h
[modify] https://crrev.com/ff18d775f07c5c2f8002dfe563f081427045a187/cc/ipc/shared_quad_state_struct_traits.h
[modify] https://crrev.com/ff18d775f07c5c2f8002dfe563f081427045a187/cc/ipc/struct_traits_unittest.cc
[modify] https://crrev.com/ff18d775f07c5c2f8002dfe563f081427045a187/cc/ipc/traits_test_service.mojom
[modify] https://crrev.com/ff18d775f07c5c2f8002dfe563f081427045a187/cc/ipc/typemaps.gni
[modify] https://crrev.com/ff18d775f07c5c2f8002dfe563f081427045a187/components/bitmap_uploader/DEPS
[modify] https://crrev.com/ff18d775f07c5c2f8002dfe563f081427045a187/components/bitmap_uploader/bitmap_uploader.cc
[modify] https://crrev.com/ff18d775f07c5c2f8002dfe563f081427045a187/components/mus/public/cpp/surfaces/BUILD.gn
[modify] https://crrev.com/ff18d775f07c5c2f8002dfe563f081427045a187/components/mus/public/cpp/surfaces/surfaces_type_converters.cc
[modify] https://crrev.com/ff18d775f07c5c2f8002dfe563f081427045a187/components/mus/public/cpp/surfaces/surfaces_type_converters.h
[delete] https://crrev.com/9d49a280262a26c8ce517322b97fe796a4ed3204/components/mus/public/cpp/surfaces/surfaces_utils.cc
[delete] https://crrev.com/9d49a280262a26c8ce517322b97fe796a4ed3204/components/mus/public/cpp/surfaces/surfaces_utils.h
[delete] https://crrev.com/9d49a280262a26c8ce517322b97fe796a4ed3204/components/mus/public/cpp/surfaces/tests/BUILD.gn
[delete] https://crrev.com/9d49a280262a26c8ce517322b97fe796a4ed3204/components/mus/public/cpp/surfaces/tests/surface_unittest.cc
[modify] https://crrev.com/ff18d775f07c5c2f8002dfe563f081427045a187/components/mus/ws/platform_display.cc
[modify] https://crrev.com/ff18d775f07c5c2f8002dfe563f081427045a187/content/renderer/mus/render_widget_mus_connection.cc
[modify] https://crrev.com/ff18d775f07c5c2f8002dfe563f081427045a187/mojo/BUILD.gn
[modify] https://crrev.com/ff18d775f07c5c2f8002dfe563f081427045a187/testing/buildbot/chromium.android.json
[modify] https://crrev.com/ff18d775f07c5c2f8002dfe563f081427045a187/testing/buildbot/chromium.linux.json
[modify] https://crrev.com/ff18d775f07c5c2f8002dfe563f081427045a187/testing/buildbot/chromium.win.json
[modify] https://crrev.com/ff18d775f07c5c2f8002dfe563f081427045a187/testing/buildbot/gn_isolate_map.pyl

Project Member

Comment 57 by bugdroid1@chromium.org, Jun 24 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/d63137acbe0976b05cbda68db72b466755310f5a

commit d63137acbe0976b05cbda68db72b466755310f5a
Author: fsamuel <fsamuel@chromium.org>
Date: Fri Jun 24 23:39:51 2016

Make cc::CompositorFrames movable [Part 1 of 2]

There was some odd code where we created CompositorFrames on the stack
and passed them around by raw pointers, and then on the other other side
of the IPC boundary we wrapped the CompositorFrame in a unique_ptr. Odder
still, there are cases where we create a copy of the CompositorFrame just
to pass it to a SurfaceFactory or elsewhere.

This CL cleans up the insanity in preparation of shipping
CompositorFrames over mojo. CompositorFrames are now
movable. SurfaceFactory still uses a unique_ptr and so we
create a new CompositorFrame, and move into that to pass to
SurfaceFactory in this CL. In a subsequent CL,
SurfaceFactory holds a CompositorFrame by value.

BUG= 611802 
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Review-Url: https://codereview.chromium.org/2096493002
Cr-Commit-Position: refs/heads/master@{#402014}

[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/android_webview/browser/parent_output_surface.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/android_webview/browser/parent_output_surface.h
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/blimp/client/feature/compositor/blimp_output_surface.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/blimp/client/feature/compositor/blimp_output_surface.h
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/cc/layers/surface_layer_unittest.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/cc/layers/texture_layer_unittest.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/cc/output/compositor_frame.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/cc/output/compositor_frame.h
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/cc/output/compositor_frame_metadata.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/cc/output/compositor_frame_metadata.h
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/cc/output/delegating_renderer.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/cc/output/delegating_renderer.h
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/cc/output/delegating_renderer_unittest.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/cc/output/gl_renderer.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/cc/output/gl_renderer.h
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/cc/output/gl_renderer_unittest.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/cc/output/output_surface.h
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/cc/output/output_surface_unittest.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/cc/output/overlay_unittest.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/cc/output/renderer.h
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/cc/output/renderer_unittest.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/cc/output/software_renderer.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/cc/output/software_renderer.h
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/cc/surfaces/display.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/cc/surfaces/display_unittest.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/cc/surfaces/surface_display_output_surface.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/cc/surfaces/surface_display_output_surface.h
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/cc/surfaces/surface_display_output_surface_unittest.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/cc/surfaces/surface_hittest_unittest.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/cc/test/fake_output_surface.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/cc/test/fake_output_surface.h
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/cc/test/pixel_test_delegating_output_surface.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/cc/test/pixel_test_delegating_output_surface.h
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/cc/test/pixel_test_output_surface.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/cc/test/pixel_test_output_surface.h
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/cc/test/surface_hittest_test_helpers.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/cc/test/surface_hittest_test_helpers.h
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/cc/trees/layer_tree_host_impl.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/cc/trees/layer_tree_host_impl_unittest.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/cc/trees/layer_tree_host_unittest.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/components/mus/public/cpp/lib/output_surface.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/components/mus/public/cpp/output_surface.h
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/components/mus/public/cpp/surfaces/surfaces_type_converters.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/components/mus/surfaces/direct_output_surface.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/components/mus/surfaces/direct_output_surface.h
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/components/mus/surfaces/direct_output_surface_ozone.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/components/mus/surfaces/direct_output_surface_ozone.h
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/content/browser/android/synchronous_compositor_host.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/content/browser/android/synchronous_compositor_host.h
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/content/browser/compositor/gpu_browser_compositor_output_surface.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/content/browser/compositor/gpu_browser_compositor_output_surface.h
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/content/browser/compositor/gpu_output_surface_mac.h
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/content/browser/compositor/gpu_output_surface_mac.mm
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/content/browser/compositor/gpu_surfaceless_browser_compositor_output_surface.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/content/browser/compositor/gpu_surfaceless_browser_compositor_output_surface.h
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/content/browser/compositor/offscreen_browser_compositor_output_surface.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/content/browser/compositor/offscreen_browser_compositor_output_surface.h
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/content/browser/compositor/reflector_impl_unittest.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/content/browser/compositor/software_browser_compositor_output_surface.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/content/browser/compositor/software_browser_compositor_output_surface.h
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/content/browser/compositor/software_browser_compositor_output_surface_unittest.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/content/browser/devtools/devtools_frame_trace_recorder.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/content/browser/devtools/protocol/page_handler.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/content/browser/devtools/protocol/page_handler.h
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/content/browser/devtools/render_frame_devtools_agent_host.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/content/browser/devtools/render_frame_devtools_agent_host.h
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/content/browser/frame_host/render_widget_host_view_child_frame.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/content/browser/frame_host/render_widget_host_view_child_frame.h
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/content/browser/frame_host/render_widget_host_view_child_frame_unittest.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/content/browser/frame_host/render_widget_host_view_guest.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/content/browser/frame_host/render_widget_host_view_guest.h
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/content/browser/frame_host/render_widget_host_view_guest_unittest.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/content/browser/renderer_host/compositor_impl_android.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/content/browser/renderer_host/delegated_frame_host.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/content/browser/renderer_host/delegated_frame_host.h
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/content/browser/renderer_host/render_widget_host_impl.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/content/browser/renderer_host/render_widget_host_view_android.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/content/browser/renderer_host/render_widget_host_view_android.h
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/content/browser/renderer_host/render_widget_host_view_aura.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/content/browser/renderer_host/render_widget_host_view_aura.h
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/content/browser/renderer_host/render_widget_host_view_aura_unittest.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/content/browser/renderer_host/render_widget_host_view_base.h
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/content/browser/renderer_host/render_widget_host_view_mac.h
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/content/browser/renderer_host/render_widget_host_view_mac.mm
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/content/public/test/browser_test_utils.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/content/renderer/android/synchronous_compositor_output_surface.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/content/renderer/android/synchronous_compositor_output_surface.h
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/content/renderer/android/synchronous_compositor_proxy.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/content/renderer/android/synchronous_compositor_proxy.h
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/content/renderer/gpu/compositor_output_surface.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/content/renderer/gpu/compositor_output_surface.h
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/content/test/mailbox_output_surface.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/content/test/mailbox_output_surface.h
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/content/test/test_render_view_host.cc
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/content/test/test_render_view_host.h
[modify] https://crrev.com/d63137acbe0976b05cbda68db72b466755310f5a/ui/compositor/test/in_process_context_factory.cc

Project Member

Comment 58 by bugdroid1@chromium.org, Jun 27 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/ce098fb67017c4afe44b6c23664723a0a057b737

commit ce098fb67017c4afe44b6c23664723a0a057b737
Author: fsamuel <fsamuel@chromium.org>
Date: Mon Jun 27 16:43:06 2016

Implement CompositorFrame StructTraits

This CL implements CompositorFrame StructTraits. This CL
deletes surfaces type converters, and updates call sites.

In a subsequent CL, I'll add a perf test comparable to
ParamTraits.

CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
TBR=ben@chromium.org for content/renderer/mus
BUG= 611802 

Review-Url: https://codereview.chromium.org/2092843002
Cr-Commit-Position: refs/heads/master@{#402199}

[modify] https://crrev.com/ce098fb67017c4afe44b6c23664723a0a057b737/cc/ipc/DEPS
[add] https://crrev.com/ce098fb67017c4afe44b6c23664723a0a057b737/cc/ipc/compositor_frame.typemap
[modify] https://crrev.com/ce098fb67017c4afe44b6c23664723a0a057b737/cc/ipc/compositor_frame_metadata.typemap
[add] https://crrev.com/ce098fb67017c4afe44b6c23664723a0a057b737/cc/ipc/compositor_frame_metadata_struct_traits.cc
[modify] https://crrev.com/ce098fb67017c4afe44b6c23664723a0a057b737/cc/ipc/compositor_frame_metadata_struct_traits.h
[add] https://crrev.com/ce098fb67017c4afe44b6c23664723a0a057b737/cc/ipc/compositor_frame_struct_traits.cc
[add] https://crrev.com/ce098fb67017c4afe44b6c23664723a0a057b737/cc/ipc/compositor_frame_struct_traits.h
[modify] https://crrev.com/ce098fb67017c4afe44b6c23664723a0a057b737/cc/ipc/struct_traits_unittest.cc
[modify] https://crrev.com/ce098fb67017c4afe44b6c23664723a0a057b737/cc/ipc/traits_test_service.mojom
[modify] https://crrev.com/ce098fb67017c4afe44b6c23664723a0a057b737/cc/ipc/transferable_resource.typemap
[add] https://crrev.com/ce098fb67017c4afe44b6c23664723a0a057b737/cc/ipc/transferable_resource_struct_traits.cc
[modify] https://crrev.com/ce098fb67017c4afe44b6c23664723a0a057b737/cc/ipc/transferable_resource_struct_traits.h
[modify] https://crrev.com/ce098fb67017c4afe44b6c23664723a0a057b737/cc/ipc/typemaps.gni
[modify] https://crrev.com/ce098fb67017c4afe44b6c23664723a0a057b737/components/bitmap_uploader/BUILD.gn
[modify] https://crrev.com/ce098fb67017c4afe44b6c23664723a0a057b737/components/bitmap_uploader/bitmap_uploader.cc
[modify] https://crrev.com/ce098fb67017c4afe44b6c23664723a0a057b737/components/mus/gpu/display_compositor/BUILD.gn
[modify] https://crrev.com/ce098fb67017c4afe44b6c23664723a0a057b737/components/mus/gpu/display_compositor/compositor_frame_sink_impl.cc
[modify] https://crrev.com/ce098fb67017c4afe44b6c23664723a0a057b737/components/mus/gpu/display_compositor/compositor_frame_sink_impl.h
[modify] https://crrev.com/ce098fb67017c4afe44b6c23664723a0a057b737/components/mus/public/cpp/BUILD.gn
[modify] https://crrev.com/ce098fb67017c4afe44b6c23664723a0a057b737/components/mus/public/cpp/lib/output_surface.cc
[modify] https://crrev.com/ce098fb67017c4afe44b6c23664723a0a057b737/components/mus/public/cpp/lib/window_surface.cc
[delete] https://crrev.com/b16eadaef85aa0128bf4f74fb96270f0a2479b78/components/mus/public/cpp/surfaces/BUILD.gn
[delete] https://crrev.com/b16eadaef85aa0128bf4f74fb96270f0a2479b78/components/mus/public/cpp/surfaces/DEPS
[delete] https://crrev.com/b16eadaef85aa0128bf4f74fb96270f0a2479b78/components/mus/public/cpp/surfaces/OWNERS
[delete] https://crrev.com/b16eadaef85aa0128bf4f74fb96270f0a2479b78/components/mus/public/cpp/surfaces/mojo_surfaces_export.h
[delete] https://crrev.com/b16eadaef85aa0128bf4f74fb96270f0a2479b78/components/mus/public/cpp/surfaces/surfaces_type_converters.cc
[delete] https://crrev.com/b16eadaef85aa0128bf4f74fb96270f0a2479b78/components/mus/public/cpp/surfaces/surfaces_type_converters.h
[modify] https://crrev.com/ce098fb67017c4afe44b6c23664723a0a057b737/components/mus/public/cpp/window_surface.h
[modify] https://crrev.com/ce098fb67017c4afe44b6c23664723a0a057b737/components/mus/surfaces/BUILD.gn
[modify] https://crrev.com/ce098fb67017c4afe44b6c23664723a0a057b737/components/mus/ws/BUILD.gn
[modify] https://crrev.com/ce098fb67017c4afe44b6c23664723a0a057b737/components/mus/ws/platform_display.cc
[modify] https://crrev.com/ce098fb67017c4afe44b6c23664723a0a057b737/components/mus/ws/server_window_surface.cc
[modify] https://crrev.com/ce098fb67017c4afe44b6c23664723a0a057b737/components/mus/ws/server_window_surface.h
[modify] https://crrev.com/ce098fb67017c4afe44b6c23664723a0a057b737/components/mus/ws/window_server.cc
[modify] https://crrev.com/ce098fb67017c4afe44b6c23664723a0a057b737/components/mus/ws/window_tree.cc
[modify] https://crrev.com/ce098fb67017c4afe44b6c23664723a0a057b737/content/renderer/mus/BUILD.gn
[modify] https://crrev.com/ce098fb67017c4afe44b6c23664723a0a057b737/ui/views/mus/BUILD.gn
[modify] https://crrev.com/ce098fb67017c4afe44b6c23664723a0a057b737/ui/views/mus/surface_binding.cc

Project Member

Comment 60 by bugdroid1@chromium.org, Jun 28 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/2d9cc5abb4745280a002822a2380e925f835ef51

commit 2d9cc5abb4745280a002822a2380e925f835ef51
Author: fsamuel <fsamuel@chromium.org>
Date: Tue Jun 28 21:22:11 2016

Make cc::CompositorFrames movable [Part 2 of 2]

This CL is part 2 of 2 for making cc::CompositorFrames
movable. This CL primarily tackles making cc::Surface
hold a cc::CompositorFrame by value. This impacts tons of
callsites which are updated in this patch.

https://codereview.chromium.org/2096493002/

BUG= 611802 
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Review-Url: https://codereview.chromium.org/2098953003
Cr-Commit-Position: refs/heads/master@{#402545}

[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/android_webview/browser/hardware_renderer.cc
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/cc/surfaces/display.cc
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/cc/surfaces/display_unittest.cc
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/cc/surfaces/surface.cc
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/cc/surfaces/surface.h
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/cc/surfaces/surface_aggregator.cc
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/cc/surfaces/surface_aggregator.h
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/cc/surfaces/surface_aggregator_perftest.cc
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/cc/surfaces/surface_aggregator_unittest.cc
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/cc/surfaces/surface_display_output_surface.cc
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/cc/surfaces/surface_factory.cc
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/cc/surfaces/surface_factory.h
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/cc/surfaces/surface_factory_unittest.cc
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/cc/surfaces/surface_hittest.cc
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/cc/surfaces/surface_hittest_unittest.cc
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/cc/surfaces/surfaces_pixeltest.cc
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/cc/test/pixel_test_delegating_output_surface.cc
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/components/exo/surface.cc
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/components/mus/gpu/display_compositor/compositor_frame_sink_impl.cc
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/components/mus/surfaces/display_compositor.cc
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/components/mus/surfaces/display_compositor.h
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/components/mus/ws/platform_display.cc
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/components/mus/ws/platform_display.h
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/components/mus/ws/server_window_surface.cc
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/content/browser/frame_host/render_widget_host_view_child_frame.cc
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/content/browser/frame_host/render_widget_host_view_guest.cc
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/content/browser/renderer_host/delegated_frame_host.cc
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/content/browser/renderer_host/render_widget_host_view_android.cc

Project Member

Comment 61 by bugdroid1@chromium.org, Jun 28 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/2d9cc5abb4745280a002822a2380e925f835ef51

commit 2d9cc5abb4745280a002822a2380e925f835ef51
Author: fsamuel <fsamuel@chromium.org>
Date: Tue Jun 28 21:22:11 2016

Make cc::CompositorFrames movable [Part 2 of 2]

This CL is part 2 of 2 for making cc::CompositorFrames
movable. This CL primarily tackles making cc::Surface
hold a cc::CompositorFrame by value. This impacts tons of
callsites which are updated in this patch.

https://codereview.chromium.org/2096493002/

BUG= 611802 
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Review-Url: https://codereview.chromium.org/2098953003
Cr-Commit-Position: refs/heads/master@{#402545}

[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/android_webview/browser/hardware_renderer.cc
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/cc/surfaces/display.cc
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/cc/surfaces/display_unittest.cc
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/cc/surfaces/surface.cc
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/cc/surfaces/surface.h
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/cc/surfaces/surface_aggregator.cc
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/cc/surfaces/surface_aggregator.h
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/cc/surfaces/surface_aggregator_perftest.cc
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/cc/surfaces/surface_aggregator_unittest.cc
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/cc/surfaces/surface_display_output_surface.cc
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/cc/surfaces/surface_factory.cc
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/cc/surfaces/surface_factory.h
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/cc/surfaces/surface_factory_unittest.cc
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/cc/surfaces/surface_hittest.cc
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/cc/surfaces/surface_hittest_unittest.cc
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/cc/surfaces/surfaces_pixeltest.cc
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/cc/test/pixel_test_delegating_output_surface.cc
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/components/exo/surface.cc
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/components/mus/gpu/display_compositor/compositor_frame_sink_impl.cc
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/components/mus/surfaces/display_compositor.cc
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/components/mus/surfaces/display_compositor.h
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/components/mus/ws/platform_display.cc
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/components/mus/ws/platform_display.h
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/components/mus/ws/server_window_surface.cc
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/content/browser/frame_host/render_widget_host_view_child_frame.cc
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/content/browser/frame_host/render_widget_host_view_guest.cc
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/content/browser/renderer_host/delegated_frame_host.cc
[modify] https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51/content/browser/renderer_host/render_widget_host_view_android.cc

Project Member

Comment 62 by bugdroid1@chromium.org, Jun 28 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/03b9d1bd39dd290230d5b6a0860fa0ec4e0401bf

commit 03b9d1bd39dd290230d5b6a0860fa0ec4e0401bf
Author: petewil <petewil@chromium.org>
Date: Tue Jun 28 22:05:52 2016

Revert of Make cc::CompositorFrames movable [Part 2 of 2] (patchset #4 id:60001 of https://codereview.chromium.org/2098953003/ )

Reason for revert:
It looks like this might have broken the tree compile: https://build.chromium.org/p/chromium.chromiumos/builders/Linux%20ChromiumOS%20GN/builds/41520

Reverting as Sheriff

Original issue's description:
> Make cc::CompositorFrames movable [Part 2 of 2]
>
> This CL is part 2 of 2 for making cc::CompositorFrames
> movable. This CL primarily tackles making cc::Surface
> hold a cc::CompositorFrame by value. This impacts tons of
> callsites which are updated in this patch.
>
> https://codereview.chromium.org/2096493002/
>
> BUG= 611802 
> CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
>
> Committed: https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51
> Cr-Commit-Position: refs/heads/master@{#402545}

TBR=reveman@chromium.org,boliu@chromium.org,danakj@chromium.org,piman@chromium.org,sky@chromium.org,fsamuel@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG= 611802 

Review-Url: https://codereview.chromium.org/2103333002
Cr-Commit-Position: refs/heads/master@{#402561}

[modify] https://crrev.com/03b9d1bd39dd290230d5b6a0860fa0ec4e0401bf/android_webview/browser/hardware_renderer.cc
[modify] https://crrev.com/03b9d1bd39dd290230d5b6a0860fa0ec4e0401bf/cc/surfaces/display.cc
[modify] https://crrev.com/03b9d1bd39dd290230d5b6a0860fa0ec4e0401bf/cc/surfaces/display_unittest.cc
[modify] https://crrev.com/03b9d1bd39dd290230d5b6a0860fa0ec4e0401bf/cc/surfaces/surface.cc
[modify] https://crrev.com/03b9d1bd39dd290230d5b6a0860fa0ec4e0401bf/cc/surfaces/surface.h
[modify] https://crrev.com/03b9d1bd39dd290230d5b6a0860fa0ec4e0401bf/cc/surfaces/surface_aggregator.cc
[modify] https://crrev.com/03b9d1bd39dd290230d5b6a0860fa0ec4e0401bf/cc/surfaces/surface_aggregator.h
[modify] https://crrev.com/03b9d1bd39dd290230d5b6a0860fa0ec4e0401bf/cc/surfaces/surface_aggregator_perftest.cc
[modify] https://crrev.com/03b9d1bd39dd290230d5b6a0860fa0ec4e0401bf/cc/surfaces/surface_aggregator_unittest.cc
[modify] https://crrev.com/03b9d1bd39dd290230d5b6a0860fa0ec4e0401bf/cc/surfaces/surface_display_output_surface.cc
[modify] https://crrev.com/03b9d1bd39dd290230d5b6a0860fa0ec4e0401bf/cc/surfaces/surface_factory.cc
[modify] https://crrev.com/03b9d1bd39dd290230d5b6a0860fa0ec4e0401bf/cc/surfaces/surface_factory.h
[modify] https://crrev.com/03b9d1bd39dd290230d5b6a0860fa0ec4e0401bf/cc/surfaces/surface_factory_unittest.cc
[modify] https://crrev.com/03b9d1bd39dd290230d5b6a0860fa0ec4e0401bf/cc/surfaces/surface_hittest.cc
[modify] https://crrev.com/03b9d1bd39dd290230d5b6a0860fa0ec4e0401bf/cc/surfaces/surface_hittest_unittest.cc
[modify] https://crrev.com/03b9d1bd39dd290230d5b6a0860fa0ec4e0401bf/cc/surfaces/surfaces_pixeltest.cc
[modify] https://crrev.com/03b9d1bd39dd290230d5b6a0860fa0ec4e0401bf/cc/test/pixel_test_delegating_output_surface.cc
[modify] https://crrev.com/03b9d1bd39dd290230d5b6a0860fa0ec4e0401bf/components/exo/surface.cc
[modify] https://crrev.com/03b9d1bd39dd290230d5b6a0860fa0ec4e0401bf/components/mus/gpu/display_compositor/compositor_frame_sink_impl.cc
[modify] https://crrev.com/03b9d1bd39dd290230d5b6a0860fa0ec4e0401bf/components/mus/surfaces/display_compositor.cc
[modify] https://crrev.com/03b9d1bd39dd290230d5b6a0860fa0ec4e0401bf/components/mus/surfaces/display_compositor.h
[modify] https://crrev.com/03b9d1bd39dd290230d5b6a0860fa0ec4e0401bf/components/mus/ws/platform_display.cc
[modify] https://crrev.com/03b9d1bd39dd290230d5b6a0860fa0ec4e0401bf/components/mus/ws/platform_display.h
[modify] https://crrev.com/03b9d1bd39dd290230d5b6a0860fa0ec4e0401bf/components/mus/ws/server_window_surface.cc
[modify] https://crrev.com/03b9d1bd39dd290230d5b6a0860fa0ec4e0401bf/content/browser/frame_host/render_widget_host_view_child_frame.cc
[modify] https://crrev.com/03b9d1bd39dd290230d5b6a0860fa0ec4e0401bf/content/browser/frame_host/render_widget_host_view_guest.cc
[modify] https://crrev.com/03b9d1bd39dd290230d5b6a0860fa0ec4e0401bf/content/browser/renderer_host/delegated_frame_host.cc
[modify] https://crrev.com/03b9d1bd39dd290230d5b6a0860fa0ec4e0401bf/content/browser/renderer_host/render_widget_host_view_android.cc

Project Member

Comment 63 by bugdroid1@chromium.org, Jun 28 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/e9fab4366d0270c7879f6e40ef73233626c9904d

commit e9fab4366d0270c7879f6e40ef73233626c9904d
Author: fsamuel <fsamuel@chromium.org>
Date: Tue Jun 28 23:17:02 2016

Remove DelegatedFrameData::device_scale_factor

Device scale factor appeared TWICE inside a CompositorFrame:
  1. CompositorFrameMetadata
  2. DelegatedFrameData

The version in DelegatedFrameData seemed largely dead except for some
Mus code so I've updated Mus code.

BUG= 611802 
TBR=dpranke@chromium.org
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Review-Url: https://codereview.chromium.org/2101623002
Cr-Commit-Position: refs/heads/master@{#402590}

[modify] https://crrev.com/e9fab4366d0270c7879f6e40ef73233626c9904d/android_webview/browser/hardware_renderer.cc
[modify] https://crrev.com/e9fab4366d0270c7879f6e40ef73233626c9904d/cc/ipc/cc_param_traits.cc
[modify] https://crrev.com/e9fab4366d0270c7879f6e40ef73233626c9904d/cc/output/delegated_frame_data.cc
[modify] https://crrev.com/e9fab4366d0270c7879f6e40ef73233626c9904d/cc/output/delegated_frame_data.h
[modify] https://crrev.com/e9fab4366d0270c7879f6e40ef73233626c9904d/cc/output/delegating_renderer.cc
[modify] https://crrev.com/e9fab4366d0270c7879f6e40ef73233626c9904d/components/mus/ws/platform_display.cc
[modify] https://crrev.com/e9fab4366d0270c7879f6e40ef73233626c9904d/content/browser/renderer_host/render_widget_host_view_android.cc
[modify] https://crrev.com/e9fab4366d0270c7879f6e40ef73233626c9904d/tools/ipc_fuzzer/fuzzer/fuzzer.cc

Project Member

Comment 64 by bugdroid1@chromium.org, Jun 29 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/5222b6cb6ea4cb34ce34a6b8572bdd6cf081fa7f

commit 5222b6cb6ea4cb34ce34a6b8572bdd6cf081fa7f
Author: fsamuel <fsamuel@chromium.org>
Date: Wed Jun 29 00:36:51 2016

Make cc::CompositorFrames movable [Part 2 of 2]

This CL is part 2 of 2 for making cc::CompositorFrames
movable. This CL primarily tackles making cc::Surface
hold a cc::CompositorFrame by value. This impacts tons of
callsites which are updated in this patch.

https://codereview.chromium.org/2096493002/

BUG= 611802 
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/2d9cc5abb4745280a002822a2380e925f835ef51
Review-Url: https://codereview.chromium.org/2098953003
Cr-Original-Commit-Position: refs/heads/master@{#402545}
Cr-Commit-Position: refs/heads/master@{#402615}

[modify] https://crrev.com/5222b6cb6ea4cb34ce34a6b8572bdd6cf081fa7f/android_webview/browser/hardware_renderer.cc
[modify] https://crrev.com/5222b6cb6ea4cb34ce34a6b8572bdd6cf081fa7f/cc/surfaces/display.cc
[modify] https://crrev.com/5222b6cb6ea4cb34ce34a6b8572bdd6cf081fa7f/cc/surfaces/display_unittest.cc
[modify] https://crrev.com/5222b6cb6ea4cb34ce34a6b8572bdd6cf081fa7f/cc/surfaces/surface.cc
[modify] https://crrev.com/5222b6cb6ea4cb34ce34a6b8572bdd6cf081fa7f/cc/surfaces/surface.h
[modify] https://crrev.com/5222b6cb6ea4cb34ce34a6b8572bdd6cf081fa7f/cc/surfaces/surface_aggregator.cc
[modify] https://crrev.com/5222b6cb6ea4cb34ce34a6b8572bdd6cf081fa7f/cc/surfaces/surface_aggregator.h
[modify] https://crrev.com/5222b6cb6ea4cb34ce34a6b8572bdd6cf081fa7f/cc/surfaces/surface_aggregator_perftest.cc
[modify] https://crrev.com/5222b6cb6ea4cb34ce34a6b8572bdd6cf081fa7f/cc/surfaces/surface_aggregator_unittest.cc
[modify] https://crrev.com/5222b6cb6ea4cb34ce34a6b8572bdd6cf081fa7f/cc/surfaces/surface_display_output_surface.cc
[modify] https://crrev.com/5222b6cb6ea4cb34ce34a6b8572bdd6cf081fa7f/cc/surfaces/surface_factory.cc
[modify] https://crrev.com/5222b6cb6ea4cb34ce34a6b8572bdd6cf081fa7f/cc/surfaces/surface_factory.h
[modify] https://crrev.com/5222b6cb6ea4cb34ce34a6b8572bdd6cf081fa7f/cc/surfaces/surface_factory_unittest.cc
[modify] https://crrev.com/5222b6cb6ea4cb34ce34a6b8572bdd6cf081fa7f/cc/surfaces/surface_hittest.cc
[modify] https://crrev.com/5222b6cb6ea4cb34ce34a6b8572bdd6cf081fa7f/cc/surfaces/surface_hittest_unittest.cc
[modify] https://crrev.com/5222b6cb6ea4cb34ce34a6b8572bdd6cf081fa7f/cc/surfaces/surfaces_pixeltest.cc
[modify] https://crrev.com/5222b6cb6ea4cb34ce34a6b8572bdd6cf081fa7f/cc/test/pixel_test_delegating_output_surface.cc
[modify] https://crrev.com/5222b6cb6ea4cb34ce34a6b8572bdd6cf081fa7f/components/exo/surface.cc
[modify] https://crrev.com/5222b6cb6ea4cb34ce34a6b8572bdd6cf081fa7f/components/exo/surface_unittest.cc
[modify] https://crrev.com/5222b6cb6ea4cb34ce34a6b8572bdd6cf081fa7f/components/mus/gpu/display_compositor/compositor_frame_sink_impl.cc
[modify] https://crrev.com/5222b6cb6ea4cb34ce34a6b8572bdd6cf081fa7f/components/mus/surfaces/display_compositor.cc
[modify] https://crrev.com/5222b6cb6ea4cb34ce34a6b8572bdd6cf081fa7f/components/mus/surfaces/display_compositor.h
[modify] https://crrev.com/5222b6cb6ea4cb34ce34a6b8572bdd6cf081fa7f/components/mus/ws/platform_display.cc
[modify] https://crrev.com/5222b6cb6ea4cb34ce34a6b8572bdd6cf081fa7f/components/mus/ws/platform_display.h
[modify] https://crrev.com/5222b6cb6ea4cb34ce34a6b8572bdd6cf081fa7f/components/mus/ws/server_window_surface.cc
[modify] https://crrev.com/5222b6cb6ea4cb34ce34a6b8572bdd6cf081fa7f/content/browser/frame_host/render_widget_host_view_child_frame.cc
[modify] https://crrev.com/5222b6cb6ea4cb34ce34a6b8572bdd6cf081fa7f/content/browser/frame_host/render_widget_host_view_guest.cc
[modify] https://crrev.com/5222b6cb6ea4cb34ce34a6b8572bdd6cf081fa7f/content/browser/renderer_host/delegated_frame_host.cc
[modify] https://crrev.com/5222b6cb6ea4cb34ce34a6b8572bdd6cf081fa7f/content/browser/renderer_host/render_widget_host_view_android.cc

Status: Fixed (was: Assigned)
This is essentially done, although there's a lot of follow up bugs that I'll file separately.
Blocking:
Components: -MUS Internals>Services>WindowService

Sign in to add a comment