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

Issue 1983453002: Dart: Refactor Stubs (Closed)

Created:
4 years, 7 months ago by zra
Modified:
4 years, 7 months ago
Reviewers:
Cutch
CC:
mojo-reviews_chromium.org, gregsimon, qsr+mojo_chromium.org, viettrungluu+watch_chromium.org, abarth-chromium, Aaron Boodman, darin (slow to review), ben+mojo_chromium.org, yzshen+mojopublicwatch_chromium.org
Base URL:
git@github.com:domokit/mojo.git@master
Target Ref:
refs/heads/master
Project:
mojo
Visibility:
Public.

Description

Dart: Refactor Stubs This change refactors Stubs to be more similar to Proxies. In particular, after this change, both FooStub and FooProxy will implement the Foo interface. After this change there are also more opportunities for Stubs and Proxies to share interfaces, which will allow both client and internal code to avoid using Object and dynamic, and I'll pursue those in a follow up CL. As with the change for Proxies, this is a breaking change that will require indirection through the .ctrl field of a Stub to manipulate the underlying MojoMessageHandler. There are no other user-visible changes. related #647 R=johnmccutchan@google.com Committed: https://chromium.googlesource.com/external/mojo/+/48993a8a843ad4b7ee14c6cb42dd7f7497f25f20

Patch Set 1 #

Patch Set 2 : Cleanup #

Patch Set 3 : Cleanup #

Patch Set 4 : Fix test #

Patch Set 5 : #

Patch Set 6 : Merge #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6955 lines, -2780 lines) Patch
M benchmarks/mojo_rtt_benchmark/lib/echo_server.dart View 1 chunk +1 line, -1 line 0 comments Download
M mojo/dart/apptests/test_apps/echo/lib/main.dart View 1 chunk +1 line, -1 line 0 comments Download
M mojo/dart/apptests/test_apps/pingpong/lib/main.dart View 2 chunks +2 lines, -2 lines 0 comments Download
M mojo/dart/packages/_mojo_for_test_only/lib/dart_to_cpp/dart_to_cpp.mojom.dart View 1 2 3 4 12 chunks +101 lines, -38 lines 0 comments Download
M mojo/dart/packages/_mojo_for_test_only/lib/imported/sample_import.mojom.dart View 1 2 6 chunks +43 lines, -19 lines 0 comments Download
M mojo/dart/packages/_mojo_for_test_only/lib/math/math_calculator.mojom.dart View 1 2 6 chunks +49 lines, -19 lines 0 comments Download
M mojo/dart/packages/_mojo_for_test_only/lib/mojo/examples/echo.mojom.dart View 1 2 6 chunks +43 lines, -19 lines 0 comments Download
M mojo/dart/packages/_mojo_for_test_only/lib/mojo/test/test_structs.mojom.dart View 1 2 3 4 6 chunks +43 lines, -19 lines 0 comments Download
M mojo/dart/packages/_mojo_for_test_only/lib/mojo/test/test_unions.mojom.dart View 1 2 3 4 12 chunks +89 lines, -38 lines 0 comments Download
M mojo/dart/packages/_mojo_for_test_only/lib/mojo/test/validation_test_interfaces.mojom.dart View 1 2 3 4 24 chunks +217 lines, -76 lines 0 comments Download
M mojo/dart/packages/_mojo_for_test_only/lib/mojo/test/versioning/versioning_test_client.mojom.dart View 1 2 6 chunks +52 lines, -19 lines 0 comments Download
M mojo/dart/packages/_mojo_for_test_only/lib/regression_tests/regression_tests.mojom.dart View 1 2 3 4 36 chunks +261 lines, -114 lines 0 comments Download
M mojo/dart/packages/_mojo_for_test_only/lib/sample/sample_factory.mojom.dart View 1 2 3 4 12 chunks +101 lines, -38 lines 0 comments Download
M mojo/dart/packages/_mojo_for_test_only/lib/sample/sample_interfaces.mojom.dart View 1 2 18 chunks +150 lines, -57 lines 0 comments Download
M mojo/dart/packages/_mojo_for_test_only/lib/sample/sample_service.mojom.dart View 1 2 3 4 12 chunks +89 lines, -38 lines 0 comments Download
M mojo/dart/packages/_mojo_for_test_only/lib/test/echo_service.mojom.dart View 1 2 6 chunks +52 lines, -19 lines 0 comments Download
M mojo/dart/packages/_mojo_for_test_only/lib/test/pingpong_service.mojom.dart View 1 2 3 4 12 chunks +104 lines, -38 lines 0 comments Download
M mojo/dart/packages/mojo/lib/mojo/application.mojom.dart View 1 2 3 4 6 chunks +49 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo/lib/mojo/application_connector.mojom.dart View 1 2 3 4 6 chunks +46 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo/lib/mojo/bindings/types/service_describer.mojom.dart View 1 2 3 4 12 chunks +92 lines, -38 lines 0 comments Download
M mojo/dart/packages/mojo/lib/mojo/service_provider.mojom.dart View 1 2 6 chunks +43 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo/lib/mojo/shell.mojom.dart View 1 2 3 4 6 chunks +46 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo/lib/src/application.dart View 2 chunks +4 lines, -4 lines 0 comments Download
M mojo/dart/packages/mojo/lib/src/application_connection.dart View 1 chunk +2 lines, -2 lines 0 comments Download
M mojo/dart/packages/mojo/lib/src/codec.dart View 1 2 3 4 2 chunks +8 lines, -8 lines 0 comments Download
M mojo/dart/packages/mojo/lib/src/control_message.dart View 3 chunks +10 lines, -10 lines 0 comments Download
M mojo/dart/packages/mojo/lib/src/proxy.dart View 1 2 3 4 3 chunks +4 lines, -3 lines 0 comments Download
M mojo/dart/packages/mojo/lib/src/stub.dart View 1 2 3 4 3 chunks +70 lines, -31 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/activity/activity.mojom.dart View 1 2 3 4 18 chunks +153 lines, -57 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/authentication/authentication.mojom.dart View 1 2 6 chunks +55 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/contacts/contacts.mojom.dart View 1 2 6 chunks +52 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/http_server/http_server.mojom.dart View 1 2 3 4 12 chunks +89 lines, -38 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/http_server/http_server_factory.mojom.dart View 1 2 3 4 6 chunks +43 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/icu_data/icu_data.mojom.dart View 1 2 6 chunks +43 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/input/input.mojom.dart View 1 2 3 4 12 chunks +86 lines, -38 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/keyboard/keyboard.mojom.dart View 1 2 3 4 18 chunks +162 lines, -57 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/asset_bundle/asset_bundle.mojom.dart View 1 2 3 4 12 chunks +86 lines, -38 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/authenticating_url_loader_interceptor_meta_factory.mojom.dart View 1 2 3 4 6 chunks +43 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/camera.mojom.dart View 1 2 12 chunks +92 lines, -38 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/clipboard.mojom.dart View 1 2 6 chunks +52 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/content_handler.mojom.dart View 1 2 3 4 6 chunks +43 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/cookie_store.mojom.dart View 1 2 6 chunks +46 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/device_info.mojom.dart View 1 2 6 chunks +43 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/files/directory.mojom.dart View 1 2 3 4 6 chunks +61 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/files/file.mojom.dart View 1 2 3 4 6 chunks +82 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/files/files.mojom.dart View 1 2 3 4 6 chunks +43 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/geocoder.mojom.dart View 1 2 6 chunks +46 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/gfx/composition/compositor.mojom.dart View 1 2 3 4 6 chunks +46 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/gfx/composition/hit_tests.mojom.dart View 1 2 6 chunks +43 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/gfx/composition/renderers.mojom.dart View 1 2 3 4 6 chunks +49 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/gfx/composition/resources.mojom.dart View 1 2 3 4 6 chunks +43 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/gfx/composition/scenes.mojom.dart View 1 2 3 4 12 chunks +95 lines, -38 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/gfx/composition/scheduling.mojom.dart View 1 2 6 chunks +43 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/host_resolver.mojom.dart View 1 2 6 chunks +43 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/http_connection.mojom.dart View 1 2 3 4 12 chunks +92 lines, -38 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/http_server.mojom.dart View 1 2 3 4 6 chunks +43 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/location_service.mojom.dart View 1 2 6 chunks +43 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/media/audio_server.mojom.dart View 1 2 3 4 6 chunks +43 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/media/audio_track.mojom.dart View 1 2 3 4 6 chunks +52 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/media/media_demux.mojom.dart View 1 2 3 4 6 chunks +58 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/media/media_factory.mojom.dart View 1 2 3 4 6 chunks +58 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/media/media_player.mojom.dart View 1 2 6 chunks +52 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/media/media_sink.mojom.dart View 1 2 3 4 6 chunks +52 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/media/media_source.mojom.dart View 1 2 3 4 6 chunks +64 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/media/media_transport.mojom.dart View 1 2 3 4 18 chunks +147 lines, -57 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/media/media_type_converter.mojom.dart View 1 2 3 4 6 chunks +49 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/media/rate_control.mojom.dart View 1 2 6 chunks +61 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/media/seeking_reader.mojom.dart View 1 2 6 chunks +46 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/media/timeline_controller.mojom.dart View 1 2 3 4 12 chunks +98 lines, -38 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/native_viewport_event_dispatcher.mojom.dart View 1 2 6 chunks +43 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/navigation.mojom.dart View 1 2 6 chunks +49 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/network_service.mojom.dart View 1 2 3 4 6 chunks +67 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/service_registry.mojom.dart View 1 2 3 4 6 chunks +43 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/sharing.mojom.dart View 1 2 6 chunks +43 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/surfaces.mojom.dart View 1 2 3 4 12 chunks +98 lines, -38 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/tcp_bound_socket.mojom.dart View 1 2 3 4 6 chunks +46 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/tcp_connected_socket.mojom.dart View 1 2 6 chunks +40 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/tcp_server_socket.mojom.dart View 1 2 3 4 6 chunks +43 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/terminal/terminal.mojom.dart View 1 2 3 4 6 chunks +52 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/terminal/terminal_client.mojom.dart View 1 2 3 4 6 chunks +43 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/timelines.mojom.dart View 1 2 6 chunks +43 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/udp_socket.mojom.dart View 1 2 3 4 12 chunks +107 lines, -38 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/ui/input_connection.mojom.dart View 1 2 3 4 12 chunks +86 lines, -38 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/ui/input_dispatcher.mojom.dart View 1 2 6 chunks +43 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/ui/view_associates.mojom.dart View 1 2 3 4 12 chunks +95 lines, -38 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/ui/view_containers.mojom.dart View 1 2 3 4 12 chunks +98 lines, -38 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/ui/view_manager.mojom.dart View 1 2 3 4 6 chunks +46 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/ui/view_provider.mojom.dart View 1 2 3 4 6 chunks +43 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/ui/view_token.mojom.dart View 1 2 6 chunks +43 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/ui/view_trees.mojom.dart View 1 2 3 4 12 chunks +95 lines, -38 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/ui/views.mojom.dart View 1 2 3 4 12 chunks +95 lines, -38 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/url_loader.mojom.dart View 1 2 6 chunks +49 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/url_loader_interceptor.mojom.dart View 1 2 3 4 12 chunks +92 lines, -38 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/url_response_disk_cache.mojom.dart View 1 2 6 chunks +55 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/mojo/web_socket.mojom.dart View 1 2 3 4 12 chunks +107 lines, -38 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/native_support/process.mojom.dart View 1 2 3 4 12 chunks +92 lines, -38 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/nfc/nfc.mojom.dart View 1 2 3 4 18 chunks +135 lines, -57 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/notifications/notifications.mojom.dart View 1 2 3 4 18 chunks +135 lines, -57 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/prediction/prediction.mojom.dart View 1 2 6 chunks +43 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/sensors/sensors.mojom.dart View 1 2 3 4 12 chunks +89 lines, -38 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/speech_recognizer/speech_recognizer.mojom.dart View 1 2 3 4 12 chunks +95 lines, -38 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/tracing/trace_provider_registry.mojom.dart View 1 2 3 4 6 chunks +43 lines, -19 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/tracing/tracing.mojom.dart View 1 2 3 4 18 chunks +135 lines, -57 lines 0 comments Download
M mojo/dart/packages/mojo_services/lib/vsync/vsync.mojom.dart View 1 2 6 chunks +43 lines, -19 lines 0 comments Download
M mojo/dart/unittests/embedder_tests/validation_test.dart View 1 2 3 1 chunk +1 line, -1 line 0 comments Download
M mojo/public/tools/bindings/generators/dart_templates/interface_definition.tmpl View 1 6 chunks +66 lines, -19 lines 0 comments Download
M mojo/public/tools/bindings/generators/mojom_dart_generator.py View 1 2 3 4 1 chunk +2 lines, -0 lines 0 comments Download

Messages

Total messages: 7 (4 generated)
zra
4 years, 7 months ago (2016-05-16 17:03:31 UTC) #4
Cutch
LGTM with a breaking-change announcement
4 years, 7 months ago (2016-05-16 21:38:55 UTC) #5
zra
4 years, 7 months ago (2016-05-17 15:47:54 UTC) #7
Message was sent while issue was closed.
Committed patchset #6 (id:100001) manually as
48993a8a843ad4b7ee14c6cb42dd7f7497f25f20 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698