Unverified Commit 130944e7 authored by yaakovschectman's avatar yaakovschectman Committed by GitHub

Alert engine upon registering ServiceBinding (#126075)

Send a platform message to the engine when the `ServiceBinding` is
registered. Framework side of
https://github.com/flutter/engine/pull/41733

Addresses https://github.com/flutter/flutter/issues/126033

## Pre-launch Checklist

- [x] I read the [Contributor Guide] and followed the process outlined
there for submitting PRs.
- [x] I read the [Tree Hygiene] wiki page, which explains my
responsibilities.
- [x] I read and followed the [Flutter Style Guide], including [Features
we expect every widget to implement].
- [ ] I signed the [CLA].
- [x] I listed at least one issue that this PR fixes in the description
above.
- [x] I updated/added relevant documentation (doc comments with `///`).
- [ ] I added new tests to check the change I am making, or this PR is
[test-exempt].
- [ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel
on [Discord].

<!-- Links -->
[Contributor Guide]:
https://github.com/flutter/flutter/wiki/Tree-hygiene#overview
[Tree Hygiene]: https://github.com/flutter/flutter/wiki/Tree-hygiene
[test-exempt]:
https://github.com/flutter/flutter/wiki/Tree-hygiene#tests
[Flutter Style Guide]:
https://github.com/flutter/flutter/wiki/Style-guide-for-Flutter-repo
[Features we expect every widget to implement]:
https://github.com/flutter/flutter/wiki/Style-guide-for-Flutter-repo#features-we-expect-every-widget-to-implement
[CLA]: https://cla.developers.google.com/
[flutter/tests]: https://github.com/flutter/tests
[breaking change policy]:
https://github.com/flutter/flutter/wiki/Tree-hygiene#handling-breaking-changes
[Discord]: https://github.com/flutter/flutter/wiki/Chat

---------
Co-authored-by: 's avatarGreg Spencer <gspencergoog@users.noreply.github.com>
parent cc44359b
...@@ -45,6 +45,7 @@ mixin ServicesBinding on BindingBase, SchedulerBinding { ...@@ -45,6 +45,7 @@ mixin ServicesBinding on BindingBase, SchedulerBinding {
SystemChannels.platform.setMethodCallHandler(_handlePlatformMessage); SystemChannels.platform.setMethodCallHandler(_handlePlatformMessage);
TextInput.ensureInitialized(); TextInput.ensureInitialized();
readInitialLifecycleStateFromNativeWindow(); readInitialLifecycleStateFromNativeWindow();
initializationComplete();
} }
/// The current [ServicesBinding], if one has been created. /// The current [ServicesBinding], if one has been created.
...@@ -415,6 +416,14 @@ mixin ServicesBinding on BindingBase, SchedulerBinding { ...@@ -415,6 +416,14 @@ mixin ServicesBinding on BindingBase, SchedulerBinding {
void setSystemUiChangeCallback(SystemUiChangeCallback? callback) { void setSystemUiChangeCallback(SystemUiChangeCallback? callback) {
_systemUiChangeCallback = callback; _systemUiChangeCallback = callback;
} }
/// Alert the engine that the binding is registered. This instructs the engine to
/// register its top level window handler on Windows. This signals that the app
/// is able to process "System.requestAppExit" signals from the engine.
@protected
Future<void> initializationComplete() async {
await SystemChannels.platform.invokeMethod('System.initializationComplete');
}
} }
/// Signature for listening to changes in the [SystemUiMode]. /// Signature for listening to changes in the [SystemUiMode].
......
...@@ -133,6 +133,10 @@ abstract final class SystemChannels { ...@@ -133,6 +133,10 @@ abstract final class SystemChannels {
/// * `System.requestAppExit`: The application has requested that it be /// * `System.requestAppExit`: The application has requested that it be
/// terminated. See [ServicesBinding.exitApplication]. /// terminated. See [ServicesBinding.exitApplication].
/// ///
/// * `System.initializationComplete`: Indicate to the engine the
/// initialization of a binding that may, among other tasks, register a
/// handler for application exit attempts.
///
/// Calls to methods that are not implemented on the shell side are ignored /// Calls to methods that are not implemented on the shell side are ignored
/// (so it is safe to call methods when the relevant plugin might be missing). /// (so it is safe to call methods when the relevant plugin might be missing).
static const MethodChannel platform = OptionalMethodChannel( static const MethodChannel platform = OptionalMethodChannel(
......
// Copyright 2014 The Flutter Authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
import 'package:flutter/foundation.dart';
import 'package:flutter/scheduler.dart';
import 'package:flutter/services.dart';
import 'package:flutter_test/flutter_test.dart';
class _TestBinding extends BindingBase with SchedulerBinding, ServicesBinding {
@override
Future<void> initializationComplete() async {
return super.initializationComplete();
}
@override
TestDefaultBinaryMessenger get defaultBinaryMessenger => super.defaultBinaryMessenger as TestDefaultBinaryMessenger;
@override
TestDefaultBinaryMessenger createBinaryMessenger() {
Future<ByteData?> keyboardHandler(ByteData? message) async {
return const StandardMethodCodec().encodeSuccessEnvelope(<int, int>{1:1});
}
return TestDefaultBinaryMessenger(
super.createBinaryMessenger(),
outboundHandlers: <String, MessageHandler>{'flutter/keyboard': keyboardHandler},
);
}
}
void main() {
final _TestBinding binding = _TestBinding();
test('can send message on completion of binding initialization', () async {
bool called = false;
binding.defaultBinaryMessenger.setMockMethodCallHandler(SystemChannels.platform, (MethodCall method) async {
if (method.method == 'System.initializationComplete') {
called = true;
}
return null;
});
await binding.initializationComplete();
expect(called, isTrue);
});
}
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment