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

Unified Diff: mojo/dart/packages/mojo_services/lib/mojo/ui/input_connection.mojom.dart

Issue 2006093002: Dart: Futures -> Callbacks. (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: Merge Created 4 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: mojo/dart/packages/mojo_services/lib/mojo/ui/input_connection.mojom.dart
diff --git a/mojo/dart/packages/mojo_services/lib/mojo/ui/input_connection.mojom.dart b/mojo/dart/packages/mojo_services/lib/mojo/ui/input_connection.mojom.dart
index 84974d360b8b3f11631aa25a207e3ca59ba0c07f..18cd5f45a2c78a6ff30e0a504286e5ff7099dbfe 100644
--- a/mojo/dart/packages/mojo_services/lib/mojo/ui/input_connection.mojom.dart
+++ b/mojo/dart/packages/mojo_services/lib/mojo/ui/input_connection.mojom.dart
@@ -18,6 +18,10 @@ class _InputConnectionSetListenerParams extends bindings.Struct {
_InputConnectionSetListenerParams() : super(kVersions.last.size);
+ _InputConnectionSetListenerParams.init(
+ InputListenerInterface this.listener
+ ) : super(kVersions.last.size);
+
static _InputConnectionSetListenerParams deserialize(bindings.Message message) {
var decoder = new bindings.Decoder(message);
var result = decode(decoder);
@@ -89,6 +93,10 @@ class _InputListenerOnEventParams extends bindings.Struct {
_InputListenerOnEventParams() : super(kVersions.last.size);
+ _InputListenerOnEventParams.init(
+ input_events_mojom.Event this.event
+ ) : super(kVersions.last.size);
+
static _InputListenerOnEventParams deserialize(bindings.Message message) {
var decoder = new bindings.Decoder(message);
var result = decode(decoder);
@@ -162,6 +170,10 @@ class InputListenerOnEventResponseParams extends bindings.Struct {
InputListenerOnEventResponseParams() : super(kVersions.last.size);
+ InputListenerOnEventResponseParams.init(
+ bool this.consumed
+ ) : super(kVersions.last.size);
+
static InputListenerOnEventResponseParams deserialize(bindings.Message message) {
var decoder = new bindings.Decoder(message);
var result = decode(decoder);
@@ -228,14 +240,17 @@ class InputListenerOnEventResponseParams extends bindings.Struct {
const int _inputConnectionMethodSetListenerName = 0;
class _InputConnectionServiceDescription implements service_describer.ServiceDescription {
- dynamic getTopLevelInterface([Function responseFactory]) =>
- responseFactory(null);
+ void getTopLevelInterface(Function responder) {
+ responder(null);
+ }
- dynamic getTypeDefinition(String typeKey, [Function responseFactory]) =>
- responseFactory(null);
+ void getTypeDefinition(String typeKey, Function responder) {
+ responder(null);
+ }
- dynamic getAllTypeDefinitions([Function responseFactory]) =>
- responseFactory(null);
+ void getAllTypeDefinitions(Function responder) {
+ responder(null);
+ }
}
abstract class InputConnection {
@@ -385,11 +400,11 @@ class _InputConnectionStubControl
- dynamic handleMessage(bindings.ServiceMessage message) {
+ void handleMessage(bindings.ServiceMessage message) {
if (bindings.ControlMessageHandler.isControlMessage(message)) {
- return bindings.ControlMessageHandler.handleMessage(this,
- 0,
- message);
+ bindings.ControlMessageHandler.handleMessage(
+ this, 0, message);
+ return;
}
if (_impl == null) {
throw new core.MojoApiError("$this has no implementation set");
@@ -404,7 +419,6 @@ class _InputConnectionStubControl
throw new bindings.MojoCodecError("Unexpected message name");
break;
}
- return null;
}
InputConnection get impl => _impl;
@@ -466,14 +480,17 @@ class InputConnectionStub
const int _inputListenerMethodOnEventName = 0;
class _InputListenerServiceDescription implements service_describer.ServiceDescription {
- dynamic getTopLevelInterface([Function responseFactory]) =>
- responseFactory(null);
+ void getTopLevelInterface(Function responder) {
+ responder(null);
+ }
- dynamic getTypeDefinition(String typeKey, [Function responseFactory]) =>
- responseFactory(null);
+ void getTypeDefinition(String typeKey, Function responder) {
+ responder(null);
+ }
- dynamic getAllTypeDefinitions([Function responseFactory]) =>
- responseFactory(null);
+ void getAllTypeDefinitions(Function responder) {
+ responder(null);
+ }
}
abstract class InputListener {
@@ -498,7 +515,7 @@ abstract class InputListener {
s.connectToService(url, p, name);
return p;
}
- dynamic onEvent(input_events_mojom.Event event,[Function responseFactory = null]);
+ void onEvent(input_events_mojom.Event event,void callback(bool consumed));
}
abstract class InputListenerInterface
@@ -548,18 +565,14 @@ class _InputListenerProxyControl
proxyError("Expected a message with a valid request Id.");
return;
}
- Completer c = completerMap[message.header.requestId];
- if (c == null) {
+ Function callback = callbackMap[message.header.requestId];
+ if (callback == null) {
proxyError(
"Message had unknown request Id: ${message.header.requestId}");
return;
}
- completerMap.remove(message.header.requestId);
- if (c.isCompleted) {
- proxyError("Response completer already completed");
- return;
- }
- c.complete(r);
+ callbackMap.remove(message.header.requestId);
+ callback(r.consumed );
break;
default:
proxyError("Unexpected message type: ${message.header.type}");
@@ -604,17 +617,19 @@ class InputListenerProxy
}
- dynamic onEvent(input_events_mojom.Event event,[Function responseFactory = null]) {
+ void onEvent(input_events_mojom.Event event,void callback(bool consumed)) {
if (impl != null) {
- return new Future(() => impl.onEvent(event,_InputListenerStubControl._inputListenerOnEventResponseParamsFactory));
+ impl.onEvent(event,callback);
+ return;
}
var params = new _InputListenerOnEventParams();
params.event = event;
- return ctrl.sendMessageWithRequestId(
+ ctrl.sendMessageWithRequestId(
params,
_inputListenerMethodOnEventName,
-1,
- bindings.MessageHeader.kMessageExpectsResponse);
+ bindings.MessageHeader.kMessageExpectsResponse,
+ callback);
}
}
@@ -640,17 +655,24 @@ class _InputListenerStubControl
String get serviceName => InputListener.serviceName;
- static InputListenerOnEventResponseParams _inputListenerOnEventResponseParamsFactory(bool consumed) {
- var result = new InputListenerOnEventResponseParams();
- result.consumed = consumed;
- return result;
+ Function _inputListenerOnEventResponseParamsResponder(
+ int requestId) {
+ return (bool consumed) {
+ var result = new InputListenerOnEventResponseParams();
+ result.consumed = consumed;
+ sendResponse(buildResponseWithId(
+ result,
+ _inputListenerMethodOnEventName,
+ requestId,
+ bindings.MessageHeader.kMessageIsResponse));
+ };
}
- dynamic handleMessage(bindings.ServiceMessage message) {
+ void handleMessage(bindings.ServiceMessage message) {
if (bindings.ControlMessageHandler.isControlMessage(message)) {
- return bindings.ControlMessageHandler.handleMessage(this,
- 0,
- message);
+ bindings.ControlMessageHandler.handleMessage(
+ this, 0, message);
+ return;
}
if (_impl == null) {
throw new core.MojoApiError("$this has no implementation set");
@@ -659,30 +681,12 @@ class _InputListenerStubControl
case _inputListenerMethodOnEventName:
var params = _InputListenerOnEventParams.deserialize(
message.payload);
- var response = _impl.onEvent(params.event,_inputListenerOnEventResponseParamsFactory);
- if (response is Future) {
- return response.then((response) {
- if (response != null) {
- return buildResponseWithId(
- response,
- _inputListenerMethodOnEventName,
- message.header.requestId,
- bindings.MessageHeader.kMessageIsResponse);
- }
- });
- } else if (response != null) {
- return buildResponseWithId(
- response,
- _inputListenerMethodOnEventName,
- message.header.requestId,
- bindings.MessageHeader.kMessageIsResponse);
- }
+ _impl.onEvent(params.event, _inputListenerOnEventResponseParamsResponder(message.header.requestId));
break;
default:
throw new bindings.MojoCodecError("Unexpected message name");
break;
}
- return null;
}
InputListener get impl => _impl;
@@ -736,8 +740,8 @@ class InputListenerStub
}
- dynamic onEvent(input_events_mojom.Event event,[Function responseFactory = null]) {
- return impl.onEvent(event,responseFactory);
+ void onEvent(input_events_mojom.Event event,void callback(bool consumed)) {
+ return impl.onEvent(event,callback);
}
}

Powered by Google App Engine
This is Rietveld 408576698